Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ensure installation tests cover all expected tools #1016

Open
wants to merge 10 commits into
base: master
Choose a base branch
from
2 changes: 1 addition & 1 deletion scripts/common_image
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ then
exit -1
fi

# set jvm and node version and print
# set java and node version and print
export JAVA_VERSION="-java${JAVA_MAJOR_VERSION}"
export NODE_VERSION="-node${NODE_MAJOR_VERSION}"
echo "Java version: ${JAVA_VERSION}, Node version: ${NODE_VERSION}"
8 changes: 5 additions & 3 deletions test/goss/goss_template.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,8 @@ package:
installed: false
git:
installed: true
build-essential:
installed: true
python2.7:
installed: true
python3:
Expand Down Expand Up @@ -39,13 +41,13 @@ command:
exit-status: 0
stdout:
- vNODE_VERSION_TEMPLATE
# verify installed UI5
# verify installed UI5 version
ui5 --version:
exit-status: 0
# verify installed grunt
# verify installed grunt version
grunt --version:
exit-status: 0
# verify installed go
# verify installed go version
go version:
exit-status:0
# verify NPM @sap-scope registry NOT configured
Expand Down