fix: extract platform drivers and management to commandbox-cbplaywright

This package now depends on commandbox-cbplaywright.
commandbox-cbplaywright handles downloading and maintaining
the Playwright driver as well as providing access to
the CLI.

Author: elpete

.github/workflows/cron.yml

@@ -41,8 +41,8 @@ jobs:
 
       - name: Install Playwright dependencies
         run: |
-          java -cp "lib/*" -Dplaywright.cli.dir="lib/driver/linux/" com.microsoft.playwright.CLI install-deps
-          java -cp "lib/*" -Dplaywright.cli.dir="lib/driver/linux/" com.microsoft.playwright.CLI install --with-deps chromium
+          box playwright-cli install-deps
+          box playwright-cli install chromium
 
       - name: Run TestBox Tests
         run: box testbox run

.github/workflows/pr.yml

@@ -48,8 +48,8 @@ jobs:
 
       - name: Install Playwright dependencies
         run: |
-          java -cp "lib/*" -Dplaywright.cli.dir="lib/driver/linux/" com.microsoft.playwright.CLI install-deps
-          java -cp "lib/*" -Dplaywright.cli.dir="lib/driver/linux/" com.microsoft.playwright.CLI install --with-deps chromium
+          box playwright-cli install-deps
+          box playwright-cli install chromium
 
       - name: Run TestBox Tests
         run: box testbox run

.github/workflows/prerelease.yml

@@ -42,8 +42,8 @@ jobs:
 
       - name: Install Playwright dependencies
         run: |
-          java -cp "lib/*" -Dplaywright.cli.dir="lib/driver/linux/" com.microsoft.playwright.CLI install-deps
-          java -cp "lib/*" -Dplaywright.cli.dir="lib/driver/linux/" com.microsoft.playwright.CLI install --with-deps chromium
+          box playwright-cli install-deps
+          box playwright-cli install chromium
 
       - name: Run TestBox Tests
         run: box testbox run

.github/workflows/release.yml

@@ -43,8 +43,8 @@ jobs:
 
       - name: Install Playwright dependencies
         run: |
-          java -cp "lib/*" -Dplaywright.cli.dir="lib/driver/linux/" com.microsoft.playwright.CLI install-deps
-          java -cp "lib/*" -Dplaywright.cli.dir="lib/driver/linux/" com.microsoft.playwright.CLI install --with-deps chromium
+          box playwright-cli install-deps
+          box playwright-cli install chromium
 
       - name: Run TestBox Tests
         run: box testbox run

README.md

@@ -33,21 +33,18 @@ this.mappings[ "/cbPlaywright" ] = rootPath & "/modules/cbPlaywright";
 
 > Note: You can't use the mapping in the `javaSettings` since they are both in the psuedo-constructor.
 
-3. Add a `PLAYWRIGHT_SKIP_BROWSER_DOWNLOAD=1` environment variable.
-
-**Playwright won't start in a CFML server without this. (If someone finds a workaround for this, please open a PR.)**
+3. (OPTIONAL) If you have installed CommandBox or commandbox-cbplaywright in a non-standard location, set the `CBPLAYWRIGHT_DRIVER_DIR`
+environment variable to the correct location of the Playwright driver.
 
 ### Playwright CLI
 
-To interact with the Playwright CLI, use the following snippet:
+To interact with the Playwright CLI, use `commandbox-cbplaywright:
 
+```sh
+playwright-cli
+# or
+playwright
 ```
-java -cp "modules/cbPlaywright/lib/*" -Dplaywright.cli.dir="lib/driver/linux/" com.microsoft.playwright.CLI
-```
-
-> Replace `linux` with the correct architecture for your OS.
-> Valid values are `linux`, `linux-arm64`, `mac`, or `win32_x64`
-> (In the future, we'll have a cbPlaywright CommandBox module that will make this easier.)
 
 You can see the documentation for the [Playwright Java CLI here](https://playwright.dev/java/docs/cli).
 
@@ -59,16 +56,12 @@ for more information.
 
 Examples:
 
+```sh
+box playwright install chromium
+box playwright install firefox
+box playwright install webkit
+box playwright install msedge
 ```
-java -cp "modules/cbPlaywright/lib/*" -Dplaywright.cli.dir="lib/driver/linux/" com.microsoft.playwright.CLI install chromium
-java -cp "modules/cbPlaywright/lib/*" -Dplaywright.cli.dir="lib/driver/linux/" com.microsoft.playwright.CLI install firefox
-java -cp "modules/cbPlaywright/lib/*" -Dplaywright.cli.dir="lib/driver/linux/" com.microsoft.playwright.CLI install webkit
-java -cp "modules/cbPlaywright/lib/*" -Dplaywright.cli.dir="lib/driver/linux/" com.microsoft.playwright.CLI install msedge
-```
-
-> Replace `linux` with the correct architecture for your OS.
-> Valid values are `linux`, `linux-arm64`, `mac`, or `win32_x64`
-> (In the future, we'll have a cbPlaywright CommandBox module that will make this easier.)
 
 ### Usage
 

box.json

@@ -20,7 +20,9 @@
     "shortDescription":"CFML integration with TestBox and Playwright",
     "description":"CFML integration with TestBox and Playwright to run tests in actual browsers",
     "type":"modules",
-    "dependencies":{},
+    "dependencies":{
+        "commandbox-cbplaywright": "^1.0.0"
+    },
     "devDependencies":{
         "testbox":"stable"
     },

build/downloadAndExtractJars.cfc renamed to build/downloadJars.cfc

@@ -19,22 +19,9 @@ component {
 		}
 		bundleFile = bundleFile[ 1 ];
 
-		print.line( "Unzipping the drivers..." ).toConsole();
-		cfzip( action = "unzip", file = bundleFile, destination = "../lib/driver-bundle" );
-		directoryCopy( "../lib/driver-bundle/driver", "../lib/driver", true );
-		directoryDelete( "../lib/driver-bundle", true );
+		print.line( "Removing the drivers. These are provided by commandbox-cbplaywright..." ).toConsole();
 		fileDelete( bundleFile );
 
-		print.line( "Setting the correct permissions for the driver files..." ).toConsole();
-		var files = directoryList( "../lib/driver", true );
-		var fileCount = files.len();
-		progressBarGeneric.update( percent = 0 );
-		files.each( ( fileName, i ) => {
-			fileSetAccessMode( fileName, "777" );
-			progressBarGeneric.update( percent = ( i / fileCount ) * 100 );
-		} );
-
-
 		print.greenLine( 'Download and extraction complete!' ).toConsole();
 	}