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

8136597: java/lang/ProcessHandle/InfoTest.java fails

Browse Source 
Correct for truncated command line

Reviewed-by: redestad
This commit is contained in:
Roger Riggs 2015-12-21 15:26:56 -05:00
parent 5f1918e76e
commit aebd8a6c32
1 changed files with 5 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
jdk/test/java/lang/ProcessHandle/InfoTest.java
View File

@ -49,7 +49,7 @@ import org.testng.TestNG;
/*
* @test
* @bug 8077350 8081566 8081567 8098852
* @bug 8077350 8081566 8081567 8098852 8136597
* @build jdk.testlibrary.*
* @library /lib/testlibrary
* @summary Functions of ProcessHandle.Info
@ -210,10 +210,12 @@ public class InfoTest {
Assert.assertTrue(commandLine.contains(allArgs.get(i)),
"commandLine() must contain argument: " + allArgs.get(i));
}
} else if (info.commandLine().isPresent()) {
} else if (info.commandLine().isPresent() &&
command.isPresent() &&
command.get().length() < info.commandLine().get().length()) {
// If we only have the commandLine() we can only do some basic checks...
String commandLine = info.commandLine().get();
String javaExe = "java" + (Platform.isWindows() ? ".exe": "");
String javaExe = "java" + (Platform.isWindows() ? ".exe" : "");
int pos = commandLine.indexOf(javaExe);
Assert.assertTrue(pos > 0, "commandLine() should at least contain 'java'");