Skip to content

[Phase 1] Fix Flaky Integration Tests + Migrate some tests to Test Containers#1741

Open
siri-varma wants to merge 7 commits into
dapr:masterfrom
siri-varma:users/svegiraju/fix-integ-tests
Open

[Phase 1] Fix Flaky Integration Tests + Migrate some tests to Test Containers#1741
siri-varma wants to merge 7 commits into
dapr:masterfrom
siri-varma:users/svegiraju/fix-integ-tests

Conversation

@siri-varma
Copy link
Copy Markdown
Contributor

@siri-varma siri-varma commented May 10, 2026

Description

Please explain the changes you've made

Issue reference

We strive to have all PR being opened based on an issue, where the problem or feature have been discussed prior to implementation.

Please reference the issue this PR will close:
#1732

Checklist

Please make sure you've completed the relevant tasks for this PR, out of the following list:

  • Code compiles correctly
  • Created/updated tests
  • Extended the documentation

@siri-varma siri-varma force-pushed the users/svegiraju/fix-integ-tests branch 4 times, most recently from 89fd3e0 to c01299d Compare May 10, 2026 22:16
@siri-varma
Copy link
Copy Markdown
Contributor Author

@javier-aliaga Would you know why this is failing ? did not touch any protos or xml files

@siri-varma siri-varma marked this pull request as ready for review May 11, 2026 14:34
@siri-varma siri-varma requested review from a team as code owners May 11, 2026 14:34
@javier-aliaga
Copy link
Copy Markdown
Contributor

@siri-varma , the proto files are generated from master dapr/dapr, every time they change something there it break our master. I already fixed that and it is merged to master

@siri-varma siri-varma force-pushed the users/svegiraju/fix-integ-tests branch 8 times, most recently from a9b4907 to d499f64 Compare May 14, 2026 19:07
Siri Varma Vegiraju and others added 5 commits May 14, 2026 12:10
Port the BookTrip compensation (Saga) workflow from the plain Java
examples into the Spring Boot workflow patterns module, adding
@Component-annotated activities and a /wfp/compensation REST endpoint.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Signed-off-by: Siri Varma Vegiraju <svegiraju@Siris-MacBook-Pro.local>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Signed-off-by: Siri Varma Vegiraju <siri.varma@outlook.com>
Signed-off-by: Siri Varma Vegiraju <siri.varma@outlook.com>
Signed-off-by: Siri Varma Vegiraju <siri.varma@outlook.com>
Signed-off-by: Siri Varma Vegiraju <siri.varma@outlook.com>
@siri-varma siri-varma force-pushed the users/svegiraju/fix-integ-tests branch from d499f64 to 8a7e0d0 Compare May 14, 2026 19:10
@siri-varma siri-varma changed the title Fix Flaky Integration Tests [Phase 1] Fix Flaky Integration Tests + Migrate some tests to Test Containers May 14, 2026
@siri-varma siri-varma force-pushed the users/svegiraju/fix-integ-tests branch 6 times, most recently from 90fa312 to 5c4bb55 Compare May 15, 2026 14:35
@siri-varma siri-varma force-pushed the users/svegiraju/fix-integ-tests branch 2 times, most recently from 2dea405 to a341b06 Compare May 15, 2026 15:09
Signed-off-by: Siri Varma Vegiraju <s_vegiraju@apple.com>
@siri-varma siri-varma force-pushed the users/svegiraju/fix-integ-tests branch from a341b06 to 6ba3643 Compare May 15, 2026 15:33
Signed-off-by: Siri Varma Vegiraju <siri.varma@outlook.com>
@codecov
Copy link
Copy Markdown

codecov Bot commented May 15, 2026

Codecov Report

❌ Patch coverage is 60.00000% with 4 lines in your changes missing coverage. Please review.
✅ Project coverage is 76.73%. Comparing base (89708ae) to head (fca2573).

Files with missing lines Patch % Lines
sdk/src/main/java/io/dapr/client/Subscription.java 60.00% 3 Missing and 1 partial ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##             master    #1741      +/-   ##
============================================
- Coverage     76.77%   76.73%   -0.04%     
- Complexity     2259     2260       +1     
============================================
  Files           241      241              
  Lines          7066     7076      +10     
  Branches        740      742       +2     
============================================
+ Hits           5425     5430       +5     
- Misses         1277     1282       +5     
  Partials        364      364              

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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