stan/jdk-24
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

8258799: [Testbug] RandomCommandsTest must check if tested directive is added via jcmd

Browse Source 
Reviewed-by: kvn, iignatyev
This commit is contained in:
Nils Eliasson 2021-02-03 16:01:59 +00:00
parent b0ee7a8624
commit 472bf629cd
2 changed files with 19 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
test/hotspot/jtreg/compiler/compilercontrol/share/scenario/JcmdStateBuilder.java
View File

@ -46,7 +46,7 @@ public class JcmdStateBuilder implements StateBuilder<JcmdCommand> {
private final Map<MethodDescriptor, List<CompileCommand>> matchBlocks
= new LinkedHashMap<>();
private final List<CompileCommand> inlines = new ArrayList<>();
private boolean isFileValid = true;
public boolean isFileValid = true;
public JcmdStateBuilder(String fileName) {
directiveBuilder = new DirectiveBuilder(fileName);

20
test/hotspot/jtreg/compiler/compilercontrol/share/scenario/Scenario.java
View File

@ -49,18 +49,21 @@ import java.util.function.Consumer;
*/
public final class Scenario {
private final boolean isValid;
private final boolean isJcmdValid;
private final Map<Executable, State> states;
private final List<Consumer<OutputAnalyzer>> processors;
private final Executor executor;
private final Consumer<List<OutputAnalyzer>> jcmdProcessor;
private Scenario(boolean isValid,
boolean isJcmdValid,
List<String> vmopts,
Map<Executable, State> states,
List<CompileCommand> compileCommands,
List<JcmdCommand> jcmdCommands,
List<CompileCommand> directives) {
this.isValid = isValid;
this.isJcmdValid = isJcmdValid;
this.states = states;
processors = new ArrayList<>();
processors.add(new LogProcessor(states));
@ -121,7 +124,18 @@ public final class Scenario {
} else {
// two cases for invalid inputs.
if (mainOutput.getExitValue() == 0) {
mainOutput.shouldContain("CompileCommand: An error occurred during parsing");
if (!isJcmdValid) {
boolean parse_error_found = false;
for(OutputAnalyzer out : outputList) {
if (out.getOutput().contains("Parsing of compiler directives failed")) {
parse_error_found = true;
break;
}
}
Asserts.assertTrue(parse_error_found, "'Parsing of compiler directives failed' missing from output");
} else {
mainOutput.shouldContain("CompileCommand: An error occurred during parsing");
}
} else {
Asserts.assertNE(mainOutput.getExitValue(), 0, "VM should exit with "
+ "error for incorrect directives");
@ -253,6 +267,7 @@ public final class Scenario {
public Scenario build() {
boolean isValid = true;
boolean isJcmdValid = true;
// Get states from each of the state builders
Map<Executable, State> commandFileStates
@ -315,6 +330,7 @@ public final class Scenario {
options.addAll(builder.getOptions());
isValid &= builder.isValid();
}
isJcmdValid = jcmdStateBuilder.isFileValid;
options.addAll(jcmdStateBuilder.getOptions());
/*
@ -328,7 +344,7 @@ public final class Scenario {
.forEach(entry -> entry.getValue().setLog(true));
}
return new Scenario(isValid, options, finalStates, ccList,
return new Scenario(isValid, isJcmdValid, options, finalStates, ccList,
jcmdCommands, directives);
}