Skip to content

test: flagd harness updates#88

Merged
tcarrio merged 3 commits intomainfrom
test/flagd-harness-updates
Mar 11, 2023
Merged

test: flagd harness updates#88
tcarrio merged 3 commits intomainfrom
test/flagd-harness-updates

Conversation

@tcarrio
Copy link
Copy Markdown
Member

@tcarrio tcarrio commented Mar 11, 2023

This PR

  • updates integration test suite to the latest test-harness
  • ensures support for existing Flagd feature-set within provider
  • does NOT provide support for caching yet (which is not a concern of PHP SDK directly anyway)

Related Issues

Fixes #85

Notes

Follow-up Tasks

Incorporate caching support when added to Flagd provider

How to test

CI, or execute the integration test suite locally.

@tcarrio tcarrio changed the title Test/flagd harness updates test: flagd harness updates Mar 11, 2023
tcarrio added 2 commits March 10, 2023 22:06
Signed-off-by: Tom Carrio <tom@carrio.dev>
… feature

this does NOT include the caching feature

Signed-off-by: Tom Carrio <tom@carrio.dev>
@tcarrio tcarrio force-pushed the test/flagd-harness-updates branch from 48e8a50 to d7839a1 Compare March 11, 2023 03:06
@codecov
Copy link
Copy Markdown

codecov bot commented Mar 11, 2023

Codecov Report

Merging #88 (48e8a50) into main (4b55078) will not change coverage.
The diff coverage is n/a.

❗ Current head 48e8a50 differs from pull request most recent head c6ebab9. Consider uploading reports for the commit c6ebab9 to get more accurate results

@@            Coverage Diff            @@
##               main      #88   +/-   ##
=========================================
  Coverage     95.19%   95.19%           
  Complexity      227      227           
=========================================
  Files            40       40           
  Lines           583      583           
=========================================
  Hits            555      555           
  Misses           28       28           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

Signed-off-by: Tom Carrio <tom@carrio.dev>
@tcarrio tcarrio merged commit 9eb396c into main Mar 11, 2023
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.

[FEATURE] Latest test-harness

1 participant