From: Michał Gołębiowski-Owczarek <m.goleb@gmail.com>
Date: Mon, 20 Mar 2023 16:13:31 +0000 (+0100)
Subject: Build: Only install Playwright dependencies when needed
X-Git-Tag: 4.0.0-beta~63
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=e77bd9d64fc696cadfe1f8c9ebb50d7609a97b07;p=jquery.git

Build: Only install Playwright dependencies when needed

PR gh-5190 added support for running tests on Playwright WebKit
in CI. For efficiency reasons, Playwright dependencies are only
installed for the `test:browser` npm script. However, that same
script is also used for Firefox ESR testing.

This change makes Playwright dependencies installed only for cases
where `WebKitHeadless` exists on the list of tested browsers.

Closes gh-5204
Ref gh-5190
---

diff --git a/.github/workflows/node.js.yml b/.github/workflows/node.js.yml
index b773404e5..4096efffe 100644
--- a/.github/workflows/node.js.yml
+++ b/.github/workflows/node.js.yml
@@ -73,7 +73,7 @@ jobs:
 
     - name: Install Playwright dependencies
       run: npx playwright-webkit install-deps
-      if: "matrix.NPM_SCRIPT == 'test:browser'"
+      if: "matrix.NPM_SCRIPT == 'test:browser' && contains(matrix.BROWSERS, 'WebkitHeadless')"
 
     - name: Run tests
       env: