Skip to content

Commit

Permalink
chore: drop use of task var in test mock
Browse files Browse the repository at this point in the history
I think this worked before, but it surprised Johnny in code review.

Better to keep it simple and unsurprising. The "*" should work too.

Signed-off-by: Ralph Bean <rbean@redhat.com>
  • Loading branch information
ralphbean committed Oct 29, 2024
1 parent 21753c0 commit 5babb41
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion tasks/create-pyxis-image/tests/mocks.sh
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ function create_container_image() {
# e.g. 0001, 0002, 0003...
echo The image id is $(awk 'END{printf("%04i", NR)}' $(workspaces.data.path)/mock_create_container_image.txt)

if [[ "$*" != "--pyxis-url https://pyxis.preprod.api.redhat.com/ --certified false --tags "*" --is-latest false --verbose --oras-manifest-fetch ${MANIFEST_FILE} --name "*" --media-type my_media_type+gzip --digest "*" --architecture-digest "*" --architecture "*" --rh-push "* ]]
if [[ "$*" != "--pyxis-url https://pyxis.preprod.api.redhat.com/ --certified false --tags "*" --is-latest false --verbose --oras-manifest-fetch "*" --name "*" --media-type my_media_type+gzip --digest "*" --architecture-digest "*" --architecture "*" --rh-push "* ]]
then
echo Error: Unexpected call
echo Mock create_container_image called with: $*
Expand Down

0 comments on commit 5babb41

Please sign in to comment.