Prettify

TrueWill · May 12, 2021 · 9ca3df3 · 9ca3df3
1 parent 9bcf6c7
commit 9ca3df3
Show file tree

Hide file tree

Showing 3 changed files with 32 additions and 56 deletions.
diff --git a/.eslintrc.js b/.eslintrc.js
@@ -5,9 +5,7 @@ module.exports = {
     ecmaVersion: 2017,
     project: './tsconfig.json'
   },
-  plugins: [
-    '@typescript-eslint',
-  ],
+  plugins: ['@typescript-eslint'],
   extends: [
     'eslint:recommended',
     'plugin:@typescript-eslint/eslint-recommended',

diff --git a/src/index.spec.ts b/src/index.spec.ts
@@ -1,10 +1,8 @@
 import * as scientist from './index';
 
 describe('experiment', () => {
-  const publishMock: jest.Mock<void, [scientist.Results<any[], any>]> = jest.fn<
-    void,
-    [scientist.Results<any[], any>]
-  >();
+  const publishMock: jest.Mock<void, [scientist.Results<any[], any>]> =
+    jest.fn<void, [scientist.Results<any[], any>]>();
 
   afterEach(() => {
     publishMock.mockClear();
@@ -270,10 +268,8 @@ describe('experiment', () => {
   });
 
   describe('when enabled option is specified', () => {
-    const candidateMock: jest.Mock<string, [string]> = jest.fn<
-      string,
-      [string]
-    >();
+    const candidateMock: jest.Mock<string, [string]> =
+      jest.fn<string, [string]>();
 
     afterEach(() => {
       candidateMock.mockClear();
@@ -651,10 +647,8 @@ describe('experimentAsync', () => {
   });
 
   describe('when function results differ', () => {
-    const publishMock: jest.Mock<
-      void,
-      [scientist.Results<[string], string>]
-    > = jest.fn<void, [scientist.Results<[string], string>]>();
+    const publishMock: jest.Mock<void, [scientist.Results<[string], string>]> =
+      jest.fn<void, [scientist.Results<[string], string>]>();
 
     afterEach(() => {
       publishMock.mockClear();
@@ -713,10 +707,8 @@ describe('experimentAsync', () => {
   });
 
   describe('when candidate rejects', () => {
-    const publishMock: jest.Mock<
-      void,
-      [scientist.Results<[], string>]
-    > = jest.fn<void, [scientist.Results<[], string>]>();
+    const publishMock: jest.Mock<void, [scientist.Results<[], string>]> =
+      jest.fn<void, [scientist.Results<[], string>]>();
 
     afterEach(() => {
       publishMock.mockClear();
@@ -774,10 +766,8 @@ describe('experimentAsync', () => {
   });
 
   describe('when control rejects', () => {
-    const publishMock: jest.Mock<
-      void,
-      [scientist.Results<[], string>]
-    > = jest.fn<void, [scientist.Results<[], string>]>();
+    const publishMock: jest.Mock<void, [scientist.Results<[], string>]> =
+      jest.fn<void, [scientist.Results<[], string>]>();
 
     afterEach(() => {
       publishMock.mockClear();
@@ -837,10 +827,8 @@ describe('experimentAsync', () => {
   });
 
   describe('when both reject', () => {
-    const publishMock: jest.Mock<
-      void,
-      [scientist.Results<[], string>]
-    > = jest.fn<void, [scientist.Results<[], string>]>();
+    const publishMock: jest.Mock<void, [scientist.Results<[], string>]> =
+      jest.fn<void, [scientist.Results<[], string>]>();
 
     afterEach(() => {
       publishMock.mockClear();
@@ -899,15 +887,11 @@ describe('experimentAsync', () => {
   });
 
   describe('when enabled option is specified', () => {
-    const publishMock: jest.Mock<
-      void,
-      [scientist.Results<[string], string>]
-    > = jest.fn<void, [scientist.Results<[string], string>]>();
+    const publishMock: jest.Mock<void, [scientist.Results<[string], string>]> =
+      jest.fn<void, [scientist.Results<[string], string>]>();
 
-    const candidateMock: jest.Mock<Promise<string>, [string]> = jest.fn<
-      Promise<string>,
-      [string]
-    >();
+    const candidateMock: jest.Mock<Promise<string>, [string]> =
+      jest.fn<Promise<string>, [string]>();
 
     afterEach(() => {
       publishMock.mockClear();
@@ -1041,10 +1025,8 @@ describe('experimentAsync', () => {
   });
 
   describe('when functions are slow', () => {
-    const publishMock: jest.Mock<
-      void,
-      [scientist.Results<[], string>]
-    > = jest.fn<void, [scientist.Results<[], string>]>();
+    const publishMock: jest.Mock<void, [scientist.Results<[], string>]> =
+      jest.fn<void, [scientist.Results<[], string>]>();
 
     afterEach(() => {
       publishMock.mockClear();

diff --git a/src/index.ts b/src/index.ts
@@ -2,10 +2,8 @@ export type ExperimentFunction<TParams extends any[], TResult> = (
   ...args: TParams
 ) => TResult;
 
-export type ExperimentAsyncFunction<
-  TParams extends any[],
-  TResult
-> = ExperimentFunction<TParams, Promise<TResult>>;
+export type ExperimentAsyncFunction<TParams extends any[], TResult> =
+  ExperimentFunction<TParams, Promise<TResult>>;
 
 export interface Results<TParams extends any[], TResult> {
   experimentName: string;
@@ -177,19 +175,17 @@ export function experimentAsync<TParams extends any[], TResult>({
 
     if (isEnabled) {
       // Run in parallel
-      [
-        [candidateResult, candidateTimeMs],
-        [controlResult, controlTimeMs]
-      ] = await Promise.all([
-        executeAndTime(candidate, args).catch((e) => {
-          candidateError = e;
-          return [undefined, undefined];
-        }),
-        executeAndTime(control, args).catch((e) => {
-          controlError = e;
-          return [undefined, undefined];
-        })
-      ]);
+      [[candidateResult, candidateTimeMs], [controlResult, controlTimeMs]] =
+        await Promise.all([
+          executeAndTime(candidate, args).catch((e) => {
+            candidateError = e;
+            return [undefined, undefined];
+          }),
+          executeAndTime(control, args).catch((e) => {
+            controlError = e;
+            return [undefined, undefined];
+          })
+        ]);
     } else {
       controlResult = await control(...args).catch((e) => {
         controlError = e;