Skip to content

Conversation

nthmost-orkes
Copy link
Contributor

Pull Request type

  • Bugfix
  • Feature
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • WHOSUSING.md
  • Other (please describe):

Fixes 4 test failures related to NotFoundException behavior changes:

  1. SQSEventQueueE2ESpec.groovy:420

    • Convert Groovy GString to String when passing to AWS SDK
    • Fixes ClassCastException in setQueueAttributes call
  2. QueueResiliencySpec.groovy:517 - getTaskLogs test

    • Add thrown(NotFoundException) expectation
    • TaskResource.getTaskLogs now throws NotFoundException for missing tasks
  3. QueueResiliencySpec.groovy:525 - getTask test

    • Add thrown(NotFoundException) expectation
    • TaskResource.getTask now throws NotFoundException for missing tasks
  4. AbstractHttpEndToEndTest.java:451 - testTaskByTaskId

Related to issue #602

No new functionality, just catching up on tech debt.

Fixes 4 test failures related to NotFoundException behavior changes:

1. SQSEventQueueE2ESpec.groovy:420
   - Convert Groovy GString to String when passing to AWS SDK
   - Fixes ClassCastException in setQueueAttributes call

2. QueueResiliencySpec.groovy:517 - getTaskLogs test
   - Add thrown(NotFoundException) expectation
   - TaskResource.getTaskLogs now throws NotFoundException for missing tasks

3. QueueResiliencySpec.groovy:525 - getTask test
   - Add thrown(NotFoundException) expectation
   - TaskResource.getTask now throws NotFoundException for missing tasks

4. AbstractHttpEndToEndTest.java:451 - testTaskByTaskId
   - Update error message assertion from "No such task found by" to "Task not found for"
   - Matches new error message format from PR #69

Related to issue #602

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <[email protected]>
@nthmost-orkes nthmost-orkes changed the title Fix test failures from PR #69 exception handling changes Fix test failures stemming from exception handling changes Oct 10, 2025
@nthmost-orkes nthmost-orkes requested review from c4lm and v1r3n October 10, 2025 23:39
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