diff options
author | Paul Duffin <paulduffin@google.com> | 2017-03-24 13:34:29 +0000 |
---|---|---|
committer | android-build-merger <android-build-merger@google.com> | 2017-03-24 13:34:29 +0000 |
commit | 67e1cb11019cf70266b6c0509b06a17ce0742429 (patch) | |
tree | f709289deffa11c315718fd7f30b9ed3493aef6c | |
parent | 6c3f4570e0c94fa43e1161eba44d141b24d456e5 (diff) | |
parent | ab5f9f1d788a2cc59ae0e6b18d527a4f1acd5563 (diff) | |
download | junit-params-67e1cb11019cf70266b6c0509b06a17ce0742429.tar.gz |
Hard code text case name template for AndroidJUnitRunner and CTSandroid-wear-o-preview-4android-wear-o-preview-3android-o-preview-2
am: ab5f9f1d78
Change-Id: I9ffd6f4c59eee40bd1da192efc803179cd13c8bc
-rw-r--r-- | src/main/java/junitparams/naming/MacroSubstitutionNamingStrategy.java | 11 |
1 files changed, 9 insertions, 2 deletions
diff --git a/src/main/java/junitparams/naming/MacroSubstitutionNamingStrategy.java b/src/main/java/junitparams/naming/MacroSubstitutionNamingStrategy.java index ce56847..1bdcafb 100644 --- a/src/main/java/junitparams/naming/MacroSubstitutionNamingStrategy.java +++ b/src/main/java/junitparams/naming/MacroSubstitutionNamingStrategy.java @@ -14,7 +14,9 @@ public class MacroSubstitutionNamingStrategy implements TestCaseNamingStrategy { private static final Pattern MACRO_SPLIT_PATTERN = Pattern.compile(String.format("(?=%s)|(?<=%s)", MACRO_PATTERN, MACRO_PATTERN)); private static final String MACRO_START = "{"; private static final String MACRO_END = "}"; - static final String DEFAULT_TEMPLATE = "[{index}] {params} ({method})"; + // Android-changed: CTS and AndroidJUnitRunner rely on specific format to test names, changing + // them will prevent CTS and AndroidJUnitRunner from working properly; see b/36541809 + static final String DEFAULT_TEMPLATE = "{method}[{index}]"; private TestMethod method; public MacroSubstitutionNamingStrategy(TestMethod testMethod) { @@ -37,7 +39,12 @@ public class MacroSubstitutionNamingStrategy implements TestCaseNamingStrategy { private String getTemplate(TestCaseName testCaseName) { if (testCaseName != null) { - return testCaseName.value(); + // Android-changed: CTS and AndroidJUnitRunner rely on specific format to test names, + // changing them will prevent CTS and AndroidJUnitRunner from working properly; + // see b/36541809 + throw new IllegalStateException( + "@TestCaseName not currently supported as it breaks running tests in CTS"); + // return testCaseName.value(); } return DEFAULT_TEMPLATE; |