patch(vest): move suite selectors out into their own module for easie…

…r consumption by external sources (#906)

packages/vest/src/core/ctx/ctx.ts

@@ -4,9 +4,9 @@ import { assign } from 'vest-utils';
 
 import { Isolate, IsolateTypes } from 'IsolateTypes';
 import { Modes } from 'Modes';
+import { SuiteSummary } from 'SuiteSummaryTypes';
 import VestTest from 'VestTest';
 import type { StateRef } from 'createStateRef';
-import { SuiteSummary } from 'genTestsSummary';
 import { generateIsolate } from 'generateIsolate';
 
 export default createContext<CTXType>((ctxRef, parentContext) =>

packages/vest/src/core/suite/produce/produceSuiteResult.ts

@@ -2,12 +2,8 @@ import { cache as createCache, assign } from 'vest-utils';
 
 import ctx from 'ctx';
 import genTestsSummary from 'genTestsSummary';
-import { getErrors, getWarnings } from 'getFailures';
-import { getErrorsByGroup, getWarningsByGroup } from 'getFailuresByGroup';
-import { hasErrors, hasWarnings } from 'hasFailures';
-import { hasErrorsByGroup, hasWarningsByGroup } from 'hasFailuresByGroup';
-import { isValid, isValidByGroup } from 'isValid';
 import { useStateRef, useTestsFlat, useSuiteName } from 'stateHooks';
+import { SuiteSelectors, suiteSelectors } from 'suiteSelectors';
 
 const cache = createCache(1);
 
@@ -21,38 +17,11 @@ export function produceSuiteResult(): SuiteResult {
     ctx.bind(ctxRef, () => {
       const summary = genTestsSummary();
       const suiteName = useSuiteName();
-      const ref = { summary };
-      return assign(summary, {
-        getErrors: ctx.bind(ref, getErrors),
-        getErrorsByGroup: ctx.bind(ref, getErrorsByGroup),
-        getWarnings: ctx.bind(ref, getWarnings),
-        getWarningsByGroup: ctx.bind(ref, getWarningsByGroup),
-        hasErrors: ctx.bind(ref, hasErrors),
-        hasErrorsByGroup: ctx.bind(ref, hasErrorsByGroup),
-        hasWarnings: ctx.bind(ref, hasWarnings),
-        hasWarningsByGroup: ctx.bind(ref, hasWarningsByGroup),
-        isValid: ctx.bind(ref, isValid),
-        isValidByGroup: ctx.bind(ref, isValidByGroup),
+      return assign(summary, suiteSelectors(summary), {
         suiteName,
       });
     })
   );
 }
 
-export type SuiteResult = ReturnType<typeof genTestsSummary> & {
-  /**
-   * Returns whether the suite as a whole is valid.
-   * Determined if there are no errors, and if no
-   * required fields are skipped.
-   */
-  isValid: typeof isValid;
-  isValidByGroup: typeof isValidByGroup;
-  hasErrors: typeof hasErrors;
-  hasWarnings: typeof hasWarnings;
-  getErrors: typeof getErrors;
-  getWarnings: typeof getWarnings;
-  hasErrorsByGroup: typeof hasErrorsByGroup;
-  hasWarningsByGroup: typeof hasWarningsByGroup;
-  getErrorsByGroup: typeof getErrorsByGroup;
-  getWarningsByGroup: typeof getWarningsByGroup;
-};
+export type SuiteResult = ReturnType<typeof genTestsSummary> & SuiteSelectors;

packages/vest/src/core/suite/produce/summaryGenerators/SuiteSummaryTypes.ts

@@ -0,0 +1,28 @@
+export type SuiteSummary = {
+  groups: Groups;
+  tests: Tests;
+  valid: boolean;
+} & SummaryBase;
+
+export type TestsContainer = Group | Tests;
+export type GroupTestSummary = SingleTestSummary;
+
+export type Groups = Record<string, Group>;
+export type Group = Record<string, GroupTestSummary>;
+export type Tests = Record<string, SingleTestSummary>;
+
+export type SingleTestSummary = SummaryBase & {
+  errors: string[];
+  warnings: string[];
+  valid: boolean;
+};
+
+type SummaryBase = {
+  errorCount: number;
+  warnCount: number;
+  testCount: number;
+};
+
+export type GetFailuresResponse = FailureMessages | string[];
+
+export type FailureMessages = Record<string, string[]>;

packages/vest/src/core/suite/produce/summaryGenerators/genTestsSummary.ts

@@ -1,6 +1,14 @@
 import { assign, invariant } from 'vest-utils';
 
 import { countKeyBySeverity, Severity } from 'Severity';
+import {
+  Group,
+  Groups,
+  SingleTestSummary,
+  SuiteSummary,
+  Tests,
+  TestsContainer,
+} from 'SuiteSummaryTypes';
 import VestTest from 'VestTest';
 import ctx from 'ctx';
 import {
@@ -140,28 +148,3 @@ function baseTestStats() {
     warnings: [],
   });
 }
-
-export type SuiteSummary = {
-  groups: Groups;
-  tests: Tests;
-  valid: boolean;
-} & SummaryBase;
-
-export type TestsContainer = Group | Tests;
-export type GroupTestSummary = SingleTestSummary;
-
-type Groups = Record<string, Group>;
-type Group = Record<string, GroupTestSummary>;
-type Tests = Record<string, SingleTestSummary>;
-
-type SingleTestSummary = SummaryBase & {
-  errors: string[];
-  warnings: string[];
-  valid: boolean;
-};
-
-type SummaryBase = {
-  errorCount: number;
-  warnCount: number;
-  testCount: number;
-};

packages/vest/src/core/suite/produce/summarySelectors/getFailures/__tests__/collectFailureMessages.test.ts → packages/vest/src/core/suite/produce/summarySelectors/__tests__/collectFailureMessages.test.ts

@@ -1,5 +1,5 @@
 // TODO: Verify this test file is not needed and delete it
-import { dummyTest } from '../../../../../../../testUtils/testDummy';
+import { dummyTest } from '../../../../../../testUtils/testDummy';
 
 import create from 'create';
 import group from 'group';

packages/vest/src/core/suite/produce/summarySelectors/getFailures/__tests__/getFailures.test.ts → packages/vest/src/core/suite/produce/summarySelectors/__tests__/getFailures.test.ts

@@ -1,4 +1,4 @@
-import { dummyTest } from '../../../../../../../testUtils/testDummy';
+import { dummyTest } from '../../../../../../testUtils/testDummy';
 
 import * as vest from 'vest';
 

packages/vest/src/core/suite/produce/summarySelectors/getFailures/__tests__/getFailuresByGroup.test.ts → packages/vest/src/core/suite/produce/summarySelectors/__tests__/getFailuresByGroup.test.ts

@@ -1,4 +1,4 @@
-import { dummyTest } from '../../../../../../../testUtils/testDummy';
+import { dummyTest } from '../../../../../../testUtils/testDummy';
 
 import group from 'group';
 import { create } from 'vest';

packages/vest/src/core/suite/produce/summarySelectors/hasFailuers/__tests__/hasFailures.test.ts → packages/vest/src/core/suite/produce/summarySelectors/__tests__/hasFailures.test.ts

@@ -1,6 +1,6 @@
 import faker from 'faker';
 
-import { dummyTest } from '../../../../../../../testUtils/testDummy';
+import { dummyTest } from '../../../../../../testUtils/testDummy';
 
 import * as vest from 'vest';
 

packages/vest/src/core/suite/produce/summarySelectors/hasFailuers/__tests__/hasFailuresByGroup.test.ts → packages/vest/src/core/suite/produce/summarySelectors/__tests__/hasFailuresByGroup.test.ts

@@ -1,6 +1,6 @@
 import faker from 'faker';
 
-import { dummyTest } from '../../../../../../../testUtils/testDummy';
+import { dummyTest } from '../../../../../../testUtils/testDummy';
 
 import * as vest from 'vest';
 

packages/vest/src/core/suite/produce/summarySelectors/getFailures/collectFailures.ts → packages/vest/src/core/suite/produce/summarySelectors/collectFailures.ts

@@ -1,7 +1,7 @@
 import { isPositive } from 'vest-utils';
 
 import { countKeyBySeverity, Severity } from 'Severity';
-import { TestsContainer } from 'genTestsSummary';
+import { FailureMessages, TestsContainer } from 'SuiteSummaryTypes';
 
 // calls collectAll or getByFieldName depending on whether fieldName is provided
 
@@ -41,5 +41,3 @@ function collectAll(
 
   return output;
 }
-
-export type FailureMessages = Record<string, string[]>;