feat: do not take screenshots in argos folder

DavidAtImpactCubed · Sep 2, 2022 · 27c594e · 27c594e
1 parent 8f9a25a
commit 27c594e
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 9 deletions.
diff --git a/cypress/e2e/argosScreenshot.cy.js b/cypress/e2e/argosScreenshot.cy.js
@@ -1,6 +1,6 @@
 const screenshotsFolder = Cypress.browser.isHeaded
-  ? `./cypress/screenshots/argos`
-  : `./cypress/screenshots/${Cypress.spec.name}/argos`;
+  ? `./cypress/screenshots`
+  : `./cypress/screenshots/${Cypress.spec.name}`;
 
 describe("argosScreenshot", () => {
   before(() => {
@@ -24,7 +24,7 @@ describe("argosScreenshot", () => {
 
     it("takes a screenshot with generic name", () => {
       cy.readFile(
-        `${screenshotsFolder}/argosScreenshot without name before all hook.png`
+        `${screenshotsFolder}/argosScreenshot -- without name -- waits for loader hiding -- before all hook.png`
       );
     });
   });

diff --git a/support.js b/support.js
@@ -54,9 +54,6 @@ function waitUntilNoBusy() {
   );
 }
 
-/**
- * Takes a screenshot for Argos.
- */
 Cypress.Commands.add(
   "argosScreenshot",
   { prevSubject: ["optional", "element", "window", "document"] },
@@ -66,11 +63,10 @@ Cypress.Commands.add(
       name = null;
     }
 
-    const screenshotName = `argos/${name || cy.state("runnable").fullTitle()}`;
     Cypress.log({
       name: "argosScreenshot",
       displayName: `Argos Screenshot`,
-      message: screenshotName,
+      message: name,
     });
     // Inject styles
     cy.document().then((doc) => injectStyles(doc));
@@ -80,7 +76,7 @@ Cypress.Commands.add(
     cy.document().its("fonts.status").should("equal", "loaded");
 
     // Screenshot
-    cy.wrap(subject).screenshot(screenshotName, {
+    cy.wrap(subject).screenshot(name, {
       blackout: ['[data-visual-test="blackout"]'].concat(
         options.blackout || []
       ),