Merge pull request #636 from appsignal/disable-next-js-test

Disable Next.js test

.semaphore/semaphore.yml

@@ -75,6 +75,7 @@ blocks:
     prologue:
       commands:
       - sem-version c 8
+      - npm i -g [email protected]
       - cache restore
       - mono bootstrap --ci
       - cache store
@@ -101,6 +102,7 @@ blocks:
     prologue:
       commands:
       - sem-version c 8
+      - npm i -g [email protected]
       - cache restore
       - cache restore $_PACKAGES_CACHE-packages-$SEMAPHORE_GIT_SHA-v$NODE_VERSION
       - mono bootstrap --ci
@@ -177,6 +179,7 @@ blocks:
     prologue:
       commands:
       - sem-version c 8
+      - npm i -g [email protected]
       - cache restore
       - mono bootstrap --ci
       - cache store
@@ -199,6 +202,7 @@ blocks:
     prologue:
       commands:
       - sem-version c 8
+      - npm i -g [email protected]
       - cache restore
       - cache restore $_PACKAGES_CACHE-packages-$SEMAPHORE_GIT_SHA-v$NODE_VERSION
       - mono bootstrap --ci
@@ -274,7 +278,7 @@ blocks:
       value: '14'
     prologue:
       commands:
-      - npm i -g npm@7
+      - npm i -g npm@8.5.3
       - cache restore
       - mono bootstrap --ci
       - cache store
@@ -296,7 +300,7 @@ blocks:
       value: 'false'
     prologue:
       commands:
-      - npm i -g npm@7
+      - npm i -g npm@8.5.3
       - cache restore
       - cache restore $_PACKAGES_CACHE-packages-$SEMAPHORE_GIT_SHA-v$NODE_VERSION
       - mono bootstrap --ci
@@ -372,7 +376,7 @@ blocks:
       value: '12'
     prologue:
       commands:
-      - npm i -g npm@7
+      - npm i -g npm@8.5.3
       - cache restore
       - mono bootstrap --ci
       - cache store
@@ -394,7 +398,7 @@ blocks:
       value: 'false'
     prologue:
       commands:
-      - npm i -g npm@7
+      - npm i -g npm@8.5.3
       - cache restore
       - cache restore $_PACKAGES_CACHE-packages-$SEMAPHORE_GIT_SHA-v$NODE_VERSION
       - mono bootstrap --ci

build_matrix.yml

@@ -71,11 +71,16 @@ matrix:
 
   nodejs:
     - nodejs: "17"
-      setup: &gcc8
+      setup: 
         # Configure the host to use GCC 8.3 for Node.js 16 and above. This
         # is the minimal required version for Node.js 16 and above, and the
         # extension won't compile without it.
-        - sem-version c 8
+        - &gcc8 sem-version c 8
+        # Use NPM version 8.5.3 to work around a bug when using NPM 8.5.4 or
+        # higher, where commands that are ran at the root of the workspace
+        # do not trigger lifecycle hooks for the workspace packages.
+        # https://github.com/npm/cli/issues/4552
+        - &npm853 npm i -g [email protected]
       env_vars:
         # Set `NODE_OPTIONS` to `--openssl-legacy-provider`, as a workaround
         # for this `webpack` bug affecting the `@appsignal/nextjs` tests
@@ -85,15 +90,15 @@ matrix:
         - name: NODE_OPTIONS
           value: "--openssl-legacy-provider"
     - nodejs: "16"
-      setup: *gcc8
+      setup:
+        - *gcc8
+        - *npm853
     - nodejs: "14"
-      setup: &npm7
-        # Use NPM version 7 to work around a bug when using NPM 8 and Node 14
-        # or lower, where commands that are ran at the root of the workspace
-        # do not trigger lifecycle hooks for the workspace packages.
-        - npm i -g npm@7
+      setup:
+        - *npm853
     - nodejs: "12"
-      setup: *npm7
+      setup:
+        - *npm853
   packages:
     - package: "@appsignal/nodejs"
       path: "packages/nodejs"

packages/nextjs/test/spec/integration_spec.rb

@@ -19,6 +19,7 @@
     end
 
     it "renders the index page" do
+      skip("https://github.com/appsignal/appsignal-nodejs/issues/632")
       expect(@result).to match(/Welcome to .+Next\.js!/)
     end