Skip to content

Resolve unintended sync test skipping. #1600

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.

Sign up for GitHub

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

Merged
merged 2 commits into from
Jan 9, 2025
Merged

Resolve unintended sync test skipping. #1600

Changes from 1 commit
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
b31cbb3
Resolve unintended sync test skipping.
vbabanin Jan 9, 2025
3df2e41
Merge branch 'main' into JAVA-5718
vbabanin Jan 9, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Next Next commit
Resolve unintended sync test skipping. 
JAVA-5718
  • Loading branch information
@vbabanin
vbabanin committed Jan 9, 2025
commit b31cbb3c2d34d35ee87cf00fc56d5a5f95c76985
3 changes: 2 additions & 1 deletion driver-sync/src/test/functional/com/mongodb/client/unified/UnifiedTestModifications.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -323,7 +323,8 @@ public TestApplicator skipNoncompliant(final String reason) {
* @param reason reason for skipping the test
*/
public TestApplicator skipNoncompliantReactive(final String reason) {
return new TestApplicator(this, reason, SKIP);
return new TestApplicator(this, reason, SKIP)
.when(this::isReactive);
}

/**
Expand Down