Install required packages for testing inside the worfklow rather than…

… as prerequisites.

.github/workflows/e2e-tests-boost.yml

@@ -45,6 +45,8 @@ jobs:
           pnpm install
           pnpx jetpack build plugins/jetpack -v --production
           pnpx jetpack build plugins/boost -v --production
+          pnpx jetpack build packages/lazy-images -v --production
+          pnpx jetpack build packages/jitm -v --production
 
       - name: Environment set-up
         working-directory: projects/plugins/boost/tests/e2e

projects/plugins/boost/tests/e2e/lib/env/prerequisites.js

@@ -1,5 +1,5 @@
 import logger from 'jetpack-e2e-commons/logger.cjs';
-import { execWpCommand, buildPackage } from 'jetpack-e2e-commons/helpers/utils-helper.cjs';
+import { execWpCommand } from 'jetpack-e2e-commons/helpers/utils-helper.cjs';
 
 import { expect } from '@playwright/test';
 import { JetpackBoostPage } from '../pages/index.js';
@@ -9,7 +9,6 @@ export function boostPrerequisitesBuilder( page ) {
 		testPostTitles: [],
 		clean: undefined,
 		modules: { active: undefined, inactive: undefined },
-		packages: [],
 		connected: undefined,
 		jetpackDeactivated: undefined,
 	};
@@ -23,10 +22,6 @@ export function boostPrerequisitesBuilder( page ) {
 			state.modules.inactive = modules;
 			return this;
 		},
-		withPackages( packages = [] ) {
-			state.packages = packages;
-			return this;
-		},
 		withConnection( shouldBeConnected ) {
 			state.connected = shouldBeConnected;
 			return this;
@@ -48,7 +43,6 @@ export function boostPrerequisitesBuilder( page ) {
 async function buildPrerequisites( state, page ) {
 	const functions = {
 		modules: () => ensureModulesState( state.modules ),
-		packages: () => ensurePackages( state.packages ),
 		connected: () => ensureConnectedState( state.connected, page ),
 		testPostTitles: () => ensureTestPosts( state.testPostTitles ),
 		clean: () => ensureCleanState( state.clean ),
@@ -97,13 +91,6 @@ export async function deactivateModules( modules ) {
 	}
 }
 
-export async function ensurePackages( packages ) {
-	for ( const jetpackPackage of packages ) {
-		logger.prerequisites( `Installing package ${ jetpackPackage }` );
-		await buildPackage( jetpackPackage );
-	}
-}
-
 export async function ensureConnectedState( requiredConnected = undefined, page ) {
 	const isConnected = await checkIfConnected();
 

projects/plugins/boost/tests/e2e/lib/setupTests.js

@@ -6,5 +6,5 @@ export default async function () {
 	const browser = await chromium.launch();
 	const page = await browser.newPage();
 	await prerequisitesBuilder( page ).withLoggedIn( true ).build();
-	await boostPrerequisitesBuilder( page ).withPackages( [ 'jitm' ] ).withConnection( true ).build();
+	await boostPrerequisitesBuilder( page ).withConnection( true ).build();
 }

projects/plugins/boost/tests/e2e/specs/jetpack-boost/lazy-images.test.js

@@ -11,10 +11,7 @@ test.describe.serial( 'Lazy Images module', () => {
 
 	test.beforeAll( async ( { browser } ) => {
 		page = await browser.newPage();
-		await boostPrerequisitesBuilder( page )
-			.withTestContent( [ testPostTitle ] )
-			.withPackages( [ 'lazy-images' ] )
-			.build();
+		await boostPrerequisitesBuilder( page ).withTestContent( [ testPostTitle ] ).build();
 		await execWpCommand( 'user session destroy wordpress --all' );
 	} );
 

tools/e2e-commons/helpers/utils-helper.cjs

@@ -335,11 +335,6 @@ async function getJetpackVersion() {
 	return version;
 }
 
-async function buildPackage( jetpackPackage ) {
-	const cmd = `pnpx jetpack build packages/${ jetpackPackage } -v --production`;
-	await execShellCommand( cmd );
-}
-
 module.exports = {
 	execShellCommand,
 	execSyncShellCommand,
@@ -360,5 +355,4 @@ module.exports = {
 	getMailchimpCredentials,
 	logEnvironment,
 	getJetpackVersion,
-	buildPackage,
 };