Browse Source

[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
Slobodan Adamović 2 months ago
parent
commit
c1d7e79ffd

+ 0 - 3
muted-tests.yml

@@ -407,9 +407,6 @@ tests:
 - class: org.elasticsearch.xpack.esql.analysis.VerifierTests
   method: testMatchInsideEval
   issue: https://github.com/elastic/elasticsearch/issues/131336
-- class: org.elasticsearch.packaging.test.DockerTests
-  method: test022InstallPluginsFromLocalArchive
-  issue: https://github.com/elastic/elasticsearch/issues/116866
 - class: org.elasticsearch.packaging.test.DockerTests
   method: test071BindMountCustomPathWithDifferentUID
   issue: https://github.com/elastic/elasticsearch/issues/120917

+ 1 - 1
qa/packaging/src/test/java/org/elasticsearch/packaging/test/DockerTests.java

@@ -205,7 +205,7 @@ public class DockerTests extends PackagingTestCase {
 
         listPluginArchive().forEach(System.out::println);
         assertThat("Expected " + plugin + " to not be installed", listPlugins(), not(hasItems(plugin)));
-        assertThat("Expected " + plugin + " available in archive", listPluginArchive(), hasSize(16));
+        assertThat("Expected " + plugin + " available in archive", listPluginArchive(), hasItems(containsString(plugin)));
 
         // Stuff the proxy settings with garbage, so any attempt to go out to the internet would fail
         sh.getEnv()