Sending failed event logs to cli

index.js

@@ -1,6 +1,8 @@
 const { AppiumDriver } = require('./percy/driver/driverWrapper');
 const { ProviderResolver } = require('./percy/providers/providerResolver');
 const { TimeIt } = require('./percy/util/timing');
+
+const postFailedEvents = require('./percy/util/postFailedEvents');
 const percyOnAutomate = require('./percy/percyOnAutomate');
 
 const log = require('./percy/util/log');
@@ -61,6 +63,7 @@ module.exports = async function percyScreenshot(driver, name, options = {}) {
       [driver, name] = [browser, driver];
     } catch (e) { // ReferenceError: browser is not defined.
       driver = undefined;
+      await postFailedEvents(e);
     }
   };
   if (!driver) throw new Error('The WebdriverIO `browser` object or wd `driver` object is required.');
@@ -105,6 +108,7 @@ module.exports = async function percyScreenshot(driver, name, options = {}) {
     } catch (e) {
       log.error(`[${name}] failed to take screenshot`);
       log.debug(`[${name}] ${e}, \n ${e.stack}`);
+      await postFailedEvents(e);
       if (!(await driver.getPercyOptions()).ignoreErrors) throw e;
     }
   });

percy/util/postFailedEvents.js

@@ -0,0 +1,20 @@
+const utils = require('@percy/sdk-utils');
+
+// Collect client and environment information
+const sdkPkg = require('../../package.json');
+const CLIENT_INFO = `${sdkPkg.name.split('/')[1]}-js/${sdkPkg.version}`;
+
+module.exports = async function postFailedEvents(error) {
+  let options = {
+    clientInfo: CLIENT_INFO,
+    errorMessage: error
+  };
+
+  return await module.exports.request(options);
+};
+
+module.exports.request = async function request(data) {
+  try {
+    await utils.postFailedEvent(data);
+  } catch {}
+}; // To mock in test case

test/index.test.mjs

@@ -5,6 +5,7 @@ import wdioDriver from './mocks/appium/wdio_driver.js';
 import { Cache } from '../percy/util/cache.js';
 import utils from '@percy/sdk-utils';
 import percyOnAutomate from '../percy/percyOnAutomate.js';
+import postFailedEvents from '../percy/util/postFailedEvents.js';
 
 describe('percyScreenshot', () => {
   let driver;
@@ -25,6 +26,7 @@ describe('percyScreenshot', () => {
     });
 
     it('throws an error when a driver is not provided', async () => {
+      spyOn(postFailedEvents, 'request').and.callFake(() => {});
       await expectAsync(percyScreenshot())
         .toBeRejectedWithError('The WebdriverIO `browser` object or wd `driver` object is required.');
     });
@@ -52,8 +54,8 @@ describe('percyScreenshot', () => {
     describe('errors', () => {
       describe('with percy:options.ignoreErrors false', () => {
         it('logs errors if any', async () => {
+          spyOn(postFailedEvents, 'request').and.callFake(() => {});
           driver.takeScreenshot = jasmine.createSpy().and.throwError(new Error('Screenshot failed'));
-
           await expectAsync(percyScreenshot(driver, 'Screenshot 1'))
             .toBeRejectedWithError('Screenshot failed');
         });
@@ -62,6 +64,7 @@ describe('percyScreenshot', () => {
       describe('with percy:options.ignoreErrors true', () => {
         it('logs errors if any', async () => {
           driver = wdDriver({ ignoreErrors: true });
+          spyOn(postFailedEvents, 'request').and.callFake(() => {});
           driver.takeScreenshot = jasmine.createSpy().and.throwError(new Error('Screenshot failed'));
 
           await percyScreenshot(driver, 'Screenshot 1');
@@ -73,6 +76,7 @@ describe('percyScreenshot', () => {
   describe('wdio standalone context', () => {
     describe('with browser not defined', () => {
       it('throws an error when a driver is not provided', async () => {
+        spyOn(postFailedEvents, 'request').and.callFake(() => {});
         await expectAsync(percyScreenshot('Screenshot 1'))
           .toBeRejectedWithError('The WebdriverIO `browser` object or wd `driver` object is required.');
       });