[CI] Rework tests-php job matrix, better covers different PHP/SF versions, test all components in a single job
#6182
Workflow file for this run
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| name: Symfony UX | ||
| on: | ||
| push: | ||
| paths-ignore: | ||
| - 'src/*/doc/**' | ||
| - 'src/**/*.md' | ||
| - 'ux.symfony.com/**' | ||
| pull_request: | ||
| paths-ignore: | ||
| - 'src/*/doc/**' | ||
| - 'src/**/*.md' | ||
| - 'ux.symfony.com/**' | ||
| concurrency: | ||
| group: ${{ github.workflow }}-${{ github.head_ref || github.run_id }} | ||
| cancel-in-progress: true | ||
| jobs: | ||
| coding-style-js: | ||
| name: JavaScript Coding Style | ||
| runs-on: ubuntu-latest | ||
| steps: | ||
| - uses: actions/checkout@v4 | ||
| - run: npm i -g corepack && corepack enable | ||
| - uses: actions/setup-node@v4 | ||
| with: | ||
| cache: 'yarn' | ||
| - run: yarn --immutable | ||
| - run: yarn ci | ||
| js-dist-current: | ||
| name: Check for UnBuilt JS Dist Files | ||
| runs-on: ubuntu-latest | ||
| steps: | ||
| - uses: actions/checkout@v4 | ||
| - run: npm i -g corepack && corepack enable | ||
| - uses: actions/setup-node@v4 | ||
| with: | ||
| cache: 'yarn' | ||
| cache-dependency-path: | | ||
| yarn.lock | ||
| **/package.json | ||
| - run: yarn --immutable && yarn build | ||
| - name: Check if JS dist files are current | ||
| run: | | ||
| if ! git diff --quiet; then | ||
| echo "The Git workspace is unclean! Changes detected:" | ||
| git status --porcelain | ||
| git diff | ||
| exit 1 | ||
| else | ||
| echo "The Git workspace is clean. No changes detected." | ||
| fi | ||
| tests-php: | ||
| runs-on: ubuntu-latest | ||
| name: "PHP ${{ matrix.php-version }} - Symfony ${{ matrix.symfony-version }} - ${{ matrix.minimum-stability }} ${{ matrix.dependency-version }} deps" | ||
| strategy: | ||
| fail-fast: false | ||
| matrix: | ||
| php-version: ['8.1', '8.3', '8.4'] | ||
| symfony-version: ['6.4.x', '7.2.x'] | ||
| minimum-stability: ['stable', 'dev'] | ||
| dependency-version: ['lowest', 'highest'] | ||
| exclude: | ||
| - php-version: '8.1' | ||
| minimum-stability: 'dev' | ||
| - php-version: '8.3' | ||
| minimum-stability: 'dev' | ||
| - php-version: '8.1' | ||
| dependency-version: 'highest' | ||
| - php-version: '8.3' | ||
| dependency-version: 'lowest' | ||
| - php-version: '8.4' | ||
| dependency-version: 'lowest' | ||
| - component: Swup # has no tests | ||
|
Check failure on line 78 in .github/workflows/test.yaml
|
||
| - component: Turbo # has its own workflow (test-turbo.yml) | ||
| - component: Typed # has no tests | ||
| env: | ||
| SYMFONY_REQUIRE: ${{ matrix.config.symfony-version }} | ||
| steps: | ||
| - uses: actions/checkout@v4 | ||
| - name: Configure environment | ||
| run: | | ||
| _run_task() { | ||
| local ok=0 | ||
| local title="$1" | ||
| local start=$(date -u +%s) | ||
| OUTPUT=$(bash -xc "$2" 2>&1) || ok=$? | ||
| local end=$(date -u +%s) | ||
| if [[ $ok -ne 0 ]]; then | ||
| printf "\n%-70s%10s\n" $title $(($end-$start))s | ||
| echo "$OUTPUT" | ||
| echo "Job exited with: $ok" | ||
| echo -e "\n::error::KO $title\\n" | ||
| else | ||
| printf "::group::%-68s%10s\n" $title $(($end-$start))s | ||
| echo "$OUTPUT" | ||
| echo -e "\n\\e[32mOK\\e[0m $title\\n\\n::endgroup::" | ||
| fi | ||
| exit $ok | ||
| } | ||
| export -f _run_task | ||
| echo COLUMNS=120 >> $GITHUB_ENV | ||
| echo COMPOSER_MIN_STAB='composer config minimum-stability ${{ matrix.minimum-stability }}) --ansi' >> $GITHUB_ENV | ||
| echo COMPOSER_UP='composer update ${{ matrix.dependency-version == 'lowest' && '--prefer-lowest' }} --no-progress --ansi' >> $GITHUB_ENV | ||
| echo PHPUNIT='vendor/bin/simple-phpunit' >> $GITHUB_ENV | ||
| echo PACKAGES=$(find src/ -mindepth 2 -type f -name composer.json -not -path "*/vendor/*" -printf '%h\n' | jq -R -s -c 'split("\n")[:-1] | map(. | sub("^src/";"")) | sort') >> $GITHUB_OUTPUT | ||
| - name: Setup PHP | ||
| uses: shivammathur/setup-php@v2 | ||
| with: | ||
| php-version: ${{ matrix.php-version }} | ||
| tools: flex | ||
| - name: Install root dependencies | ||
| run: composer install | ||
| - name: Build root packages | ||
| run: php .github/build-packages.php | ||
| - name: Install packages dependencies | ||
| run: echo "$PACKAGES" | xargs -n1 | parallel -j +3 "_run_task {} 'cd src/{} && $COMPOSER_MIN_STAB && $COMPOSER_UP && $PHPUNIT install'" | ||
| - name: Run packages tests | ||
| run: echo "$PACKAGES" | xargs -n1 | parallel -j +3 "_run_task {} 'cd src/{} && $PHPUNIT'" | ||
| tests-js: | ||
| runs-on: ubuntu-latest | ||
| steps: | ||
| - uses: actions/checkout@v4 | ||
| - run: npm i -g corepack && corepack enable | ||
| - uses: actions/setup-node@v4 | ||
| with: | ||
| cache: 'yarn' | ||
| cache-dependency-path: | | ||
| yarn.lock | ||
| package.json | ||
| src/**/package.json | ||
| - run: yarn --immutable | ||
| - run: yarn playwright install | ||
| - run: yarn test | ||
| test-app-encore-app: | ||
| name: "Test Apps / Encore (${{ matrix.name}})" | ||
| runs-on: ubuntu-latest | ||
| strategy: | ||
| fail-fast: false | ||
| matrix: | ||
| include: | ||
| - name: Internal, from "vendor/" | ||
| ux-packages-source: php-vendor | ||
| - name: External, from "npm add" | ||
| ux-packages-source: js-packages | ||
| steps: | ||
| - uses: actions/checkout@v4 | ||
| - run: npm i -g corepack && corepack enable | ||
| - uses: actions/setup-node@v4 | ||
| with: | ||
| cache: 'yarn' | ||
| cache-dependency-path: | | ||
| yarn.lock | ||
| package.json | ||
| src/**/package.json | ||
| test_apps/encore-app/package.json | ||
| - uses: shivammathur/setup-php@v2 | ||
| - name: Install root dependencies | ||
| uses: ramsey/composer-install@v3 | ||
| with: | ||
| working-directory: ${{ github.workspace }} | ||
| - name: Build root packages | ||
| run: php .github/build-packages.php | ||
| working-directory: ${{ github.workspace }} | ||
| # We always install PHP deps because we of the UX Translator, which requires `var/translations` to exists | ||
| - uses: ramsey/composer-install@v3 | ||
| with: | ||
| dependency-versions: 'highest' | ||
| working-directory: test_apps/encore-app | ||
| - if: matrix.ux-packages-source == 'php-vendor' | ||
| name: Refresh dependencies from vendor/ | ||
| working-directory: test_apps/encore-app | ||
| run: yarn | ||
| env: | ||
| YARN_ENABLE_HARDENED_MODE: 0 | ||
| YARN_ENABLE_IMMUTABLE_INSTALLS: 0 | ||
| - 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 ../..; yarn workspaces list --no-private --json); do | ||
| PACKAGE_DIR=../../$(echo $PACKAGE | jq -r '.location') | ||
| PACKAGES_TO_INSTALL="$PACKAGES_TO_INSTALL $PACKAGE_DIR" | ||
| done | ||
| echo "Installing packages: $PACKAGES_TO_INSTALL" | ||
| yarn add --dev $PACKAGES_TO_INSTALL | ||
| - name: Ensure UX packages are installed from "${{ matrix.ux-packages-source == 'php-vendor' && 'vendor/symfony/ux-...' || '../../../src/**/assets' }}" | ||
| 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_VERSION=$(echo $PACKAGE | jq -r '.version') | ||
| echo -n "Checking $PACKAGE_NAME@$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." | ||
| exit 1 | ||
| fi | ||
| done; | ||
| env: | ||
| EXPECTED_PATTERN: ${{ matrix.ux-packages-source == 'php-vendor' && 'file:vendor/symfony/*' || '../../src/*' }} | ||
| - name: Run Encore (dev) | ||
| working-directory: test_apps/encore-app | ||
| run: yarn encore dev | ||
| - name: Run Encore (prod) | ||
| working-directory: test_apps/encore-app | ||
| run: yarn encore production | ||