From b819a9349c47e47633d5f23724b8f1ba09389885 Mon Sep 17 00:00:00 2001 From: Marc Khouzam Date: Tue, 31 Dec 2019 17:49:02 -0500 Subject: [PATCH] Add completion test on centos It was reported in https://github.com/helm/helm/issues/7304 that centos does not support 'tail +2' but must use the form 'tail -n +2'. This commits adds a test for centos to prevent any regressions with centos Signed-off-by: Marc Khouzam --- scripts/completion-tests/test-completion.sh | 18 ++++++++++++++++++ 1 file changed, 18 insertions(+) diff --git a/scripts/completion-tests/test-completion.sh b/scripts/completion-tests/test-completion.sh index 53aeefb..2272364 100755 --- a/scripts/completion-tests/test-completion.sh +++ b/scripts/completion-tests/test-completion.sh @@ -146,6 +146,24 @@ docker run --rm \ -e COMP_DIR=${COMP_DIR} \ ${BASH3_IMAGE} bash -c "source ${COMP_SCRIPT}" +######################################## +# Bash centos completion tests +# https://github.com/helm/helm/pull/7304 +######################################## +BASH_IMAGE=completion-bash-centos + +echo;echo; +docker build -t ${BASH_IMAGE} - <<- EOF + FROM centos + RUN yum install -y bash-completion which +EOF +docker run --rm \ + -v ${COMP_DIR}:${COMP_DIR} \ + -e ROBOT_HELM_V3=${ROBOT_HELM_V3} \ + -e ROBOT_DEBUG_LEVEL=${ROBOT_DEBUG_LEVEL} \ + -e COMP_DIR=${COMP_DIR} \ + ${BASH_IMAGE} bash -c "source ${COMP_SCRIPT}" + ######################################## # Zsh completion tests ########################################