Restore E2E npm behavior after having introduced cache

wordpress-mobile · Mar 12, 2024 · 7d9c1b0 · 7d9c1b0
1 parent a305d99
commit 7d9c1b0
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 2 deletions.
diff --git a/.buildkite/commands/test-android.sh b/.buildkite/commands/test-android.sh
@@ -16,7 +16,12 @@ while [ "$INPUT" != "" ]; do
     INPUT="${1-}"
 done
 
-.buildkite/commands/install-node-dependencies.sh
+# First, set up the gutenberg-mobile dependencies without building the i18n cache (--ignore-scripts)
+# It takes time and we don't need at this point as we are running the tests on top of something already built.
+.buildkite/commands/install-node-dependencies.sh --ignore-scripts
+# Then, set up the gutenberg submodule dependencies, bypassed by the step above.
+# We need them because some E2E logic lives in gutenber.
+.buildkite/commands/install-node-dependencies.sh --prefix gutenberg
 
 echo '--- :ios: Set env var for Android E2E testing'
 set -x

diff --git a/.buildkite/commands/test-ios.sh b/.buildkite/commands/test-ios.sh
@@ -19,7 +19,12 @@ while [ "$INPUT" != "" ]; do
     INPUT="${1-}"
 done
 
-.buildkite/commands/install-node-dependencies.sh
+# First, set up the gutenberg-mobile dependencies without building the i18n cache (--ignore-scripts)
+# It takes time and we don't need at this point as we are running the tests on top of something already built.
+.buildkite/commands/install-node-dependencies.sh --ignore-scripts
+# Then, set up the gutenberg submodule dependencies, bypassed by the step above.
+# We need them because some E2E logic lives in gutenber.
+.buildkite/commands/install-node-dependencies.sh --prefix gutenberg
 
 echo '--- :ios: Set env var for iOS E2E testing'
 set -x