Skip to content

Commit 69c0b24

Browse files
author
Justin Lu
committedNov 28, 2023
8320714: java/util/Locale/LocaleProvidersRun.java and java/util/ResourceBundle/modules/visibility/VisibilityTest.java timeout after passing
Reviewed-by: naoto, bpb, lancea
1 parent 66ae6d5 commit 69c0b24

File tree

2 files changed

+6
-4
lines changed

2 files changed

+6
-4
lines changed
 

‎test/jdk/java/util/Locale/LocaleProvidersRun.java

+3-2
Original file line numberDiff line numberDiff line change
@@ -28,6 +28,7 @@
2828
* 8150432 8215913 8220227 8228465 8232871 8232860 8236495 8245241
2929
* 8246721 8248695 8257964 8261919
3030
* @summary tests for "java.locale.providers" system property
31+
* @requires vm.flagless
3132
* @library /test/lib
3233
* @build LocaleProviders
3334
* providersrc.spi.src.tznp
@@ -179,8 +180,8 @@ public static void main(String[] args) throws Throwable {
179180
private static void testRun(String prefList, String methodName,
180181
String param1, String param2, String param3) throws Throwable {
181182

182-
// Build process (with VM flags)
183-
ProcessBuilder pb = ProcessTools.createTestJavaProcessBuilder(
183+
// Build process (without VM flags)
184+
ProcessBuilder pb = ProcessTools.createLimitedTestJavaProcessBuilder(
184185
"-ea", "-esa",
185186
"-cp", Utils.TEST_CLASS_PATH,
186187
"-Djava.util.logging.config.class=LocaleProviders$LogConfig",

‎test/jdk/java/util/ResourceBundle/modules/visibility/VisibilityTest.java

+3-2
Original file line numberDiff line numberDiff line change
@@ -26,6 +26,7 @@
2626
* @bug 8137317 8139238 8210408
2727
* @summary Visibility tests for ResourceBundle.getBundle with and without
2828
* an unnamed module argument.
29+
* @requires vm.flagless
2930
* @library /test/lib
3031
* ..
3132
* @build jdk.test.lib.JDKToolLauncher
@@ -330,8 +331,8 @@ public void RunWithPkgRes(List<String> argsList) throws Throwable {
330331
}
331332

332333
private int runCmd(List<String> argsList) throws Throwable {
333-
// Build process (with VM flags)
334-
ProcessBuilder pb = ProcessTools.createTestJavaProcessBuilder(
334+
// Build process (without VM flags)
335+
ProcessBuilder pb = ProcessTools.createLimitedTestJavaProcessBuilder(
335336
Stream.concat(Stream.of("-ea", "-esa"), argsList.stream()).toList());
336337
// Evaluate process status
337338
return ProcessTools.executeCommand(pb).getExitValue();

0 commit comments

Comments
 (0)
Please sign in to comment.