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.

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

Do not perform server selection to determine sessions support #1092

Merged
merged 6 commits into from
Mar 9, 2023
Merged

Do not perform server selection to determine sessions support #1092

Changes from 1 commit
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
ddb336c
Do not perform server selection to determine sessions support
jyemin Mar 1, 2023
7e8951b
Fix tests
jyemin Mar 2, 2023
17a2c02
Fix tests
jyemin Mar 2, 2023
68c32ac
Checkstyle
jyemin Mar 2, 2023
2c421ff
Test updates
jyemin Mar 3, 2023
40e102b
Test updates
jyemin Mar 3, 2023
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
Prev Previous commit
Next Next commit
Fix tests 
JAVA-4860
  • Loading branch information
@jyemin
jyemin committed Mar 2, 2023
commit 7e8951b29b1320f3719bdd2fd894107018186a42
2 changes: 2 additions & 0 deletions driver-sync/src/test/functional/com/mongodb/client/AbstractSessionsProseTest.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -115,6 +115,7 @@ public void commandStarted(final CommandStartedEvent event) {

@Test
public void shouldIgnoreImplicitSessionIfConnectionDoesNotSupportSessions() throws IOException {
jyemin marked this conversation as resolved.
Show resolved Hide resolved
assumeTrue(serverVersionAtLeast(4, 2));
katcharov marked this conversation as resolved.
Show resolved Hide resolved
Process mongocryptdProcess = startMongocryptdProcess();
try {
AtomicBoolean containsLsid = new AtomicBoolean(true);
katcharov marked this conversation as resolved.
Show resolved Hide resolved
Expand Down Expand Up @@ -143,6 +144,7 @@ public void commandStarted(final CommandStartedEvent event) {

@Test
public void shouldThrowOnExplicitSessionIfConnectionDoesNotSupportSessions() throws IOException {
katcharov marked this conversation as resolved.
Show resolved Hide resolved
assumeTrue(serverVersionAtLeast(4, 2));
Process mongocryptdProcess = startMongocryptdProcess();
try {
katcharov marked this conversation as resolved.
Show resolved Hide resolved
try (MongoClient client = getMongoClient(getMongoCryptdMongoClientSettingsBuilder().build())) {
Expand Down