Switch to prop-types & PropTypes.checkPropTypes

PropTypes.checkPropTypes cannot throw yet (see [1]), so checking
console.error output

[1] facebook/prop-types#54

Author: ojab

package.json

@@ -32,5 +32,8 @@
     "eslint-plugin-import": "^2.2.0",
     "jest": "^21.2.1",
     "react": "^0.14.7"
+  },
+  "dependencies": {
+    "prop-types": "^15.6.0"
   }
 }

src/index.js

@@ -1,9 +1,4 @@
-function validateProp(validator, props, key, modelName) {
-    const result = validator(props, key, modelName, 'prop');
-    if (result instanceof Error) {
-        throw result;
-    }
-}
+import { PropTypes } from 'prop-types';
 
 function hasPropTypes(obj) {
     return typeof obj.propTypes === 'object';
@@ -14,9 +9,7 @@ function hasDefaultProps(obj) {
 }
 
 function validateProps(props, propTypes, modelName) {
-    Object.keys(propTypes).forEach((key) => {
-        validateProp(propTypes[key], props, key, modelName);
-    });
+    PropTypes.checkPropTypes(propTypes, props, 'prop', modelName);
 }
 
 export function getPropTypesMixin(userOpts) {

src/index.test.js

@@ -1,5 +1,4 @@
-/* eslint-disable import/no-extraneous-dependencies */
-import { PropTypes } from 'react';
+import { PropTypes } from 'prop-types';
 
 import propTypesMixin, { getPropTypesMixin } from './index';
 
@@ -9,6 +8,7 @@ describe('PropTypesMixin', () => {
     let modelInstance;
     let createSpy;
     let updateSpy;
+    let consoleErrorSpy;
 
     beforeEach(() => {
         Model = class {
@@ -27,6 +27,8 @@ describe('PropTypesMixin', () => {
 
         modelInstance = new ModelWithMixin();
         updateSpy = jest.spyOn(modelInstance, 'update');
+        consoleErrorSpy = jest.spyOn(global.console, 'error');
+        consoleErrorSpy.mockReset();
     });
 
     it('getPropTypesMixin works correctly', () => {
@@ -65,23 +67,26 @@ describe('PropTypesMixin', () => {
         };
 
         ModelWithMixin.create({ name: 'shouldnt raise error' });
+        expect(consoleErrorSpy.mock.calls.length).toBe(0);
 
-        const funcShouldThrow = () => ModelWithMixin.create({ notName: 'asd' });
-
-        expect(funcShouldThrow).toThrow('ModelWithMixin', 'name');
+        ModelWithMixin.create({ notName: 'asd' });
+        expect(consoleErrorSpy.mock.calls.length).toBe(1);
+        expect(consoleErrorSpy).toBeCalledWith('Warning: Failed prop type: The prop `name` is marked as required in `ModelWithMixin.create`, but its value is `undefined`.');
     });
 
     it('raises validation error on update correctly', () => {
         ModelWithMixin.propTypes = {
             name: PropTypes.string.isRequired,
             age: PropTypes.number.isRequired,
         };
+        expect(consoleErrorSpy.mock.calls.length).toBe(0);
+        const instance = new ModelWithMixin({ name: 'asd', age: 123 });
+        expect(consoleErrorSpy.mock.calls.length).toBe(0);
 
-        const instance = new ModelWithMixin();
-
-        const funcShouldThrow = () => instance.update({ name: 123 });
+        instance.update({ name: 123 });
 
-        expect(funcShouldThrow).toThrow('ModelWithMixin', 'name');
+        expect(consoleErrorSpy.mock.calls.length).toBe(1);
+        expect(consoleErrorSpy).toBeCalledWith('Warning: Failed prop type: Invalid prop `name` of type `number` supplied to `ModelWithMixin.update`, expected `string`.');
     });
 
     it('correctly uses defaultProps', () => {