Skip to content

Commit 6a917c7

Browse files
authored
ESQL: Fix test name for duplicate tests (elastic#106297)
When we run the csv-spec tests for ESQL against a real http endpoint we actually run them twice - once async and once sync. But the names of the tests didn't reflect that - they just looked like they were accidentally duplicated. This updates the format. So this: ``` test {string.Trim} test {string.Trim #2} ``` becomes: ``` test {string.Trim ASYNC} test {string.Trim SYNC} ```
1 parent 35f4d4c commit 6a917c7

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

x-pack/plugin/esql/qa/server/src/main/java/org/elasticsearch/xpack/esql/qa/rest/EsqlSpecTestCase.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -67,7 +67,7 @@ public enum Mode {
6767
ASYNC
6868
}
6969

70-
@ParametersFactory(argumentFormatting = "%2$s.%3$s")
70+
@ParametersFactory(argumentFormatting = "%2$s.%3$s %6$s")
7171
public static List<Object[]> readScriptSpec() throws Exception {
7272
List<URL> urls = classpathResources("/*.csv-spec");
7373
assertTrue("Not enough specs found " + urls, urls.size() > 0);

0 commit comments

Comments
 (0)