Remove verify cli version step from a job to reduce complexity.
This should be properly tested in op-cli-installer package. Current test will confirm that secrets are loaded correctly.
This commit is contained in:
2
.github/workflows/acceptance-test.yml
vendored
2
.github/workflows/acceptance-test.yml
vendored
@@ -96,8 +96,6 @@ jobs:
|
|||||||
SECRET: ${{ inputs.secret }}
|
SECRET: ${{ inputs.secret }}
|
||||||
SECRET_IN_SECTION: ${{ inputs.secret-in-section }}
|
SECRET_IN_SECTION: ${{ inputs.secret-in-section }}
|
||||||
MULTILINE_SECRET: ${{ inputs.multiline-secret }}
|
MULTILINE_SECRET: ${{ inputs.multiline-secret }}
|
||||||
- name: Verify installed op cli version
|
|
||||||
run: ./tests/assert-cli-version.sh ${{ inputs.version }}
|
|
||||||
- name: Assert test secret values [step output]
|
- name: Assert test secret values [step output]
|
||||||
if: ${{ !inputs.export-env }}
|
if: ${{ !inputs.export-env }}
|
||||||
env:
|
env:
|
||||||
|
|||||||
2
.gitignore
vendored
2
.gitignore
vendored
@@ -1,2 +1,2 @@
|
|||||||
coverage/
|
coverage/
|
||||||
node_modules/
|
node_modules/
|
||||||
|
|||||||
@@ -1,30 +0,0 @@
|
|||||||
#!/bin/bash
|
|
||||||
set -e
|
|
||||||
|
|
||||||
OP_CLI_VERSION="$1"
|
|
||||||
CLI_URL="https://app-updates.agilebits.com/product_history/CLI2"
|
|
||||||
|
|
||||||
get_latest_cli_version() {
|
|
||||||
conditional_path="/beta/"
|
|
||||||
if [ "$1" == "non_beta" ]; then
|
|
||||||
conditional_path="!/beta/"
|
|
||||||
fi
|
|
||||||
# This long command parses the HTML page at "CLI_URL" and finds the latest CLI version
|
|
||||||
# based on the release channel we're looking for (stable or beta).
|
|
||||||
#
|
|
||||||
# The ideal call (i.e. 'curl https://app-updates.agilebits.com/check/1/0/CLI2/en/2.0.0/Y -s | jq -r .version')
|
|
||||||
# doesn't retrieve the latest CLI version on a channel basis.
|
|
||||||
# If the latest release is stable and we want the latest beta, this command will return the stable still.
|
|
||||||
OP_CLI_VERSION="$(curl -s $CLI_URL | awk -v RS='<h3>|</h3>' 'NR % 2 == 0 {gsub(/[[:blank:]]+/, ""); gsub(/<span[^>]*>|<\/span>|[\r\n]+/, ""); gsub(/ .*$/, ""); if (!'"$1"' && '"$conditional_path"'){print; '"$1"'=1;}}')"
|
|
||||||
}
|
|
||||||
|
|
||||||
if [ "$OP_CLI_VERSION" == "latest" ]; then
|
|
||||||
get_latest_cli_version non_beta
|
|
||||||
elif [ "$OP_CLI_VERSION" == "latest-beta" ]; then
|
|
||||||
get_latest_cli_version beta
|
|
||||||
fi
|
|
||||||
|
|
||||||
if [ "$(op --version)" != "$OP_CLI_VERSION" ]; then
|
|
||||||
echo -e "Expected CLI version to be:\n$OP_CLI_VERSION\nBut got:\n$(op --version)"
|
|
||||||
exit 1
|
|
||||||
fi
|
|
||||||
Reference in New Issue
Block a user