[CI] Rework app-tests job to use npm as user's packages manager

Author: Kocal

.github/workflows/app-tests.yaml

@@ -60,35 +60,53 @@ jobs:
             working-directory: test_apps/encore-app
             dependency-versions: highest
 
+        - name: Manually replacing "workspace:*" references
+          run: |
+            for PACKAGE_DATA in $(cd ../..; pnpm ls -r --json --depth -1 | jq 'map(select(.private != true))' | jq -c '.[]'); do
+              PACKAGE=$(echo $PACKAGE_DATA | jq -r '.name')
+              PACKAGE_ABSOLUTE_PATH=$(echo $PACKAGE_DATA | jq -r '.path')
+
+              # Manually replace "workspace:" constraints with the actual package version, it ensure packages can be installed with npm
+              jq -rc '[(.dependencies // {}) + (.devDependencies // {}) | to_entries[] | select(.value == "workspace:*") | .key][]' "$PACKAGE_ABSOLUTE_PATH/package.json" | while read -r PACKAGE_FROM_WORKSPACE ; do
+                # Extract the resolved version of the package from the workspace, e.g.: "link:../../../../assets"
+                PACKAGE_FROM_WORKSPACE_PATH=$(cd ../..; pnpm ls --filter $PACKAGE_FROM_WORKSPACE --json --depth -1 | jq -r '.[].path')
+                echo "Found package \"$PACKAGE_FROM_WORKSPACE\" in $PACKAGE_FROM_WORKSPACE_PATH/package.json"
+
+                echo "Replacing \"$PACKAGE_FROM_WORKSPACE\" with \"file:$PACKAGE_FROM_WORKSPACE_PATH\" in $PACKAGE_ABSOLUTE_PATH/package.json"
+                sed -i "s|\"$PACKAGE_FROM_WORKSPACE\": \"workspace:\*\"|\"$PACKAGE_FROM_WORKSPACE\": \"file:$PACKAGE_FROM_WORKSPACE_PATH\"|g" "$PACKAGE_ABSOLUTE_PATH/package.json"
+              done;
+            done
+
         - working-directory: test_apps/encore-app
-          run: pnpm install --ignore-workspace
+          run: npm install
 
         - if: matrix.ux-packages-source == 'js-packages'
           name: Install UX JS packages with a JS package manager
           working-directory: test_apps/encore-app
           run: |
             PACKAGES_TO_INSTALL=''
-            for PACKAGE in $(cd ../..; pnpm ls -r --json --depth -1 | jq 'map(select(.private != true))' | jq -c '.[]'); do
-              PACKAGE_ABSOLUTE_PATH=$(echo $PACKAGE | jq -r '.path')
+            for PACKAGE_DATA in $(cd ../..; pnpm ls -r --json --depth -1 | jq 'map(select(.private != true))' | jq -c '.[]'); do
+              PACKAGE_ABSOLUTE_PATH=$(echo $PACKAGE_DATA | jq -r '.path')
               PACKAGE_RELATIVE_PATH=$(realpath --relative-to=. "$PACKAGE_ABSOLUTE_PATH")
               PACKAGES_TO_INSTALL="$PACKAGES_TO_INSTALL file:$PACKAGE_RELATIVE_PATH"
             done
+
             echo "Installing packages: $PACKAGES_TO_INSTALL"
-            pnpm add --save-dev $PACKAGES_TO_INSTALL
+            npm add --save-dev $PACKAGES_TO_INSTALL
 
         - name: Ensure UX packages are installed from "${{ env.EXPECTED_PATTERN }}"
           working-directory: test_apps/encore-app
           run: |
             for PACKAGE in $(cat package.json | jq -c '(.dependencies // {}) + (.devDependencies // {}) | to_entries[] | select(.key | startswith("@symfony/ux-")) | {name: .key, version: .value}'); do
-              PACKAGE_NAME=$(echo $PACKAGE | jq -r '.name')
+              PACKAGE=$(echo $PACKAGE | jq -r '.name')
               PACKAGE_VERSION=$(echo $PACKAGE | jq -r '.version')
 
-              echo -n "Checking $PACKAGE_NAME@$PACKAGE_VERSION..."
+              echo -n "Checking $PACKAGE@$PACKAGE_VERSION..."
               if [[ $PACKAGE_VERSION == $EXPECTED_PATTERN* ]]; then
                 echo " OK"
               else
                 echo " KO"
-                echo "The package version of $PACKAGE_NAME must starts with the pattern (e.g.: $EXPECTED_PATTERN), got $PACKAGE_VERSION instead."
+                echo "The package version of $PACKAGE must starts with the pattern (e.g.: $EXPECTED_PATTERN), got $PACKAGE_VERSION instead."
                 exit 1
               fi
             done;
@@ -97,8 +115,8 @@ jobs:
 
         - name: Ensure project can be built in dev mode
           working-directory: test_apps/encore-app
-          run: pnpm run dev
+          run: npm run dev
 
         - name: Ensure project can be built in prod mode
           working-directory: test_apps/encore-app
-          run: pnpm run build
+          run: npm run build

.github/workflows/code-quality.yaml

@@ -6,11 +6,13 @@ on:
             - 'src/*/doc/**'
             - 'src/**/*.md'
             - 'ux.symfony.com/**'
+            - '.github/workflows/app-tests.yaml'
     pull_request:
         paths-ignore:
             - 'src/*/doc/**'
             - 'src/**/*.md'
             - 'ux.symfony.com/**'
+            - '.github/workflows/app-tests.yaml'
 
 concurrency:
     group: ${{ github.workflow }}-${{ github.head_ref || github.run_id }}

.github/workflows/unit-tests.yaml

@@ -10,11 +10,13 @@ on:
             - 'src/*/doc/**'
             - 'src/**/*.md'
             - 'ux.symfony.com/**'
+            - '.github/workflows/app-tests.yaml'
     pull_request:
         paths-ignore:
             - 'src/*/doc/**'
             - 'src/**/*.md'
             - 'ux.symfony.com/**'
+            - '.github/workflows/app-tests.yaml'
 
 concurrency:
     group: ${{ github.workflow }}-${{ github.head_ref || github.run_id }}

.github/workflows/ux.symfony.com.yaml

@@ -8,13 +8,15 @@ on:
             - '!src/*/doc/**'
             - '!src/**/*.md'
             - '.github/**'
+            - '!.github/workflows/app-tests.yaml'
     pull_request:
         paths:
             - 'ux.symfony.com/**'
             - 'src/*/**'
             - '!src/*/doc/**'
             - '!src/**/*.md'
             - '.github/**'
+            - '!.github/workflows/app-tests.yaml'
 
 jobs:
 

test_apps/encore-app/.gitignore

@@ -1,4 +1,5 @@
 composer.lock
+package-lock.json
 
 ###> symfony/framework-bundle ###
 /.env.local