Skip to content

Commit c1d7e79

Browse files
[Test] Fix test022InstallPluginsFromLocalArchive (#131353)
The number of plugin archives changed and triggered the assertion. Changing the assertion to check for `analysis-icu` plugin archive instead of asserting on number of archives. This will always fail when new plugin is added (like microsoft-graph-authz). Resolves #116866
1 parent 8b3857b commit c1d7e79

File tree

2 files changed

+1
-4
lines changed

2 files changed

+1
-4
lines changed

muted-tests.yml

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -407,9 +407,6 @@ tests:
407407
- class: org.elasticsearch.xpack.esql.analysis.VerifierTests
408408
method: testMatchInsideEval
409409
issue: https://github.com/elastic/elasticsearch/issues/131336
410-
- class: org.elasticsearch.packaging.test.DockerTests
411-
method: test022InstallPluginsFromLocalArchive
412-
issue: https://github.com/elastic/elasticsearch/issues/116866
413410
- class: org.elasticsearch.packaging.test.DockerTests
414411
method: test071BindMountCustomPathWithDifferentUID
415412
issue: https://github.com/elastic/elasticsearch/issues/120917

qa/packaging/src/test/java/org/elasticsearch/packaging/test/DockerTests.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -205,7 +205,7 @@ public void test022InstallPluginsFromLocalArchive() {
205205

206206
listPluginArchive().forEach(System.out::println);
207207
assertThat("Expected " + plugin + " to not be installed", listPlugins(), not(hasItems(plugin)));
208-
assertThat("Expected " + plugin + " available in archive", listPluginArchive(), hasSize(16));
208+
assertThat("Expected " + plugin + " available in archive", listPluginArchive(), hasItems(containsString(plugin)));
209209

210210
// Stuff the proxy settings with garbage, so any attempt to go out to the internet would fail
211211
sh.getEnv()

0 commit comments

Comments
 (0)