Skip to content

feat: AccountStateProvider for testing#265

Merged
fabriziodemaria merged 3 commits intomainfrom
state-provider
Sep 17, 2025
Merged

feat: AccountStateProvider for testing#265
fabriziodemaria merged 3 commits intomainfrom
state-provider

Conversation

@fabriziodemaria
Copy link
Copy Markdown
Member

@fabriziodemaria fabriziodemaria commented Sep 16, 2025

Adds constructor to Local Resolver:
OpenFeatureLocalResolveProvider(AccountStateProvider accountStateProvider, String clientSecret)

This can be used for testing purposes: the state can be supplied in a custom way and no production logging is generated (no network call at all needed for testing pure resolves)

Copy link
Copy Markdown
Member

@nicklasl nicklasl left a comment

Choose a reason for hiding this comment

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

LGTM

@fabriziodemaria fabriziodemaria merged commit beb21f5 into main Sep 17, 2025
3 checks passed
@fabriziodemaria fabriziodemaria deleted the state-provider branch September 17, 2025 08:36
This was referenced Sep 17, 2025
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