---
id: w3/m9/t008
title: Test coverage — meaningful tests for the notification-firing + settings behavior
worker: worker3
status: todo
estimate: 30m
depends_on: [w3/m9/t006]
---

## Objective

Add meaningful tests asserting real notification-firing behavior and failure modes, not tautological coverage.

## Steps

1. Test that success/failure transitions fire exactly the expected notification when enabled, none when disabled.
2. Test that a mailer failure doesn't fail the deploy-status write (t002's isolation guarantee).
3. Test settings get/update on all three API surfaces.

## Acceptance criteria

- [ ] Tests cover the enabled/disabled branch and the mailer-failure isolation guarantee.
- [ ] Tests assert real behavior, not just "returns 200".
