Improve Cypress integration test performance and CI efficiency

- Add 'deleteFiles' custom command to batch file deletions, reducing
  costly 'cy.exec' calls in test setup and teardown.
- Update 'pipeline.cy.ts' to use 'deleteFiles' for cleaning up multiple
  exported pipelines and test artifacts.
- Make 'cy:run' headless to speed up CI runs and add 'cy:debug' for
  local headed test execution.
- Add 'deleteFiles' to Cypress TypeScript definitions.

Signed-off-by: Luciano Resende <[email protected]>

Author: lresende

cypress/support/commands.ts

@@ -139,6 +139,18 @@ Cypress.Commands.add('deleteFile', (name: string): void => {
   });
 });
 
+Cypress.Commands.add('deleteFiles', (names: string[]): void => {
+  if (names.length === 0) {
+    return;
+  }
+  const findCommands = names
+    .map((name) => `-name "${name}"`)
+    .join(' -o ');
+  cy.exec(`find build/cypress/ \\( ${findCommands} \\) -delete`, {
+    failOnNonZeroExit: false
+  });
+});
+
 Cypress.Commands.add(
   'createPipeline',
   ({ name, type, emptyPipeline } = {}): void => {

cypress/support/index.d.ts

@@ -27,6 +27,7 @@ declare namespace Cypress {
       type: 'kfp' | 'airflow';
     }): Chainable<void>;
     deleteFile(fileName: string): Chainable<void>;
+    deleteFiles(fileNames: string[]): Chainable<void>;
     openDirectory(fileName: string): Chainable<void>;
     addFileToPipeline(fileName: string): Chainable<void>;
     dragAndDropFileToPipeline(fileName: string): Chainable<void>;

cypress/tests/pipeline.cy.ts

@@ -38,10 +38,12 @@ const emptyPipeline = `{
 
 describe('Pipeline Editor tests', () => {
   beforeEach(() => {
-    cy.deleteFile('generic-test.yaml'); // previously exported pipeline
-    cy.deleteFile('generic-test-custom.yaml'); // previously exported pipeline
-    cy.deleteFile('generic-test.py'); // previously exported pipeline
-    cy.deleteFile('*.pipeline'); // delete pipeline files used for testing
+    cy.deleteFiles([
+      'generic-test.yaml',
+      'generic-test-custom.yaml',
+      'generic-test.py',
+      '*.pipeline'
+    ]);
 
     cy.bootstrapFile('invalid.pipeline');
     cy.bootstrapFile('generic-test.pipeline');
@@ -55,18 +57,18 @@ describe('Pipeline Editor tests', () => {
   });
 
   afterEach(() => {
-    cy.deleteFile('helloworld.ipynb'); // delete notebook file used for testing
-    cy.deleteFile('helloworld.py'); // delete python file used for testing
-    cy.deleteFile('output.txt'); // delete output files generated by tests
-    cy.deleteFile('*.pipeline'); // delete pipeline files used for testing
-    cy.deleteFile('generic-test.yaml'); // exported pipeline
-    cy.deleteFile('generic-test-custom.yaml'); // exported pipeline
-    cy.deleteFile('generic-test.py'); // exported pipeline
-    cy.deleteFile('invalid.txt');
-
-    // delete complex test directories
-    cy.deleteFile('pipelines');
-    cy.deleteFile('scripts');
+    cy.deleteFiles([
+      'helloworld.ipynb',
+      'helloworld.py',
+      'output.txt',
+      '*.pipeline',
+      'generic-test.yaml',
+      'generic-test-custom.yaml',
+      'generic-test.py',
+      'invalid.txt',
+      'pipelines',
+      'scripts'
+    ]);
 
     // delete runtime configurations used for testing
     cy.exec('elyra-metadata remove runtimes --name=kfp_test_runtime', {

package.json

@@ -13,7 +13,8 @@
   "scripts": {
     "graph": "ts-node etc/scripts/generate-make-graph.ts",
     "cy:open": "npx cypress open",
-    "cy:run": "npx nyc npx cypress run --headed",
+    "cy:run": "npx nyc npx cypress run",
+    "cy:debug": "npx nyc npx cypress run --headed",
     "eslint": "eslint . --fix --ignore-path .gitignore --ext .ts,.tsx,.js",
     "eslint:check": "eslint . --ignore-path .gitignore --ext .ts,.tsx,.js",
     "prettier": "prettier --ignore-path .gitignore --write \"**/*{.ts,.tsx,.js,.jsx,.css,.json}\"",