Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix: consider default client when loading states #309

Merged
merged 5 commits into from
Dec 4, 2024

Conversation

warber
Copy link
Contributor

@warber warber commented Nov 28, 2024

This PR

fixes #308 and #310

Signed-off-by: Bernd Warmuth <bernd.warmuth@dynatrace.com>
@warber warber force-pushed the fix/load-state-default-fallback branch from 1cb57c5 to 1b76ca0 Compare November 28, 2024 17:45
Copy link

codecov bot commented Nov 28, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 86.91%. Comparing base (96d86ba) to head (19299de).
Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #309      +/-   ##
==========================================
+ Coverage   86.86%   86.91%   +0.05%     
==========================================
  Files          12       12              
  Lines        1370     1376       +6     
==========================================
+ Hits         1190     1196       +6     
  Misses        156      156              
  Partials       24       24              
Flag Coverage Δ
e2e 86.91% <100.00%> (+0.05%) ⬆️
unit 86.91% <100.00%> (+0.05%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Signed-off-by: Bernd Warmuth <bernd.warmuth@dynatrace.com>
@warber warber force-pushed the fix/load-state-default-fallback branch from a714c3e to 8dc46af Compare December 2, 2024 08:21
Bernd Warmuth added 2 commits December 2, 2024 11:18
Signed-off-by: Bernd Warmuth <bernd.warmuth@dynatrace.com>
Signed-off-by: Bernd Warmuth <bernd.warmuth@dynatrace.com>

Signed-off-by: Bernd Warmuth <bernd.warmuth@dynatrace.com>
@warber warber marked this pull request as ready for review December 2, 2024 10:33
@warber warber requested a review from a team as a code owner December 2, 2024 10:33
Copy link
Member

@toddbaert toddbaert left a comment

Choose a reason for hiding this comment

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

Just recommended a comment.

@warber warber changed the title fix: consider default client when loading states (wip) fix: consider default client when loading states Dec 4, 2024
Co-authored-by: Todd Baert <todd.baert@dynatrace.com>
Signed-off-by: warber <72415058+warber@users.noreply.github.com>
openfeature/client_test.go Show resolved Hide resolved
@toddbaert toddbaert merged commit d906d6f into open-feature:main Dec 4, 2024
8 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BUG] flaky unit test(s)
4 participants