8220409: jdk/modules/scenarios/overlappingpackages/OverlappingPackagesTest.java - testOverlapWithBaseModule tests the wrong thing
authorclanger
Mon, 11 Mar 2019 15:34:16 +0100
changeset 54055 289fd6cb7480
parent 54054 1def2d745747
child 54056 17a6681a5118
child 57253 f0e513137db2
8220409: jdk/modules/scenarios/overlappingpackages/OverlappingPackagesTest.java - testOverlapWithBaseModule tests the wrong thing Reviewed-by: alanb
test/jdk/jdk/modules/scenarios/overlappingpackages/OverlappingPackagesTest.java
--- a/test/jdk/jdk/modules/scenarios/overlappingpackages/OverlappingPackagesTest.java	Mon Mar 11 17:34:23 2019 +0530
+++ b/test/jdk/jdk/modules/scenarios/overlappingpackages/OverlappingPackagesTest.java	Mon Mar 11 15:34:16 2019 +0100
@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2015, 2018, Oracle and/or its affiliates. All rights reserved.
+ * Copyright (c) 2015, 2019, Oracle and/or its affiliates. All rights reserved.
  * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
  *
  * This code is free software; you can redistribute it and/or modify it
@@ -37,12 +37,13 @@
 import java.util.Arrays;
 import java.util.List;
 
-import jdk.test.lib.compiler.CompilerUtils;
-import static jdk.test.lib.process.ProcessTools.*;
-
 import org.testng.annotations.BeforeTest;
 import org.testng.annotations.Test;
-import static org.testng.Assert.*;
+
+import jdk.test.lib.compiler.CompilerUtils;
+
+import static jdk.test.lib.process.ProcessTools.executeTestJava;
+import static org.testng.Assert.assertTrue;
 
 @Test
 public class OverlappingPackagesTest {
@@ -93,7 +94,7 @@
     public void testOverlapWithBaseModule() throws Exception {
         int exitValue
             = executeTestJava("--module-path", MODS_DIR.toString(),
-                              "-add-modules", "misc",
+                              "--add-modules", "misc",
                               "-m", "test/test.Main")
                 .outputTo(System.out)
                 .errorTo(System.err)