Skip to content

Phase 3: verify error message in option tests#146

Merged
teresa-ortega merged 1 commit intonext-genfrom
teresa-ortega/fix/test_error_option
Apr 17, 2026
Merged

Phase 3: verify error message in option tests#146
teresa-ortega merged 1 commit intonext-genfrom
teresa-ortega/fix/test_error_option

Conversation

@teresa-ortega
Copy link
Copy Markdown
Collaborator

Proposed changes

Add a message validation to the test to verify the error message raised by the @option decorator when an invalid flag is declared, not just the exception type.

Type of change

  • 🐛 Bugfix (change which fixes an issue)
  • 🚀 Feature (change which adds functionality)
  • 📚 Documentation (change which fixes or extends documentation)

💥 No breaking changes.

Checklist

Put an x in the boxes that apply. This is simply a reminder of what we will require before merging your code.

  • Lint and unit tests (if any) pass locally with my changes
  • I have added tests that prove my fix is effective or that my feature works
  • I have added necessary documentation (if appropriate)

Additional comments

N/A

Signed-off-by: Maria Teresa Ortega <teresa.ortega0903@gmail.com>
@teresa-ortega teresa-ortega requested a review from xaru8145 April 17, 2026 11:04
Copy link
Copy Markdown
Collaborator

@xaru8145 xaru8145 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@teresa-ortega teresa-ortega merged commit cdad72e into next-gen Apr 17, 2026
4 checks passed
@teresa-ortega teresa-ortega deleted the teresa-ortega/fix/test_error_option branch April 17, 2026 12:50
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.

2 participants