fix: skip_when_tags_exist inverted (#305)

* fix: skip_when_tags_exist inverted

When using `skip_when_tags_exist`, the behavior is inverted. When `true` it should check and skip. When `false`, it should just tag it.

The code currently on `master` does the exact opposite.

* fix: conditional

* ci: add `skip_when_tags_exist` in the test job

---------

Co-authored-by: Eric Ribeiro <[email protected]>

Author: irae

.circleci/test-deploy.yml

@@ -47,6 +47,7 @@ jobs:
           region: <<parameters.region>>
       - aws-ecr/tag_image:
           repo: <<parameters.repo>>
+          skip_when_tags_exist: <<parameters.skip_when_tags_exist>>
           source_tag: <<parameters.source_tag>>
           target_tag: <<parameters.target_tag>>
   build-test-then-push-with-buildx:

src/scripts/tag_image.sh

@@ -11,7 +11,7 @@ IFS="," read -ra ECR_TAGS <<<"${ORB_STR_TARGET_TAG}"
 
 for tag in "${ECR_TAGS[@]}"; do
     # if skip_when_tags_exist is true
-    if [ "${ORB_BOOL_SKIP_WHEN_TAGS_EXIST}" -eq "0" ]; then
+    if [ "${ORB_BOOL_SKIP_WHEN_TAGS_EXIST}" -eq 1 ]; then
         # tag image if tag does not exist
         if ! echo "${EXISTING_TAGS}" | grep "${tag}"; then
             aws ecr put-image --repository-name "${ORB_STR_REPO}" --image-tag "${tag}" --image-manifest "${MANIFEST}"
@@ -23,4 +23,4 @@ for tag in "${ECR_TAGS[@]}"; do
         aws ecr put-image --repository-name "${ORB_STR_REPO}" --image-tag "${tag}" --image-manifest "${MANIFEST}"
     fi
 done
-set +x
+set +x