Corrected bug with save level order

Author: toote

hooks/post-command

@@ -27,29 +27,39 @@ if ! validate_compression "${COMPRESS}"; then
   exit 1
 fi
 
-SAVE_LEVELS=()
-if plugin_read_list_into_result SAVE; then
-  for LEVEL in "${result[@]}"; do
-    SAVE_LEVELS+=("${LEVEL}")
+if ! plugin_read_list_into_result SAVE; then
+  echo 'Cache not setup for saving'
+  exit 0
+fi
 
-    # this validates the level as well
-    KEY=$(build_key "${LEVEL}" "${CACHE_PATH}" "${COMPRESS}")
+ORDERED_LEVELS=(file step branch pipeline all)
+SAVE_LEVELS=()
 
-    if [ "${LEVEL}" = 'file' ] && [ -z "$(plugin_read_config MANIFEST)" ]; then
-      echo "+++ 🚨 Missing manifest option in the cache plugin for file-level saving"
-      exit 1
+# array intersection keeping the order of the first array
+for O_LEVEL in "${ORDERED_LEVELS[@]}"; do
+  for R_LEVEL in "${result[@]}"; do
+    if [ "${O_LEVEL}" = "${R_LEVEL}" ]; then
+      SAVE_LEVELS+=("${O_LEVEL}")
     fi
   done
-else
-  echo 'Cache not setup for saving'
-  exit 0
+done
+
+if [ "${#SAVE_LEVELS[@]}" -ne "${#result[@]}" ]; then
+  echo 'Invalid levels in the save list'
+  exit 1
 fi
 
 ACTUAL_PATH="${CACHE_PATH}"
 
 for LEVEL in "${SAVE_LEVELS[@]}"; do
+  # this validates the level as well
   KEY=$(build_key "${LEVEL}" "${CACHE_PATH}" "${COMPRESS}")
 
+  if [ "${LEVEL}" = 'file' ] && [ -z "$(plugin_read_config MANIFEST)" ]; then
+    echo "+++ 🚨 Missing manifest option in the cache plugin for file-level saving"
+    exit 1
+  fi
+
   if [ "$(plugin_read_config FORCE 'false')" != 'false' ] ||
      [ "${lower_level_updated:-false}" = 'true' ] ||
      ! backend_exec exists "${KEY}"; then

tests/post-command.bats

@@ -60,7 +60,7 @@ teardown() {
   run "$PWD/hooks/post-command"
 
   assert_failure
-  assert_output --partial 'Invalid cache level'
+  assert_output --partial 'Invalid levels in the save list'
 }
 
 @test "Invalid compression level fails" {
@@ -190,7 +190,7 @@ teardown() {
 
   assert_failure
 
-  assert_output --partial 'Invalid cache level unreal'
+  assert_output --partial 'Invalid levels in the save list'
   refute_output --partial 'Saving pipeline-level cache'
 }