test(e2e): add Behave step definitions for context merging#611
test(e2e): add Behave step definitions for context merging#611karnyadavdev wants to merge 2 commits into
Conversation
Closes open-feature#593 Signed-off-by: Karan Yadav <47368518+karnyadavdev@users.noreply.github.com>
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request introduces behavior-driven development (BDD) test steps and environment setup for testing OpenFeature context merging across various levels (API, Transaction, Client, Invocation, and Before Hooks). The feedback suggests improving the step definitions by leveraging the built-in merge method of EvaluationContext to reduce duplication, and adding defensive checks for context.precedence_levels and context.provider to prevent unhelpful AttributeError exceptions if the tests are executed out of order.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
Signed-off-by: Karan Yadav <47368518+karnyadavdev@users.noreply.github.com>
|
review and merge this |
1 participant
Taking over PR #593 to add the required DCO signatures.
Closes #593