don-vip commented on pull request #121:
URL: https://github.com/apache/ant/pull/121#issuecomment-735423321


   @jaikiran I was speaking a bit too fast, there's a minor problem, the test 
method is missing. For this example test:
   ```java
   class TaggingPresetPreferenceTestIT extends 
AbstractExtendedSourceEntryTestCase {
   
       public static List<Object[]> data() throws Exception {
           return getTestParameters(new 
TaggingPresetPreference.TaggingPresetSourceEditor().loadAndGetAvailableSources());
       }
   
       @ParameterizedTest(name = "{0} - {1}")
       @MethodSource("data")
       void testPresetsValidity(String displayName, String url, 
ExtendedSourceEntry source) throws Exception {
       }
   ```
   
   The output for an error is:
   ```
   
org.openstreetmap.josm.gui.preferences.map.TaggingPresetPreferenceTestIT.Persian
 Presets - https://github.com/DearRude/IranianPresets/blob/master/zip/latast.zip
   ```
   
   While it was the following for JUnit 4:
   ```
   
org.openstreetmap.josm.gui.preferences.map.TaggingPresetPreferenceTestIT.testPresetsValidity[Persian
 Presets - 
https://github.com/DearRude/IranianPresets/blob/master/zip/latast.zip]
   ```
   
   Can we please get the same output as JUnit 4? 
`Test.testMethod[displayedName]`


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@ant.apache.org
For additional commands, e-mail: dev-h...@ant.apache.org

Reply via email to