call createLocation on 'to' regardless of type (#6690)

Author: MCAxiaz

packages/react-router/modules/Redirect.js

@@ -43,10 +43,7 @@ function Redirect({ computedMatch, to, push = false }) {
               method(location);
             }}
             onUpdate={(self, prevProps) => {
-              const prevLocation =
-                typeof prevProps.to === "string"
-                  ? createLocation(prevProps.to)
-                  : prevProps.to;
+              const prevLocation = createLocation(prevProps.to);
               if (
                 !locationsAreEqual(prevLocation, {
                   ...location,

packages/react-router/modules/__tests__/Redirect-test.js

@@ -23,7 +23,7 @@ describe("A <Redirect>", () => {
       }).not.toThrow();
     });
 
-    it("doesn't break / throw when rendered with location `to`", () => {
+    it("doesn't break / throw when rendered with location `to` created from string", () => {
       const to = createLocation("/go-out?search=foo#hash");
       expect(() => {
         renderStrict(
@@ -34,6 +34,40 @@ describe("A <Redirect>", () => {
         );
       }).not.toThrow();
     });
+
+    it("doesn't break / throw when rendered with object `to`", () => {
+      const to = {
+        pathname: "/path",
+        state: {
+          someState: "state"
+        }
+      };
+      expect(() => {
+        renderStrict(
+          <MemoryRouter>
+            <Redirect to={to} />
+          </MemoryRouter>,
+          node
+        );
+      }).not.toThrow();
+    });
+
+    it("doesn't break / throw when rendered with location `to` created from object", () => {
+      const to = createLocation({
+        pathname: "/path",
+        state: {
+          someState: "state"
+        }
+      });
+      expect(() => {
+        renderStrict(
+          <MemoryRouter>
+            <Redirect to={to} />
+          </MemoryRouter>,
+          node
+        );
+      }).not.toThrow();
+    });
   });
 
   describe("inside a <Switch>", () => {