-
Feature
-
Resolution: Unresolved
-
Major
-
None
-
None
-
None
- links to
[OPENJDK-3647] check /tmp/jre/bin/java and /tmp/jre/lib/modules exist post-s2i build if JLINK is enabled
Jonathan Dowland
added a comment - I've re-opened this as the new test unfortunately does not work:
- "and" in lower-case is not recognise by Behave
- "test -d" fails for /tmp/jre/bin/java and /tmp/jre/lib/modules because they are files not directories
I would suggest also testing each of those separately, e.g.
And run bash -c "test -d /tmp/jre/bin/java && echo PASS" in container and immediately check its output for PASS
so that if one or the other fails in a test run we can identify which;
also – could avoid needing a nested bash expression by using another test tool e.g.
And run stat /tmp/jre/bin/java in container and immediately check its output for regular file
Jonathan Dowland
added a comment - I've re-opened this as the new test unfortunately does not work:
"and" in lower-case is not recognise by Behave
"test -d" fails for /tmp/jre/bin/java and /tmp/jre/lib/modules because they are files not directories
I would suggest also testing each of those separately, e.g.
And run bash -c "test -d /tmp/jre/bin/java && echo PASS" in container and immediately check its output for PASS
so that if one or the other fails in a test run we can identify which;
also – could avoid needing a nested bash expression by using another test tool e.g.
And run stat /tmp/jre/bin/java in container and immediately check its output for regular file
Joshua Matsuoka
added a comment - Joshua Matsuoka
added a comment - PR Open: https://github.com/rh-openjdk/redhat-openjdk-containers/pull/553 
Hi jmatsuok , I ended up fixing these as part of trying to make sure OPENJDK-3591 was correct. Rather than repeat work I've just pushed the fixes to a new PR.