Skip to content

Commit cea1714

Browse files
committed
[OPENJDK-2993] Fix jolokia test that used 'unzip'
We dropped unzip from the images as nothing else was using it. Rework the jolokia test that used unzip to use jar instead. Adjust the test to not require s2i so it executes much faster and retitle it to be clearer as to what it is testing. Signed-off-by: Jonathan Dowland <[email protected]>
1 parent 9baeab7 commit cea1714

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

modules/jolokia/8.2/tests/features/jolokia.feature

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -3,9 +3,9 @@
33
# Tests for jboss/container/jolokia
44
Feature: Openshift OpenJDK Jolokia tests
55

6-
Scenario: Check Environment variable is correct
7-
Given s2i build https://github.com/jboss-container-images/openjdk-test-applications from undertow-servlet
8-
Then run sh -c 'unzip -q -p /usr/share/java/jolokia-jvm-agent/jolokia-jvm.jar META-INF/maven/org.jolokia/jolokia-jvm/pom.properties | grep -F ${JOLOKIA_VERSION}' in container and check its output for version=
6+
Scenario: Ensure JOLOKIA_VERSION variable aligns with JAR
7+
When container is started with command sh
8+
Then run sh -c 'jar xf /usr/share/java/jolokia-jvm-agent/jolokia-jvm.jar META-INF/maven/org.jolokia/jolokia-jvm/pom.properties && grep ${JOLOKIA_VERSION} META-INF/maven/org.jolokia/jolokia-jvm/pom.properties' in container and check its output for version=
99

1010
Scenario: Check jolokia port is available
1111
Given s2i build https://github.com/jboss-container-images/openjdk-test-applications from undertow-servlet

0 commit comments

Comments
 (0)