should warn if pure option is present

Author: ryanbrainard

src/components/connect.js

@@ -32,6 +32,8 @@ function connectFactory(defaults = {}, options = {}) {
       'However, this custom Request would only be used in the default buildRequest.'
     )
 
+    warning(options.pure === undefined, '`pure` option is no longer supported')
+
     return connect(map, defaults, finalOptions)
   }
 

test/components/connect.spec.js

@@ -1280,7 +1280,6 @@ describe('React', () => {
       expect(invocationCount).toEqual(2)
     })
 
-    // TODO: clean up test
     it('should deprecate mapPropsToRequestsToProps with context', () => {
       let consoleSpy = expect.spyOn(console, 'error')
 
@@ -1370,6 +1369,25 @@ describe('React', () => {
       ])
     })
 
+    it('should warn if pure option is present', () => {
+      let consoleSpy = expect.spyOn(console, 'error')
+
+      @connect.options({ pure: false })(() => {
+        return {}
+      })
+      class C extends Component {
+        render() {
+          return <div />
+        }
+      }
+
+      TestUtils.renderIntoDocument(
+        <C />
+      )
+
+      expect(consoleSpy.calls[0].arguments[0]).toEqual('Warning: `pure` option is no longer supported')
+    })
+
     it('should shallowly compare the requests to prevent unnecessary fetches', (done) => {
       @connect(({ foo }) => ({
         testFetch: `/resource/${foo.FOO}`,