@khanacademy/wonder-blocks-testing-core 1.0.0 → 1.0.2

package/CHANGELOG.md

@@ -1,5 +1,18 @@
 # @khanacademy/wonder-blocks-testing-core
 
+## 1.0.2
+
+### Patch Changes
+
+-   02a1b298: Make sure we don't package tsconfig and tsbuildinfo files
+
+## 1.0.1
+
+### Patch Changes
+
+-   c954464a: Fix how react-dom/server is imported in Wonder Blocks Testing Core
+-   559e82d5: Update to build tooling, generating smaller output
+
 ## 1.0.0
 
 ### Major Changes

package/dist/es/index.js

@@ -1,6 +1,8 @@
 import * as React from 'react';
 import { action } from '@storybook/addon-actions';
 import { StaticRouter, MemoryRouter, Switch, Route } from 'react-router-dom';
+import _extends from '@babel/runtime/helpers/extends';
+import * as ReactDOMServer from 'react-dom/server';
 
 const fixtures = Component => {
   const templateMap = new WeakMap();
@@ -347,21 +349,6 @@ var adapters = /*#__PURE__*/Object.freeze({
     DefaultConfigs: DefaultConfigs
 });
 
-function _extends() {
-  _extends = Object.assign ? Object.assign.bind() : function (target) {
-    for (var i = 1; i < arguments.length; i++) {
-      var source = arguments[i];
-      for (var key in source) {
-        if (Object.prototype.hasOwnProperty.call(source, key)) {
-          target[key] = source[key];
-        }
-      }
-    }
-    return target;
-  };
-  return _extends.apply(this, arguments);
-}
-
 const componentCache = new Map();
 const getNamedAdapterComponent = name => {
   const existing = componentCache.get(name);
@@ -423,12 +410,6 @@ const hookHarness = makeHookHarness(DefaultAdapters, DefaultConfigs);
 
 const testHarness = makeTestHarness(DefaultAdapters, DefaultConfigs);
 
-if (process.env.NODE_ENV === 'production') {
-  module.exports = require('./cjs/react-dom-server.browser.production.min.js');
-} else {
-  module.exports = require('./cjs/react-dom-server.browser.development.js');
-}
-
 const renderHookStatic = (render, {
   wrapper,
   initialProps
@@ -444,7 +425,7 @@ const renderHookStatic = (render, {
     renderCallbackProps: initialProps
   });
   const componentWithWrapper = wrapper == null ? component : React.createElement(wrapper, null, component);
-  undefined(componentWithWrapper);
+  ReactDOMServer.renderToString(componentWithWrapper);
   return {
     result: {
       current: result

package/dist/index.js

@@ -5,6 +5,10 @@ Object.defineProperty(exports, '__esModule', { value: true });
 var React = require('react');
 var addonActions = require('@storybook/addon-actions');
 var reactRouterDom = require('react-router-dom');
+var _extends = require('@babel/runtime/helpers/extends');
+var ReactDOMServer = require('react-dom/server');
+
+function _interopDefaultLegacy (e) { return e && typeof e === 'object' && 'default' in e ? e : { 'default': e }; }
 
 function _interopNamespace(e) {
     if (e && e.__esModule) return e;
@@ -25,6 +29,8 @@ function _interopNamespace(e) {
 }
 
 var React__namespace = /*#__PURE__*/_interopNamespace(React);
+var _extends__default = /*#__PURE__*/_interopDefaultLegacy(_extends);
+var ReactDOMServer__namespace = /*#__PURE__*/_interopNamespace(ReactDOMServer);
 
 const fixtures = Component => {
   const templateMap = new WeakMap();
@@ -371,21 +377,6 @@ var adapters = /*#__PURE__*/Object.freeze({
     DefaultConfigs: DefaultConfigs
 });
 
-function _extends() {
-  _extends = Object.assign ? Object.assign.bind() : function (target) {
-    for (var i = 1; i < arguments.length; i++) {
-      var source = arguments[i];
-      for (var key in source) {
-        if (Object.prototype.hasOwnProperty.call(source, key)) {
-          target[key] = source[key];
-        }
-      }
-    }
-    return target;
-  };
-  return _extends.apply(this, arguments);
-}
-
 const componentCache = new Map();
 const getNamedAdapterComponent = name => {
   const existing = componentCache.get(name);
@@ -423,11 +414,11 @@ const Adapt = ({
 
 const makeTestHarness = (adapters, defaultConfigs) => {
   return (Component, configs) => {
-    const fullConfig = _extends({}, defaultConfigs, configs);
+    const fullConfig = _extends__default["default"]({}, defaultConfigs, configs);
     const harnessedComponent = React__namespace.forwardRef((props, ref) => React__namespace.createElement(Adapt, {
       adapters: adapters,
       configs: fullConfig
-    }, React__namespace.createElement(Component, _extends({}, props, {
+    }, React__namespace.createElement(Component, _extends__default["default"]({}, props, {
       ref: ref
     }))));
     harnessedComponent.displayName = `testHarness(${Component.displayName || Component.name || "Component"})`;
@@ -447,12 +438,6 @@ const hookHarness = makeHookHarness(DefaultAdapters, DefaultConfigs);
 
 const testHarness = makeTestHarness(DefaultAdapters, DefaultConfigs);
 
-if (process.env.NODE_ENV === 'production') {
-  module.exports = require('./cjs/react-dom-server.browser.production.min.js');
-} else {
-  module.exports = require('./cjs/react-dom-server.browser.development.js');
-}
-
 const renderHookStatic = (render, {
   wrapper,
   initialProps
@@ -468,7 +453,7 @@ const renderHookStatic = (render, {
     renderCallbackProps: initialProps
   });
   const componentWithWrapper = wrapper == null ? component : React__namespace.createElement(wrapper, null, component);
-  undefined(componentWithWrapper);
+  ReactDOMServer__namespace.renderToString(componentWithWrapper);
   return {
     result: {
       current: result

package/package.json

@@ -1,33 +1,33 @@
 {
-    "name": "@khanacademy/wonder-blocks-testing-core",
-    "version": "1.0.0",
-    "design": "v1",
-    "publishConfig": {
-        "access": "public"
-    },
-    "description": "",
-    "main": "dist/index.js",
-    "module": "dist/es/index.js",
-    "types": "dist/index.d.ts",
-    "scripts": {
-        "test": "echo \"Error: no test specified\" && exit 1"
-    },
-    "dependencies": {
-        "@babel/runtime": "^7.18.6"
-    },
-    "peerDependencies": {
-        "@khanacademy/wonder-stuff-core": "^1.2.2",
-        "@storybook/addon-actions": "^7.0.0",
-        "aphrodite": "^1.2.5",
-        "node-fetch": "^2.6.7",
-        "react": "16.14.0",
-        "react-dom": "16.14.0",
-        "react-router-dom": "5.3.0"
-    },
-    "devDependencies": {
-        "@khanacademy/wb-dev-build-settings": "^1.0.0",
-        "@khanacademy/wonder-stuff-testing": "^3.0.1"
-    },
-    "author": "",
-    "license": "MIT"
-}
+  "name": "@khanacademy/wonder-blocks-testing-core",
+  "version": "1.0.2",
+  "design": "v1",
+  "publishConfig": {
+    "access": "public"
+  },
+  "description": "",
+  "main": "dist/index.js",
+  "module": "dist/es/index.js",
+  "types": "dist/index.d.ts",
+  "scripts": {
+    "test": "echo \"Error: no test specified\" && exit 1"
+  },
+  "dependencies": {
+    "@babel/runtime": "^7.18.6"
+  },
+  "peerDependencies": {
+    "@khanacademy/wonder-stuff-core": "^1.2.2",
+    "@storybook/addon-actions": "^7.0.0",
+    "aphrodite": "^1.2.5",
+    "node-fetch": "^2.6.7",
+    "react": "16.14.0",
+    "react-dom": "16.14.0",
+    "react-router-dom": "5.3.0"
+  },
+  "devDependencies": {
+    "@khanacademy/wb-dev-build-settings": "^1.0.1",
+    "@khanacademy/wonder-stuff-testing": "^3.0.1"
+  },
+  "author": "",
+  "license": "MIT"
+}

package/src/__tests__/mock-requester.test.ts

@@ -1,212 +0,0 @@
-import {RespondWith} from "../respond-with";
-import {mockRequester} from "../mock-requester";
-
-describe("#mockRequester", () => {
-    it("should return a function", () => {
-        // Arrange
-
-        // Act
-        const result = mockRequester(jest.fn(), jest.fn());
-
-        // Assert
-        expect(result).toBeInstanceOf(Function);
-    });
-
-    it("should provide mockOperation API", () => {
-        // Arrange
-
-        // Act
-        const result = mockRequester(jest.fn(), jest.fn());
-
-        // Assert
-        expect(result).toHaveProperty("mockOperation", expect.any(Function));
-    });
-
-    it("should provide mockOperationOnce API", () => {
-        // Arrange
-
-        // Act
-        const result = mockRequester(jest.fn(), jest.fn());
-
-        // Assert
-        expect(result).toHaveProperty(
-            "mockOperationOnce",
-            expect.any(Function),
-        );
-    });
-
-    it("should throw with helpful details formatted by operationToString if no matching mock is found", async () => {
-        // Arrange
-        const mockFn = mockRequester(
-            jest.fn(),
-            (...args: any) => `TEST FORMATTING: ${JSON.stringify(args)}`,
-        );
-
-        // Act
-        const underTest = mockFn("any", "arguments", {we: {want: 42}});
-
-        // Assert
-        await expect(underTest).rejects.toThrowErrorMatchingInlineSnapshot(`
-            "No matching mock response found for request:
-                TEST FORMATTING: ["any","arguments",{"we":{"want":42}}]"
-        `);
-    });
-
-    describe("mockOperation", () => {
-        it("should invoke matcher with mock for a request", async () => {
-            // Arrange
-            const matcher = jest.fn().mockReturnValue(true);
-            const operationToString = jest.fn();
-            const mockFn = mockRequester(matcher, operationToString);
-
-            // Act
-            mockFn.mockOperation(
-                "THE MOCK DESCRIPTION",
-                RespondWith.text("TADA!"),
-            );
-            await mockFn("any", "arguments", {we: {want: 42}});
-
-            // Assert
-            expect(matcher).toHaveBeenCalledWith(
-                "THE MOCK DESCRIPTION",
-                "any",
-                "arguments",
-                {
-                    we: {want: 42},
-                },
-            );
-        });
-
-        it("should return mocked operation response if matcher returns true", async () => {
-            // Arrange
-            const matcher = jest.fn().mockReturnValue(true);
-            const operationToString = jest.fn();
-            const mockFn = mockRequester(matcher, operationToString);
-
-            // Act
-            mockFn.mockOperation(
-                "THE MOCK DESCRIPTION",
-                RespondWith.text("TADA!"),
-            );
-            const response = await mockFn("DO SOMETHING");
-            const result = response.text();
-
-            // Assert
-            await expect(result).resolves.toBe("TADA!");
-        });
-
-        it("should skip mock if matcher returns false and try more mocks", async () => {
-            // Arrange
-            const matcher = jest
-                .fn()
-                .mockReturnValueOnce(false)
-                .mockReturnValueOnce(true);
-            const operationToString = jest.fn();
-            const mockFn = mockRequester(matcher, operationToString);
-
-            // Act
-            mockFn.mockOperation(
-                "THE MOCK DESCRIPTION 1",
-                RespondWith.text("ONE"),
-            );
-            mockFn.mockOperation(
-                "THE MOCK DESCRIPTION 2",
-                RespondWith.text("TWO"),
-            );
-            const response = await mockFn("DO SOMETHING");
-            const result = response.text();
-
-            // Assert
-            await expect(result).resolves.toBe("TWO");
-        });
-    });
-
-    describe("mockOperationOnce", () => {
-        it("should invoke matcher with mock for a request", async () => {
-            // Arrange
-            const matcher = jest.fn().mockReturnValue(true);
-            const operationToString = jest.fn();
-            const mockFn = mockRequester(matcher, operationToString);
-
-            // Act
-            mockFn.mockOperationOnce(
-                "THE MOCK DESCRIPTION",
-                RespondWith.text("TADA!"),
-            );
-            await mockFn("any", "arguments", {we: {want: 42}});
-
-            // Assert
-            expect(matcher).toHaveBeenCalledWith(
-                "THE MOCK DESCRIPTION",
-                "any",
-                "arguments",
-                {
-                    we: {want: 42},
-                },
-            );
-        });
-
-        it("should match once", async () => {
-            // Arrange
-            const matcher = jest.fn().mockReturnValue(true);
-            const operationToString = jest.fn();
-            const mockFn = mockRequester(matcher, operationToString);
-
-            // Act
-            mockFn.mockOperationOnce(
-                "THE MOCK DESCRIPTION",
-                RespondWith.text("TADA!"),
-            );
-            const response = await mockFn("DO SOMETHING");
-            const result = response.text();
-
-            // Assert
-            await expect(result).resolves.toBe("TADA!");
-        });
-
-        it("should only match once", async () => {
-            // Arrange
-            const matcher = jest.fn().mockReturnValue(true);
-            const operationToString = jest.fn();
-            const mockFn = mockRequester(matcher, operationToString);
-
-            // Act
-            mockFn.mockOperationOnce(
-                "THE MOCK DESCRIPTION",
-                RespondWith.text("TADA!"),
-            );
-            const result = Promise.all([
-                mockFn("DO SOMETHING"),
-                mockFn("DO SOMETHING"),
-            ]);
-
-            // Assert
-            await expect(result).rejects.toThrowError();
-        });
-
-        it("should skip mock if matcher returns false and try more mocks", async () => {
-            // Arrange
-            const matcher = jest
-                .fn()
-                .mockReturnValueOnce(false)
-                .mockReturnValueOnce(true);
-            const operationToString = jest.fn();
-            const mockFn = mockRequester(matcher, operationToString);
-
-            // Act
-            mockFn.mockOperationOnce(
-                "THE MOCK DESCRIPTION 1",
-                RespondWith.text("ONE"),
-            );
-            mockFn.mockOperationOnce(
-                "THE MOCK DESCRIPTION 2",
-                RespondWith.text("TWO"),
-            );
-            const response = await mockFn("DO SOMETHING");
-            const result = response.text();
-
-            // Assert
-            await expect(result).resolves.toBe("TWO");
-        });
-    });
-});

package/src/__tests__/render-hook-static.test.ts

@@ -1,48 +0,0 @@
-import {useLocation} from "react-router-dom";
-
-import {hookHarness} from "../harness/hook-harness";
-import {renderHookStatic} from "../render-hook-static";
-
-describe("renderHookStatic", () => {
-    it("should return the result of rendering the hook", () => {
-        // Arrange
-        const useTestHook = () => "Hello, World!";
-
-        // Act
-        const result = renderHookStatic(useTestHook);
-
-        // Assert
-        expect(result).toStrictEqual({result: {current: "Hello, World!"}});
-    });
-
-    it("should render the hook with the given initialProps", () => {
-        // Arrange
-        const useTestHook = (initialProps?: string) =>
-            initialProps ?? "BAD RESULT";
-
-        // Act
-        const result = renderHookStatic(useTestHook, {
-            initialProps: "Hello, World!",
-        });
-
-        // Assert
-        expect(result).toStrictEqual({result: {current: "Hello, World!"}});
-    });
-
-    it("should return the result of rendering the hook inside the wrapper", () => {
-        // Arrange
-        const Wrapper = hookHarness({
-            router: "/test/route",
-        });
-        const useTestHook = () => {
-            const location = useLocation();
-            return location.pathname;
-        };
-
-        // Act
-        const result = renderHookStatic(useTestHook, {wrapper: Wrapper});
-
-        // Assert
-        expect(result).toStrictEqual({result: {current: "/test/route"}});
-    });
-});