mirror of
https://github.com/openjdk/jdk.git
synced 2026-03-28 16:50:10 +00:00
8380033: Test java/util/jar/Manifest/IncludeInExceptionsTest.java could be more agnostic to VM default
Reviewed-by: mbaesken
This commit is contained in:
parent
f3069680e4
commit
4f2debe697
@ -21,9 +21,6 @@
|
||||
* questions.
|
||||
*/
|
||||
|
||||
import org.junit.jupiter.params.ParameterizedTest;
|
||||
import org.junit.jupiter.params.provider.MethodSource;
|
||||
|
||||
import java.io.IOException;
|
||||
import java.io.OutputStream;
|
||||
import java.nio.file.Files;
|
||||
@ -34,17 +31,23 @@ import java.util.jar.JarFile;
|
||||
import java.util.jar.JarOutputStream;
|
||||
import java.util.stream.Stream;
|
||||
|
||||
import org.junit.jupiter.params.ParameterizedTest;
|
||||
import org.junit.jupiter.params.provider.MethodSource;
|
||||
|
||||
import static java.nio.charset.StandardCharsets.UTF_8;
|
||||
import static org.junit.jupiter.api.Assertions.assertThrows;
|
||||
import static org.junit.jupiter.api.Assertions.fail;
|
||||
import static sun.security.util.SecurityProperties.INCLUDE_JAR_NAME_IN_EXCEPTIONS;
|
||||
|
||||
/*
|
||||
* @test
|
||||
* @bug 8216362
|
||||
* @run junit/othervm -Djdk.includeInExceptions=jar IncludeInExceptionsTest
|
||||
* @run junit/othervm IncludeInExceptionsTest
|
||||
* @summary Verify that the property jdk.includeInExceptions works as expected
|
||||
* when an error occurs while reading an invalid Manifest file.
|
||||
* when an error occurs while reading an invalid Manifest file.
|
||||
* @modules java.base/sun.security.util
|
||||
* @run junit/othervm -Djdk.includeInExceptions=jar IncludeInExceptionsTest
|
||||
* @run junit/othervm -Djdk.includeInExceptions= IncludeInExceptionsTest
|
||||
* @run junit/othervm IncludeInExceptionsTest
|
||||
*/
|
||||
|
||||
/*
|
||||
@ -53,8 +56,6 @@ import static org.junit.jupiter.api.Assertions.fail;
|
||||
*/
|
||||
public class IncludeInExceptionsTest {
|
||||
|
||||
private static final boolean includeInExceptions = System.getProperty("jdk.includeInExceptions") != null;
|
||||
|
||||
static final String FILENAME = "Unique-Filename-Expected-In_Msg.jar";
|
||||
|
||||
static final byte[] INVALID_MANIFEST = (
|
||||
@ -79,9 +80,9 @@ public class IncludeInExceptionsTest {
|
||||
void testInvalidManifest(Callable<?> attempt) {
|
||||
var ioException = assertThrows(IOException.class, attempt::call);
|
||||
boolean foundFileName = ioException.getMessage().contains(FILENAME);
|
||||
if (includeInExceptions && !foundFileName) {
|
||||
if (INCLUDE_JAR_NAME_IN_EXCEPTIONS && !foundFileName) {
|
||||
fail("JAR file name expected but not found in error message");
|
||||
} else if (foundFileName && !includeInExceptions) {
|
||||
} else if (foundFileName && !INCLUDE_JAR_NAME_IN_EXCEPTIONS) {
|
||||
fail("JAR file name found but should not be in error message");
|
||||
}
|
||||
}
|
||||
@ -96,4 +97,3 @@ public class IncludeInExceptionsTest {
|
||||
);
|
||||
}
|
||||
}
|
||||
|
||||
|
||||
Loading…
x
Reference in New Issue
Block a user