clean up

.circleci/config.yml

@@ -53,15 +53,13 @@ develop_master_rc_only: &develop_master_rc_only
         - master
         - /^Version-v(\d+)[.](\d+)[.](\d+)/
 
-conditional-require-get-changed-files: &conditional-require-get-changed-files
-    requires:
-      - get-changed-files-with-git-diff
-    filters:
-      branches:
-        ignore:
-          - develop
-          - master
-          - /^Version-v(\d+)[.](\d+)[.](\d+)$/
+develop_master_rc_excluded: &develop_master_rc_excluded
+  filters:
+    branches:
+      ignore:
+        - develop
+        - master
+        - /^Version-v(\d+)[.](\d+)[.](\d+)$/
 
 aliases:
   # Shallow Git Clone
@@ -116,12 +114,7 @@ workflows:
       - check-pr-tag
       - prep-deps
       - get-changed-files-with-git-diff:
-          filters:
-            branches:
-              ignore:
-                - develop
-                - master
-                - /^Version-v(\d+)[.](\d+)[.](\d+)/
+          <<: *develop_master_rc_excluded
           requires:
             - prep-deps
       - test-deps-audit:
@@ -204,53 +197,53 @@ workflows:
           requires:
             - prep-deps
       - test-e2e-chrome:
-          <<: *conditional-require-get-changed-files
           requires:
             - prep-build-test
+            - get-changed-files-with-git-diff
       - test-e2e-chrome-confirmation-redesign:
-          <<: *conditional-require-get-changed-files
           requires:
             - prep-build-confirmation-redesign-test
+            - get-changed-files-with-git-diff
       - test-e2e-firefox:
-          <<: *conditional-require-get-changed-files
           requires:
             - prep-build-test-mv2
+            - get-changed-files-with-git-diff
       - test-e2e-firefox-confirmation-redesign:
           <<: *develop_master_rc_only
-          <<: *conditional-require-get-changed-files
           requires:
             - prep-build-confirmation-redesign-test-mv2
+            - get-changed-files-with-git-diff
       - test-e2e-chrome-rpc:
-          <<: *conditional-require-get-changed-files
           requires:
             - prep-build-test
+            - get-changed-files-with-git-diff
       - test-api-specs:
           requires:
             - prep-build-test
       - test-e2e-chrome-multiple-providers:
-          <<: *conditional-require-get-changed-files
           requires:
             - prep-build-test
+            - get-changed-files-with-git-diff
       - test-e2e-chrome-flask:
-          <<: *conditional-require-get-changed-files
           requires:
             - prep-build-test-flask
+            - get-changed-files-with-git-diff
       - test-e2e-firefox-flask:
           <<: *develop_master_rc_only
-          <<: *conditional-require-get-changed-files
           requires:
             - prep-build-test-flask-mv2
+            - get-changed-files-with-git-diff
       - test-e2e-chrome-mmi:
-          <<: *conditional-require-get-changed-files
           requires:
             - prep-build-test-mmi
+            - get-changed-files-with-git-diff
       - test-e2e-mmi-playwright - OPTIONAL:
           requires:
             - prep-build-test-mmi-playwright
       - test-e2e-chrome-rpc-mmi:
-          <<: *conditional-require-get-changed-files
           requires:
             - prep-build-test-mmi
+            - get-changed-files-with-git-diff
       - test-e2e-chrome-vault-decryption:
           filters:
             branches:
@@ -503,6 +496,7 @@ jobs:
 
   # This job is used for the e2e quality gate.
   # It must be run before any job which uses the run-all.js script.
+  # The job is skipped in develop, master or RC branches.
   get-changed-files-with-git-diff:
     executor: node-browsers-small
     steps:

test/e2e/changedFilesUtil.js

@@ -20,7 +20,7 @@ async function readChangedFiles() {
     return changedFiles;
   } catch (error) {
     console.error('Error reading from file:', error);
-    return [''];
+    return [];
   }
 }