Adding testcases

percy · Dec 3, 2024 · 9d33293 · 9d33293
1 parent 92bb0c6
commit 9d33293
Show file tree

Hide file tree

Showing 2 changed files with 33 additions and 0 deletions.
diff --git a/index.js b/index.js
@@ -165,6 +165,9 @@ const percySnapshot = async function percySnapshot(driver, name, options) {
     // Handle errors
     log.error(`Could not take DOM snapshot "${name}"`);
     log.error(error);
+    if (process.env.PERCY_RAISE_ERROR === 'true') {
+      throw error;
+    }
   }
 };
 

diff --git a/test/index.test.mjs b/test/index.test.mjs
@@ -212,6 +212,21 @@ describe('percySnapshot', () => {
     ]));
     expect(error).toBeInstanceOf(Error);
   });
+
+  it('handles snapshot failures if PERCY_RAISE_ERROR is true', async () => {
+    process.env.PERCY_RAISE_ERROR = 'true';
+    await helpers.test('error', '/percy/snapshot');
+    let error = null;
+    try {
+      await percySnapshot(driver, 'Snapshot 1');
+    } catch (e) {
+      error = e;
+    }
+    expect(helpers.logger.stderr).toEqual(jasmine.arrayContaining([
+      '[percy] Could not take DOM snapshot "Snapshot 1"'
+    ]));
+    expect(error).toBeInstanceOf(Error);
+  });
 });
 
 describe('percyScreenshot', () => {
@@ -358,6 +373,21 @@ describe('percyScreenshot', () => {
     expect(error).toBeInstanceOf(Error);
   });
 
+  it('handles snapshot failures if PERCY_RAISE_ERROR is true', async () => {
+    process.env.PERCY_RAISE_ERROR = 'true';
+    await helpers.test('error', '/percy/automateScreenshot');
+    let error = null;
+    try {
+      await percyScreenshot(driver, 'Snapshot 1');
+    } catch (e) {
+      error = e;
+    }
+    expect(helpers.logger.stderr).toEqual(jasmine.arrayContaining([
+      '[percy] Could not take Screenshot "Snapshot 1"'
+    ]));
+    expect(error).toBeInstanceOf(Error);
+  });
+
   it('throws error for web session', async () => {
     spyOn(percySnapshot, 'isPercyEnabled').and.returnValue(Promise.resolve(true));
     utils.percy.type = 'web';