Skip to content

Conversation

@lei-wego
Copy link
Collaborator

Summary

  • Add comprehensive table-driven tests for Error context handling functionality
  • Test context propagation between parent and child errors
  • Verify context override behavior and edge cases

Test plan

  • All existing tests pass
  • New tests cover context propagation scenarios
  • Tests verify child context cleanup after propagation

🤖 Generated with Claude Code

- Add table-driven tests for Error context functionality
- Test context propagation, overrides, and edge cases
- Verify child context cleanup after propagation

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

Co-Authored-By: Claude <[email protected]>
@lei-wego lei-wego changed the title Add context handling tests Add context handling Sep 22, 2025
- Add table-driven tests for context.Context parameter support in New()
- Test context propagation with child errors via New parameter
- Test mixed usage of New parameter and WithContext method
- Verify edge cases like empty contexts

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

Co-Authored-By: Claude <[email protected]>
@lei-wego lei-wego force-pushed the add-context-to-new-error branch from c79656b to 86f9aeb Compare September 22, 2025 09:18
yanyi-wego
yanyi-wego previously approved these changes Sep 22, 2025
@lei-wego lei-wego merged commit 4fa908d into main Sep 22, 2025
2 checks passed
@lei-wego lei-wego deleted the add-context-to-new-error branch September 22, 2025 09:49
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.

3 participants