Refine parameterized unit tests

src/functionalTest/kotlin/com/github/jengelman/gradle/plugins/shadow/transformers/TransformersTest.kt

@@ -121,7 +121,7 @@ class TransformersTest : BaseTransformerTest() {
   }
 
   @ParameterizedTest
-  @MethodSource("transformerConfigurations")
+  @MethodSource("transformerConfigProvider")
   fun otherTransformers(pair: Pair<String, KClass<*>>) {
     val (configuration, transformer) = pair
     if (configuration.contains("test/some.file")) {
@@ -169,7 +169,7 @@ class TransformersTest : BaseTransformerTest() {
     """.trimIndent()
 
     @JvmStatic
-    fun transformerConfigurations() = listOf(
+    fun transformerConfigProvider() = listOf(
       "" to ApacheLicenseResourceTransformer::class,
       "" to ApacheNoticeResourceTransformer::class,
       "" to ComponentsXmlResourceTransformer::class,

src/test/kotlin/com/github/jengelman/gradle/plugins/shadow/transformers/Log4j2PluginsCacheFileTransformerTest.kt

@@ -6,6 +6,7 @@ import assertk.assertions.isEqualTo
 import assertk.assertions.isFalse
 import assertk.assertions.isNotEqualTo
 import assertk.assertions.isTrue
+import assertk.assertions.startsWith
 import assertk.fail
 import com.github.jengelman.gradle.plugins.shadow.internal.requireResourceAsStream
 import com.github.jengelman.gradle.plugins.shadow.relocation.SimpleRelocator
@@ -92,8 +93,8 @@ class Log4j2PluginsCacheFileTransformerTest : BaseTransformerTest<Log4j2PluginsC
   }
 
   @ParameterizedTest
-  @MethodSource("relocationParameters")
-  fun relocations(pattern: String, shadedPattern: String, target: String) {
+  @MethodSource("relocationProvider")
+  fun relocations(pattern: String, shadedPattern: String, expected: String) {
     val aggregator = PluginCache().apply {
       val resources = Collections.enumeration(listOf(pluginCacheUrl))
       loadCacheFiles(resources)
@@ -103,7 +104,7 @@ class Log4j2PluginsCacheFileTransformerTest : BaseTransformerTest<Log4j2PluginsC
 
     for (pluginEntryMap in aggregator.allCategories.values) {
       for (entry in pluginEntryMap.values) {
-        assertThat(entry.className.startsWith(target)).isTrue()
+        assertThat(entry.className).startsWith(expected)
       }
     }
   }
@@ -116,7 +117,7 @@ class Log4j2PluginsCacheFileTransformerTest : BaseTransformerTest<Log4j2PluginsC
     val pluginCacheUrl: URL = requireNotNull(this::class.java.classLoader.getResource(PLUGIN_CACHE_FILE))
 
     @JvmStatic
-    fun relocationParameters() = listOf(
+    fun relocationProvider() = listOf(
       // test with matching relocator
       Arguments.of("org.apache.logging", "new.location.org.apache.logging", "new.location.org.apache.logging"),
       // test without matching relocator

src/test/kotlin/com/github/jengelman/gradle/plugins/shadow/transformers/PropertiesFileTransformerTest.kt

@@ -54,14 +54,14 @@ class PropertiesFileTransformerTest : BaseTransformerTest<PropertiesFileTransfor
     assertThat(firstRunTargetLines).isEqualTo(secondRunTargetLines)
   }
 
-  @ParameterizedTest(name = "Path {0} {2} transformed")
+  @ParameterizedTest
   @MethodSource("pathProvider")
-  fun canTransformResourceWithPaths(path: String, expected: Boolean, transform: String) {
+  fun canTransformResourceWithPaths(path: String, expected: Boolean) {
     assertThat(transformer.canTransformResource(path)).isEqualTo(expected)
   }
 
-  @ParameterizedTest(name = "mergeStrategy={1}, mergeSeparator='{2}'")
-  @MethodSource("transformConfigurationsProvider")
+  @ParameterizedTest
+  @MethodSource("transformConfigProvider")
   fun exerciseAllTransformConfigurations(
     path: String,
     mergeStrategy: String,
@@ -81,8 +81,8 @@ class PropertiesFileTransformerTest : BaseTransformerTest<PropertiesFileTransfor
     assertThat(transformer.propertiesEntries[path].orEmpty()).isEqualTo(expectedOutput)
   }
 
-  @ParameterizedTest(name = "Paths={1}")
-  @MethodSource("transformConfigurationsWithPathsProvider")
+  @ParameterizedTest
+  @MethodSource("transformConfigWithPathsProvider")
   fun exerciseAllTransformConfigurationsWithPaths(
     path: String,
     paths: List<String>,
@@ -101,8 +101,8 @@ class PropertiesFileTransformerTest : BaseTransformerTest<PropertiesFileTransfor
     assertThat(transformer.propertiesEntries[path].orEmpty()).isEqualTo(expectedOutput)
   }
 
-  @ParameterizedTest(name = "Mappings={1}")
-  @MethodSource("transformConfigurationsWithMappingsProvider")
+  @ParameterizedTest
+  @MethodSource("transformConfigWithMappingsProvider")
   fun exerciseAllTransformConfigurationsWithMappings(
     path: String,
     mappings: Map<String, Map<String, String>>,
@@ -121,8 +121,8 @@ class PropertiesFileTransformerTest : BaseTransformerTest<PropertiesFileTransfor
     assertThat(transformer.propertiesEntries[path].orEmpty()).isEqualTo(expectedOutput)
   }
 
-  @ParameterizedTest(name = "KeyTransformer: {1}")
-  @MethodSource("appliesKeyTransformerProvider")
+  @ParameterizedTest
+  @MethodSource("keyTransformerProvider")
   fun appliesKeyTransformer(
     path: String,
     keyTransformer: (String) -> String,
@@ -141,8 +141,8 @@ class PropertiesFileTransformerTest : BaseTransformerTest<PropertiesFileTransfor
     assertThat(transformer.propertiesEntries[path].orEmpty()).isEqualTo(expectedOutput)
   }
 
-  @ParameterizedTest(name = "Charset: {1}")
-  @MethodSource("appliesCharsetProvider")
+  @ParameterizedTest
+  @MethodSource("charsetProvider")
   fun appliesCharset(
     path: String,
     charset: String,
@@ -166,13 +166,13 @@ class PropertiesFileTransformerTest : BaseTransformerTest<PropertiesFileTransfor
 
     @JvmStatic
     fun pathProvider() = listOf(
-      Arguments.of("foo.properties", true, "can be"),
-      Arguments.of("foo/bar.properties", true, "can be"),
-      Arguments.of("foo.props", false, "can not be"),
+      Arguments.of("foo.properties", true),
+      Arguments.of("foo/bar.properties", true),
+      Arguments.of("foo.props", false),
     )
 
     @JvmStatic
-    fun appliesCharsetProvider() = listOf(
+    fun charsetProvider() = listOf(
       Arguments.of(
         "utf8.properties",
         "utf-8",
@@ -182,7 +182,7 @@ class PropertiesFileTransformerTest : BaseTransformerTest<PropertiesFileTransfor
     )
 
     @JvmStatic
-    fun transformConfigurationsWithPathsProvider() = listOf(
+    fun transformConfigWithPathsProvider() = listOf(
       Arguments.of(
         "f.properties",
         listOf("f.properties"),
@@ -214,7 +214,7 @@ class PropertiesFileTransformerTest : BaseTransformerTest<PropertiesFileTransfor
     )
 
     @JvmStatic
-    fun transformConfigurationsWithMappingsProvider() = listOf(
+    fun transformConfigWithMappingsProvider() = listOf(
       Arguments.of(
         "f.properties",
         mapOf("f.properties" to mapOf("mergeStrategy" to "first")),
@@ -260,7 +260,7 @@ class PropertiesFileTransformerTest : BaseTransformerTest<PropertiesFileTransfor
     )
 
     @JvmStatic
-    fun transformConfigurationsProvider() = listOf(
+    fun transformConfigProvider() = listOf(
       Arguments.of(
         "f.properties",
         "first",
@@ -296,7 +296,7 @@ class PropertiesFileTransformerTest : BaseTransformerTest<PropertiesFileTransfor
     )
 
     @JvmStatic
-    fun appliesKeyTransformerProvider() = listOf(
+    fun keyTransformerProvider() = listOf(
       Arguments.of(
         "foo.properties",
         { key: String -> key },

src/test/kotlin/com/github/jengelman/gradle/plugins/shadow/transformers/ServiceFileTransformerTest.kt

@@ -10,17 +10,17 @@ import org.junit.jupiter.params.provider.Arguments
 import org.junit.jupiter.params.provider.MethodSource
 
 class ServiceFileTransformerTest : BaseTransformerTest<ServiceFileTransformer>() {
-  @ParameterizedTest(name = "{index} => path={0}, exclude={1}, expected={2}")
-  @MethodSource("canTransformResourceData")
+  @ParameterizedTest
+  @MethodSource("resourceProvider")
   fun canTransformResource(path: String, exclude: Boolean, expected: Boolean) {
     if (exclude) {
       transformer.exclude(path)
     }
     assertThat(transformer.canTransformResource(path)).isEqualTo(expected)
   }
 
-  @ParameterizedTest(name = "{index} => path={0}")
-  @MethodSource("transformsServiceFileData")
+  @ParameterizedTest
+  @MethodSource("serviceFileProvider")
   fun transformServiceFile(path: String, input1: String, input2: String, output: String) {
     if (transformer.canTransformResource(path)) {
       transformer.transform(context(path, input1))
@@ -44,7 +44,7 @@ class ServiceFileTransformerTest : BaseTransformerTest<ServiceFileTransformer>()
     }
 
     @JvmStatic
-    fun canTransformResourceData() = listOf(
+    fun resourceProvider() = listOf(
       // path, exclude, expected
       Arguments.of("META-INF/services/java.sql.Driver", false, true),
       Arguments.of("META-INF/services/io.dropwizard.logging.AppenderFactory", false, true),
@@ -55,7 +55,7 @@ class ServiceFileTransformerTest : BaseTransformerTest<ServiceFileTransformer>()
     )
 
     @JvmStatic
-    fun transformsServiceFileData() = listOf(
+    fun serviceFileProvider() = listOf(
       // path, input1, input2, output
       Arguments.of("META-INF/services/com.acme.Foo", "foo", "bar", "foo\nbar"),
       Arguments.of("META-INF/services/com.acme.Bar", "foo\nbar", "zoo", "foo\nbar\nzoo"),