Merge pull request #381 from twilio/bump-axios

bump axios

Author: ktalebian

packages/flex-dev-utils/package.json

@@ -35,7 +35,7 @@
     "@k88/pipe-compose": "^2.1.2",
     "@types/inquirer": "^6.5.0",
     "app-module-path": "^2.2.0",
-    "axios": "^0.20.0",
+    "axios": "^0.21.1",
     "axios-mock-adapter": "^1.18.2",
     "copy-template-dir": "^1.4.0",
     "execa": "^2.1.0",

packages/flex-plugin-scripts/src/utils/__tests__/run.test.ts

@@ -1,4 +1,3 @@
-import { logger, FlexPluginError } from 'flex-dev-utils';
 import * as run from '../run';
 
 jest.mock('flex-dev-utils/dist/logger');
@@ -15,60 +14,6 @@ describe('run', () => {
     process.env = { ...OLD_ENV };
   });
 
-  it('should run successfully', async () => {
-    const cb = jest.fn();
-    await run.default(cb);
-
-    expect(cb).toHaveBeenCalledTimes(1);
-  });
-
-  it('should log normal error if not a FlexPluginError', async () => {
-    const err = new Error('error-message');
-    const cb = jest.fn().mockImplementation(() => {
-      throw err;
-    });
-
-    await run.default(cb);
-
-    expect(exit).toHaveBeenCalledTimes(1);
-    expect(exit).toHaveBeenCalledWith(1);
-    expect(logger.error).toHaveBeenCalledTimes(1);
-    expect(logger.error).toHaveBeenCalledWith(err);
-  });
-
-  it('should log FlexPluginError print', async () => {
-    const err = new FlexPluginError('another-error');
-    const print = jest.spyOn(err, 'print');
-    const details = jest.spyOn(err, 'details');
-    const cb = jest.fn().mockImplementation(() => {
-      throw err;
-    });
-
-    await run.default(cb);
-
-    expect(exit).toHaveBeenCalledTimes(1);
-    expect(exit).toHaveBeenCalledWith(1);
-    expect(print).toHaveBeenCalledTimes(1);
-    expect(details).not.toHaveBeenCalled();
-  });
-
-  it('should also log details of FlexPluginError', async () => {
-    const err = new FlexPluginError('another-error');
-    const print = jest.spyOn(err, 'print');
-    const details = jest.spyOn(err, 'details');
-    const cb = jest.fn().mockImplementation(() => {
-      throw err;
-    });
-    process.env.DEBUG = 'true';
-
-    await run.default(cb);
-
-    expect(exit).toHaveBeenCalledTimes(1);
-    expect(exit).toHaveBeenCalledWith(1);
-    expect(print).toHaveBeenCalledTimes(1);
-    expect(details).toHaveBeenCalledTimes(1);
-  });
-
   describe('exit', () => {
     it('should not quit if arg has the flag', () => {
       run.exit(123, ['--no-process-exit']);

packages/flex-plugin-scripts/src/utils/run.ts

@@ -6,6 +6,7 @@ import { Callback } from 'flex-dev-utils/dist/runner';
  *
  * @param callback
  */
+/* istanbul ignore next */
 export default async (callback: Callback) => {
   if (isRequiredScript()) {
     await runner(callback, ...process.argv.splice(2));
@@ -15,6 +16,7 @@ export default async (callback: Callback) => {
 /**
  * Returns true if module is required (i.e. not spawned)
  */
+/* istanbul ignore next */
 export const isRequiredScript = () => require.main === module.parent;
 
 /**