RFR 7902028: Make com.sun.javatest.regtest.config.TestProperties API public

Jonathan Gibbons jonathan.gibbons at oracle.com
Wed Sep 13 22:07:48 UTC 2017


Is this OK?

-- Jon

$ hg diff -c tip
diff -r 815bc88d8313 -r 297e70583720 
src/share/classes/com/sun/javatest/regtest/config/RegressionTestSuite.java
--- 
a/src/share/classes/com/sun/javatest/regtest/config/RegressionTestSuite.java 
Mon Jul 31 13:27:30 2017 -0700
+++ 
b/src/share/classes/com/sun/javatest/regtest/config/RegressionTestSuite.java 
Wed Sep 13 15:06:07 2017 -0700
@@ -208,6 +208,10 @@
          return properties.getExternalLibs(td.getFile());
      }

+    public Set<String> getDefaultModules(TestDescription td) throws 
TestSuite.Fault {
+        return properties.getModules(td.getFile());
+    }
+
      public ExtraPropDefns getExtraPropDefns() {
          return properties.getExtraPropDefns();
      }



On 09/12/2017 04:56 PM, Alexandre (Shura) Iline wrote:
> I realized I have forgotten to CC JTReg dev alias.
>
>> On Sep 11, 2017, at 10:44 AM, Alexandre (Shura) Iline <alexandre.iline at oracle.com> wrote:
>>
>> Jon,
>>
>> Let me know if you would be OK with this fix. I, for now, only need the modules, but I through it would be somewhat weird to only make that public.
>>
>> Bug: https://bugs.openjdk.java.net/browse/CODETOOLS-7902028
>> Webrev: http://cr.openjdk.java.net/~shurailine/7902028/webrev.00/
>>
>> Thank you.
>>
>> Shura
>>



More information about the jtreg-dev mailing list