Skip to content

Commit

Permalink
ESQL: Fix test name for duplicate tests (elastic#106297)
Browse files Browse the repository at this point in the history
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}
```
  • Loading branch information
nik9000 authored Mar 13, 2024
1 parent 35f4d4c commit 6a917c7
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -67,7 +67,7 @@ public enum Mode {
ASYNC
}

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

0 comments on commit 6a917c7

Please sign in to comment.