Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Commit c847c87

Browse filesBrowse files
assert test != null for PreTestEvent
Old logic would have thrown an NullPointerException.
1 parent 7a4fda9 commit c847c87
Copy full SHA for c847c87

File tree

1 file changed

+3
-6
lines changed
Filter options

1 file changed

+3
-6
lines changed

‎sqldev/src/main/java/org/utplsql/sqldev/runner/UtplsqlRunner.java

Copy file name to clipboardExpand all lines: sqldev/src/main/java/org/utplsql/sqldev/runner/UtplsqlRunner.java
+3-6Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -314,12 +314,9 @@ assert logFalseCondition(node != null, () -> "Could not find suite id \"" + even
314314

315315
private void doProcess(final PreTestEvent event) {
316316
final Test test = run.getTest(event.getId());
317-
if (test == null) {
318-
logger.severe(() -> "Could not find test id \"" + event.getId() + "\" when processing PreTestEvent "
319-
+ event.toString() + ".");
320-
} else {
321-
test.setStartTime(getSysdate());
322-
}
317+
assert logFalseCondition(test != null, () -> "Could not find test id \"" + event.getId()
318+
+ "\" when processing PreTestEvent " + event.toString() + ".");
319+
test.setStartTime(StringTools.getSysdate());
323320
run.setStatus(event.getId() + "...");
324321
run.setCurrentTestNumber(event.getTestNumber());
325322
run.setCurrentTest(test);

0 commit comments

Comments
0 (0)
Morty Proxy This is a proxified and sanitized view of the page, visit original site.