Skip to content

Commit b0578fe

Browse files
committed
Fix NPE in TruffleTestInvoker.
1 parent 157da25 commit b0578fe

File tree

1 file changed

+12
-4
lines changed

1 file changed

+12
-4
lines changed

truffle/src/com.oracle.truffle.tck/src/com/oracle/truffle/tck/TruffleTestInvoker.java

Lines changed: 12 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
/*
2-
* Copyright (c) 2017, 2018, Oracle and/or its affiliates. All rights reserved.
2+
* Copyright (c) 2017, 2019, Oracle and/or its affiliates. All rights reserved.
33
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
44
*
55
* The Universal Permissive License (UPL), Version 1.0
@@ -169,7 +169,9 @@ private static class TruffleFrameworkMethod extends FrameworkMethod {
169169

170170
for (int i = 0; i < paramCount; i++) {
171171
Inject testRootNode = findRootNodeAnnotation(method.getParameterAnnotations()[i]);
172-
nodeConstructors[i] = getNodeConstructor(testRootNode, testClass);
172+
if (testRootNode != null) {
173+
nodeConstructors[i] = getNodeConstructor(testRootNode, testClass);
174+
}
173175
}
174176
}
175177

@@ -189,7 +191,9 @@ RootNode[] createTestRootNodes(Object test) {
189191

190192
RootNode[] ret = new RootNode[nodeConstructors.length];
191193
for (int i = 0; i < ret.length; i++) {
192-
ret[i] = nodeConstructors[i].apply(test);
194+
if (nodeConstructors[i] != null) {
195+
ret[i] = nodeConstructors[i].apply(test);
196+
}
193197
}
194198
return ret;
195199
}
@@ -209,7 +213,11 @@ public void evaluate() throws Throwable {
209213
try (C testContext = createTestContext(testName)) {
210214
ArrayList<T> callTargets = new ArrayList<>(testNodes.length);
211215
for (RootNode testNode : testNodes) {
212-
callTargets.add(createTestCallTarget(testContext, testNode));
216+
if (testNode != null) {
217+
callTargets.add(createTestCallTarget(testContext, testNode));
218+
} else {
219+
callTargets.add(null);
220+
}
213221
}
214222

215223
Object[] args = callTargets.toArray();

0 commit comments

Comments
 (0)