-
Notifications
You must be signed in to change notification settings - Fork 32
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: add test provider #971
Conversation
Signed-off-by: Todd Baert <[email protected]>
Signed-off-by: Todd Baert <[email protected]>
Signed-off-by: Todd Baert <[email protected]>
I recommend we merge #972 first. |
Signed-off-by: Todd Baert <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm...
... but as i am not so familiar with the react sdk, do we have a concept of domains also here? and if we do, how could i set it up with the testing provider? More out of curiosity, rather than we should tackle this wihtin this pr. i do think that this adds a lot of benefits to the users of the lib
The test provider supports domains (it inherits all options from the normal provider, including |
Signed-off-by: Todd Baert <[email protected]>
c2f789b
to
6acbbfd
Compare
Signed-off-by: Todd Baert <[email protected]>
🤖 I have created a release *beep* *boop* --- ## [0.4.2](react-sdk-v0.4.1...react-sdk-v0.4.2) (2024-07-29) ### ✨ New Features * add test provider ([#971](#971)) ([1c12d4d](1c12d4d)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please). Signed-off-by: OpenFeature Bot <[email protected]>
Adds some testing utilities, specifically an
<OpenFeatureTestProvider/>
react context provider.See README for details.