This is an automated email from the ASF dual-hosted git repository.

tkalkirill pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/ignite-3.git


The following commit(s) were added to refs/heads/main by this push:
     new 7459327ca0 IGNITE-18861 NPE in DumpThreadsOnTimeout extension (#1705)
7459327ca0 is described below

commit 7459327ca0a2b10303af44827d6b7ebe146a7189
Author: Roman Puchkovskiy <[email protected]>
AuthorDate: Thu Feb 23 08:22:49 2023 +0400

    IGNITE-18861 NPE in DumpThreadsOnTimeout extension (#1705)
---
 .../ignite/internal/testframework/junit/DumpThreadsOnTimeout.java      | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git 
a/modules/core/src/testFixtures/java/org/apache/ignite/internal/testframework/junit/DumpThreadsOnTimeout.java
 
b/modules/core/src/testFixtures/java/org/apache/ignite/internal/testframework/junit/DumpThreadsOnTimeout.java
index f44da76019..e448384b6e 100644
--- 
a/modules/core/src/testFixtures/java/org/apache/ignite/internal/testframework/junit/DumpThreadsOnTimeout.java
+++ 
b/modules/core/src/testFixtures/java/org/apache/ignite/internal/testframework/junit/DumpThreadsOnTimeout.java
@@ -65,7 +65,8 @@ public class DumpThreadsOnTimeout implements 
TestExecutionExceptionHandler, Life
         // This is a pretty hacky and not too reliable way to determine 
whether this is the timeout event we are
         // interested in (meaning, a timed-out testable/lifecycle method 
execution), but it looks like this is the
         // best we can do.
-        return throwable instanceof TimeoutException && 
throwable.getMessage().contains(" timed out after ");
+        return throwable instanceof TimeoutException
+                && throwable.getMessage() != null && 
throwable.getMessage().contains(" timed out after ");
     }
 
     private static void dumpThreadsToStdout() {

Reply via email to