@khanacademy/wonder-blocks-testing-core 2.1.1 → 2.2.1

package/CHANGELOG.md

@@ -1,5 +1,17 @@
 # @khanacademy/wonder-blocks-testing-core
 
+## 2.2.1
+
+### Patch Changes
+
+- 1d7be37: Use pnpm catalog to pin dependency versions across packages
+
+## 2.2.0
+
+### Minor Changes
+
+- eb47d37: Improve error messaging for the router test harness adapter
+
 ## 2.1.1
 
 ### Patch Changes

package/dist/es/index.js

@@ -1,6 +1,6 @@
 import * as React from 'react';
 import { action } from '@storybook/addon-actions';
-import { StaticRouter, MemoryRouter, Switch, Route } from 'react-router-dom';
+import { StaticRouter, MemoryRouter, useLocation, Switch, Route } from 'react-router-dom';
 import _extends from '@babel/runtime/helpers/extends';
 import * as ReactDOMServer from 'react-dom/server';
 
@@ -300,7 +300,13 @@ const adapter$1 = (children, config) => React.createElement(React.Fragment, null
 const defaultConfig = {
   location: "/"
 };
-const maybeWithRoute = (children, path) => {
+const MaybeWithRoute = ({
+  children,
+  path,
+  configLocation
+}) => {
+  const actualLocation = useLocation();
+  const configuredLocation = typeof configLocation === "string" ? configLocation : configLocation.pathname;
   if (path == null) {
     return React.createElement(React.Fragment, null, children);
   }
@@ -310,27 +316,33 @@ const maybeWithRoute = (children, path) => {
   }, children), React.createElement(Route, {
     path: "*",
     render: () => {
-      throw new Error("The configured path must match the configured location or your harnessed component will not render.");
+      throw new Error(`The current location '${actualLocation.pathname}' ` + `does not match the configured path '${path}'. ` + `Did you provide the correct configured ` + `location, '${configuredLocation}', or did the ` + `routing lead to a different place than you ` + `expected?`);
     }
   }));
 };
 const adapter = (children, config) => {
+  var _config$initialIndex;
   if (typeof config === "string") {
     config = {
       location: config
     };
   }
-  const wrappedWithRoute = maybeWithRoute(children, config.path);
   if ("forceStatic" in config && config.forceStatic) {
     return React.createElement(StaticRouter, {
       location: config.location,
       context: {}
-    }, wrappedWithRoute);
+    }, React.createElement(MaybeWithRoute, {
+      path: config.path,
+      configLocation: config.location
+    }, children));
   }
   if ("location" in config && config.location !== undefined) {
     return React.createElement(MemoryRouter, {
       initialEntries: [config.location]
-    }, wrappedWithRoute);
+    }, React.createElement(MaybeWithRoute, {
+      path: config.path,
+      configLocation: config.location
+    }, children));
   }
   if (!("initialEntries" in config) || config.initialEntries === undefined) {
     throw new Error("A location or initial history entries must be provided.");
@@ -345,7 +357,10 @@ const adapter = (children, config) => {
   if (config.getUserConfirmation != null) {
     routerProps.getUserConfirmation = config.getUserConfirmation;
   }
-  return React.createElement(MemoryRouter, routerProps, wrappedWithRoute);
+  return React.createElement(MemoryRouter, routerProps, React.createElement(MaybeWithRoute, {
+    path: config.path,
+    configLocation: entries[(_config$initialIndex = config.initialIndex) != null ? _config$initialIndex : 0]
+  }, children));
 };
 
 const DefaultAdapters = {

package/dist/index.js

@@ -328,7 +328,13 @@ const adapter$1 = (children, config) => React__namespace.createElement(React__na
 const defaultConfig = {
   location: "/"
 };
-const maybeWithRoute = (children, path) => {
+const MaybeWithRoute = ({
+  children,
+  path,
+  configLocation
+}) => {
+  const actualLocation = reactRouterDom.useLocation();
+  const configuredLocation = typeof configLocation === "string" ? configLocation : configLocation.pathname;
   if (path == null) {
     return React__namespace.createElement(React__namespace.Fragment, null, children);
   }
@@ -338,27 +344,33 @@ const maybeWithRoute = (children, path) => {
   }, children), React__namespace.createElement(reactRouterDom.Route, {
     path: "*",
     render: () => {
-      throw new Error("The configured path must match the configured location or your harnessed component will not render.");
+      throw new Error(`The current location '${actualLocation.pathname}' ` + `does not match the configured path '${path}'. ` + `Did you provide the correct configured ` + `location, '${configuredLocation}', or did the ` + `routing lead to a different place than you ` + `expected?`);
     }
   }));
 };
 const adapter = (children, config) => {
+  var _config$initialIndex;
   if (typeof config === "string") {
     config = {
       location: config
     };
   }
-  const wrappedWithRoute = maybeWithRoute(children, config.path);
   if ("forceStatic" in config && config.forceStatic) {
     return React__namespace.createElement(reactRouterDom.StaticRouter, {
       location: config.location,
       context: {}
-    }, wrappedWithRoute);
+    }, React__namespace.createElement(MaybeWithRoute, {
+      path: config.path,
+      configLocation: config.location
+    }, children));
   }
   if ("location" in config && config.location !== undefined) {
     return React__namespace.createElement(reactRouterDom.MemoryRouter, {
       initialEntries: [config.location]
-    }, wrappedWithRoute);
+    }, React__namespace.createElement(MaybeWithRoute, {
+      path: config.path,
+      configLocation: config.location
+    }, children));
   }
   if (!("initialEntries" in config) || config.initialEntries === undefined) {
     throw new Error("A location or initial history entries must be provided.");
@@ -373,7 +385,10 @@ const adapter = (children, config) => {
   if (config.getUserConfirmation != null) {
     routerProps.getUserConfirmation = config.getUserConfirmation;
   }
-  return React__namespace.createElement(reactRouterDom.MemoryRouter, routerProps, wrappedWithRoute);
+  return React__namespace.createElement(reactRouterDom.MemoryRouter, routerProps, React__namespace.createElement(MaybeWithRoute, {
+    path: config.path,
+    configLocation: entries[(_config$initialIndex = config.initialIndex) != null ? _config$initialIndex : 0]
+  }, children));
 };
 
 const DefaultAdapters = {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@khanacademy/wonder-blocks-testing-core",
-  "version": "2.1.1",
+  "version": "2.2.1",
   "design": "v1",
   "publishConfig": {
     "access": "public"