fix https://www.shellcheck.net/wiki/SC3011 and new test scenario

tektoncd · Apr 12, 2024 · e59682f · e59682f
1 parent 6cad5b6
commit e59682f
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/task/jq/0.1/jq.yaml b/task/jq/0.1/jq.yaml
@@ -58,7 +58,8 @@ spec:
         echo "JQ script result:"
         
         if [ "string" = "$(params.stringOrFile)" ]; then
-          jq "$(params.options)" '$(params.filter)' <<< "$(params.input)" | tee "$(results.jq-script-outcome.path)"
+          echo "$(params.input)" > tmp.json
+          jq "$(params.options)" '$(params.filter)' tmp.json | tee "$(results.jq-script-outcome.path)"
         else
           jq "$(params.options)" '$(params.filter)' "$(params.input)" | tee "$(results.jq-script-outcome.path)"
         fi
diff --git a/task/jq/0.1/test/pipeline.yaml b/task/jq/0.1/test/pipeline.yaml
@@ -32,7 +32,8 @@ spec:
     - name: jq-using-string-as-input-with-wild-filter
       params:
         - name: filter
-          value: "map(length) | max_by(.) as $max_length | .[] | select(length == $max_length)"
+          # value: "map(length) | max_by(.) as $max_length | .[] | select(length == $max_length)"
+          value: map_values(if type == "string" then . | split("") | reverse | join("") | ascii_upcase else . end)
         - name: input
           value: '[\"apple\", \"banana\", \"orange\", \"grape\", \"kiwi\", \"siriguela\"]'
         - name: stringOrFile