Skip to content
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

Integration Tests: Adds injector #7520

Merged
merged 22 commits into from
Mar 26, 2024

Conversation

JoshVanL
Copy link
Contributor

@JoshVanL JoshVanL commented Feb 9, 2024

Adds support for injector process in integration tests.

Adds basic healthz & ports tests for injector.

Adds support for injector process in integration tests.

Adds basic healthz & ports tests for injector.

Signed-off-by: joshvanl <me@joshvanl.dev>
@JoshVanL JoshVanL requested review from a team as code owners February 9, 2024 13:30
Copy link

codecov bot commented Feb 9, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 62.17%. Comparing base (aa25dd9) to head (5980966).

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #7520      +/-   ##
==========================================
+ Coverage   62.13%   62.17%   +0.04%     
==========================================
  Files         245      245              
  Lines       22479    22479              
==========================================
+ Hits        13967    13976       +9     
+ Misses       7345     7338       -7     
+ Partials     1167     1165       -2     

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

@JoshVanL JoshVanL added the autoupdate DaprBot will keep the Pull Request up to date with master branch label Feb 12, 2024
Copy link
Contributor

@cicoyle cicoyle left a comment

Choose a reason for hiding this comment

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

LGTM 🚀

@cicoyle
Copy link
Contributor

cicoyle commented Feb 13, 2024

Looks like the macOS unit test failed due to: TestProxySuite - not sure if this is flaky?

@daixiang0 daixiang0 changed the title Integration Tests: Adds injector binary Integration Tests: Adds injector Feb 19, 2024
@JoshVanL JoshVanL added this to the v1.14 milestone Mar 5, 2024
@yaron2 yaron2 merged commit 7dda33a into dapr:master Mar 26, 2024
31 checks passed
cicoyle pushed a commit to cicoyle/dapr that referenced this pull request May 24, 2024
Adds support for injector process in integration tests.

Adds basic healthz & ports tests for injector.

Signed-off-by: joshvanl <me@joshvanl.dev>
Co-authored-by: Dapr Bot <56698301+dapr-bot@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
autoupdate DaprBot will keep the Pull Request up to date with master branch
Projects
Development

Successfully merging this pull request may close these issues.

None yet

5 participants