internal: Uncap test workers in CI (#3510)

Author: ntucker

.circleci/config.yml

@@ -118,15 +118,15 @@ jobs:
           name: Running Jest
           command: |
             if [ "<< parameters.react-version >>" == "^17" ]; then
-              yarn test:ci --maxWorkers=3 --selectProjects ReactDOM --testPathPatterns packages/react packages/use-enhanced-reducer packages/img
+              yarn test:ci --selectProjects ReactDOM --testPathPatterns packages/react packages/use-enhanced-reducer packages/img
             elif [ "<< parameters.react-version >>" == "^18" ]; then
-              yarn test:ci --maxWorkers=4 --selectProjects ReactDOM --testPathPatterns packages/react packages/use-enhanced-reducer packages/img
+              yarn test:ci --selectProjects ReactDOM --testPathPatterns packages/react packages/use-enhanced-reducer packages/img
             elif [ "<< parameters.react-version >>" == "native" ]; then
-              yarn test:ci --maxWorkers=4 --selectProjects ReactNative
+              yarn test:ci --selectProjects ReactNative
             else
               curl -Os https://uploader.codecov.io/latest/linux/codecov;
               chmod +x codecov;
-              yarn run test:coverage --ci --maxWorkers=3 --selectProjects ReactDOM Node --coverageReporters=text-lcov > ./lcov.info;
+              yarn run test:coverage --ci --selectProjects ReactDOM Node --coverageReporters=text-lcov > ./lcov.info;
               if [ "$CODECOV_TOKEN" != "" ]; then
                 ./codecov -t ${CODECOV_TOKEN} < ./lcov.info || true;
               else

packages/core/src/manager/__tests__/pollingSubscription.ts

@@ -394,6 +394,8 @@ describe('PollingSubscription', () => {
     });
 
     it('should not run when timeoutId is deleted after coming online', () => {
+      // Silence console.warn for this test
+      const warnSpy = jest.spyOn(console, 'warn').mockImplementation(() => {});
       const listener = new MockConnectionListener(false);
       const { dispatch, pollingSubscription } = createMocks(listener);
       expect(dispatch.mock.calls.length).toBe(0);
@@ -406,6 +408,8 @@ describe('PollingSubscription', () => {
       expect(dispatch.mock.calls.length).toBe(0);
       expect(listener.offlineHandlers.length).toBe(1);
       expect(listener.onlineHandlers.length).toBe(0);
+      expect(warnSpy.mock.calls.length).toBe(1);
+      warnSpy.mockRestore();
     });
 
     it('should stop dispatching when offline again', () => {

packages/endpoint/src/schemas/__tests__/All.test.ts

@@ -18,7 +18,6 @@ import { fromJSState } from './denormalize';
 let dateSpy: jest.Spied<typeof Date.now>;
 beforeAll(() => {
   dateSpy = jest
-
     .spyOn(global.Date, 'now')
     .mockImplementation(() => new Date('2019-05-14T11:01:58.135Z').valueOf());
 });
@@ -65,6 +64,7 @@ describe.each([[]])(`${schema.All.name} normalization (%s)`, () => {
     });
 
     test('normalizes multiple entities', () => {
+      const warnSpy = jest.spyOn(console, 'warn').mockImplementation(() => {});
       const inferSchemaFn = jest.fn(input => input.type || 'dogs');
       class Person extends IDEntity {}
       const listSchema = new schema.All(
@@ -84,6 +84,8 @@ describe.each([[]])(`${schema.All.name} normalization (%s)`, () => {
       expect(result).toBeUndefined();
       expect(entities).toMatchSnapshot();
       expect(inferSchemaFn.mock.calls).toMatchSnapshot();
+      expect(warnSpy.mock.calls).toMatchSnapshot();
+      warnSpy.mockRestore();
     });
 
     test('normalizes Objects using their values', () => {

packages/endpoint/src/schemas/__tests__/Array.test.js

@@ -94,6 +94,7 @@ describe.each([
 
   describe('Class', () => {
     class Cats extends IDEntity {}
+    class Dogs extends IDEntity {}
     test('normalizes a single entity', () => {
       const listSchema = createSchema(Cats);
       expect(
@@ -108,6 +109,7 @@ describe.each([
         {
           Cat: Cats,
           people: Person,
+          dogs: Dogs,
         },
         inferSchemaFn,
       );
@@ -123,6 +125,31 @@ describe.each([
       expect(inferSchemaFn.mock.calls).toMatchSnapshot();
     });
 
+    test('normalizes multiple entities warning when type is not found', () => {
+      const warnSpy = jest.spyOn(console, 'warn').mockImplementation(() => {});
+      const inferSchemaFn = jest.fn(input => input.type || 'dogs');
+      class Person extends IDEntity {}
+      const listSchema = new schema.Array(
+        {
+          Cat: Cats,
+          people: Person,
+        },
+        inferSchemaFn,
+      );
+
+      expect(
+        normalize(listSchema, [
+          { type: 'Cat', id: '123' },
+          { type: 'people', id: '123' },
+          { type: 'not found', id: '789' },
+          { type: 'Cat', id: '456' },
+        ]),
+      ).toMatchSnapshot();
+      expect(inferSchemaFn.mock.calls).toMatchSnapshot();
+      expect(warnSpy.mock.calls).toMatchSnapshot();
+      warnSpy.mockRestore();
+    });
+
     test('normalizes Objects using their values', () => {
       class User extends IDEntity {}
       const users = createSchema(User);

packages/endpoint/src/schemas/__tests__/__snapshots__/All.test.ts.snap

@@ -1,4 +1,4 @@
-// Jest Snapshot v1, https://goo.gl/fbAQLP
+// Jest Snapshot v1, https://jestjs.io/docs/snapshot-testing
 
 exports[`AllSchema normalization (%s) Class filters out undefined and null normalized values 1`] = `
 {
@@ -262,6 +262,20 @@ exports[`AllSchema normalization (%s) Class normalizes multiple entities 2`] = `
 ]
 `;
 
+exports[`AllSchema normalization (%s) Class normalizes multiple entities 3`] = `
+[
+  [
+    "Schema attribute "dogs" is not expected.
+Expected one of: "Cat", "people"
+
+Value: {
+  "id": "789",
+  "name": "fido"
+}",
+  ],
+]
+`;
+
 exports[`AllSchema normalization (%s) normalizes Objects using their values 1`] = `
 {
   "User": {