Skip to content

Commit

Permalink
Update wrong checks in tests
Browse files Browse the repository at this point in the history
  • Loading branch information
alex268 committed Oct 27, 2023
1 parent fd7f26b commit f3ec87b
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -427,7 +427,7 @@ public void executeScanQueryAsUpdate() throws SQLException {
statement.setInt("key", 1);
statement.setString("c_Text", "value-1");

ExceptionAssert.ydbConditionallyRetryable("Scan query should have a single result set",
ExceptionAssert.ydbNonRetryable("Scan query should have a single result set",
statement::executeScanQuery);
}
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -290,7 +290,7 @@ public void executeScanQueryAsUpdate() throws SQLException {
statement.setInt("key", 1);
statement.setString("c_Text", "value-1");

ExceptionAssert.ydbConditionallyRetryable("Scan query should have a single result set", statement::execute);
ExceptionAssert.ydbNonRetryable("Scan query should have a single result set", statement::execute);
}
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -312,15 +312,15 @@ public void executeScanQueryOnSystemTable() throws SQLException {

@Test
public void executeScanQueryMultiResult() {
ExceptionAssert.ydbConditionallyRetryable("Scan query should have a single result set",
ExceptionAssert.ydbNonRetryable("Scan query should have a single result set",
() -> statement.executeUpdate("scan select 2 + 2;scan select 2 + 3")
);
}

@Test
public void executeScanQueryAsUpdate() {
// Looks weird
ExceptionAssert.ydbConditionallyRetryable("Scan query should have a single result set",
ExceptionAssert.ydbNonRetryable("Scan query should have a single result set",
() -> statement.executeUpdate("SCAN\n" + TEST_UPSERT1_SQL)
);
}
Expand Down

0 comments on commit f3ec87b

Please sign in to comment.