Tweak error messaging

Author: mjackson

packages/react-router/modules/Prompt.js

@@ -44,7 +44,7 @@ class Prompt extends React.Component {
   componentWillMount() {
     invariant(
       this.context.router,
-      'You should not use <Prompt> outside a valid <Router>'
+      'You should not use <Prompt> outside a <Router>'
     )
 
     if (this.props.when)

packages/react-router/modules/Redirect.js

@@ -37,7 +37,7 @@ class Redirect extends React.Component {
   componentWillMount() {
     invariant(
       this.context.router,
-      'You should not use <Redirect> outside a valid <Router>'
+      'You should not use <Redirect> outside a <Router>'
     )
 
     if (this.isStatic())

packages/react-router/modules/Route.js

@@ -59,7 +59,7 @@ class Route extends React.Component {
 
     invariant(
       router,
-      'You should not use <Route> or withRouter() outside a valid <Router>'
+      'You should not use <Route> or withRouter() outside a <Router>'
     )
 
     const { route } = router

packages/react-router/modules/Switch.js

@@ -22,7 +22,7 @@ class Switch extends React.Component {
   componentWillMount() {
     invariant(
       this.context.router,
-      'You should not use <Switch> outside a valid <Router>'
+      'You should not use <Switch> outside a <Router>'
     )
   }
 

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

@@ -88,14 +88,14 @@ describe('A <Route>', () => {
     expect(node.innerHTML).toContain('/sushi/spicy-tuna')
   })
 
-  it('crash explicitly with no valid <Router>', () => {
+  it('throws with no <Router>', () => {
     const node = document.createElement('div')
 
     expect(() => {
       ReactDOM.render((
         <Route path="/" render={() => null} />
       ), node)
-    }).toThrow(/You should not use <Route> or withRouter\(\) outside a valid <Router>/)
+    }).toThrow(/You should not use <Route> or withRouter\(\) outside a <Router>/)
   })
 })
 

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

@@ -135,7 +135,7 @@ describe('A <Switch>', () => {
     expect(node.innerHTML).toMatch(/one/)
   })
 
-  it('crash explicitly with no valid <Router>', () => {
+  it('throws with no <Router>', () => {
     const node = document.createElement('div')
 
     expect(() => {
@@ -149,7 +149,7 @@ describe('A <Switch>', () => {
           )}/>
         </Switch>
       ), node)
-    }).toThrow(/You should not use <Switch> outside a valid <Router>/)
+    }).toThrow(/You should not use <Switch> outside a <Router>/)
   })
 })