Skip to content

test: add 3 payroll edge-case tests#32

Open
EricGrill wants to merge 2 commits intomainfrom
justobot/heartbeat-justobot-mnkxtfxr
Open

test: add 3 payroll edge-case tests#32
EricGrill wants to merge 2 commits intomainfrom
justobot/heartbeat-justobot-mnkxtfxr

Conversation

@EricGrill
Copy link
Copy Markdown
Contributor

Summary

  • 404 for nonexistent shift — verifies payroll rejects a fake shift_id
  • 409 for open shift — verifies payroll requires shift to be closed first
  • worker_ids filter — verifies worker_ids param pays only specified workers

Payroll suite goes from 4 → 7 tests. Part of goal #13 (100% unit test coverage by April 15th).

Test plan

  • All 3 new tests pass locally
  • Pre-existing 49 tests unaffected (4 pre-existing rate-limiter failures unchanged)

🤖 Generated with Claude Code

EricGrill and others added 2 commits April 4, 2026 19:04
…filter)

Covers nonexistent shift, payroll on unclosed shift, and filtered
worker_ids payment — brings payroll suite from 4 to 7 tests.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant