Merge pull request #1 from mfrachet/feature/prettier

feature(prettier): Adding prettier

.eslintrc.js

@@ -1,15 +1,13 @@
 module.exports = {
-  "extends": "airbnb",
-  "plugins": [
-    "react",
-    "jsx-a11y",
-    "import"
-  ],
-  "rules": {
-    "react/jsx-filename-extension": "off",
-    "jsx-a11y/img-has-alt": "off"
+  extends: 'airbnb',
+  plugins: ['react', 'jsx-a11y', 'import'],
+  rules: {
+    'react/jsx-filename-extension': 'off',
+    'jsx-a11y/img-has-alt': 'off',
+    'jsx-a11y/href-no-hash': 'off',
+    'import/no-extraneous-dependencies': 'off',
+  },
+  env: {
+    jest: true,
   },
-  "env": {
-    "jest": true
-  }
 };

Example/src/FlatList.js

@@ -5,7 +5,7 @@ import Title from './Title';
 
 export default class CustomFlatList extends React.Component {
   static renderRow({ item }) {
-    return <RowItem item={{ title: item }} />
+    return <RowItem item={{ title: item }} />;
   }
 
   /**
@@ -14,12 +14,9 @@ export default class CustomFlatList extends React.Component {
   render() {
     return (
       <View>
-        <Title value="Variant FlatList" color="#5C6BC0"/>
-        <FlatList
-          data={this.props.data}
-          renderItem={CustomFlatList.renderRow}
-        />
+        <Title value="Variant FlatList" color="#5C6BC0" />
+        <FlatList data={this.props.data} renderItem={CustomFlatList.renderRow} />
       </View>
     );
   }
-}
+}

Example/src/List.ab-test.js

@@ -2,7 +2,8 @@ import AbHoc from 'react-native-ab-hoc';
 import FlatList from './FlatList';
 import ListView from './ListView';
 
-export default AbHoc('ListView',
+export default AbHoc(
+  'ListView',
   { variant: 'ListView', component: ListView },
-  { variant: 'FlatList', component: FlatList }
+  { variant: 'FlatList', component: FlatList },
 );

Example/src/ListView.js

@@ -4,28 +4,25 @@ import RowItem from './RowItem';
 import Title from './Title';
 
 export default class CustomListView extends React.Component {
-	constructor(props) {
-	  super(props);
+  constructor(props) {
+    super(props);
     const ds = new ListView.DataSource({ rowHasChanged: (r1, r2) => r1 !== r2 });
     this.state = { dataSource: ds.cloneWithRows(props.data) };
   }
 
   static renderRow(title) {
-    return <RowItem item={{ title }} />
+    return <RowItem item={{ title }} />;
   }
 
-	/**
+  /**
 	 * Render the CustomListView
 	 */
-	render() {
-		return (
-		  <View>
-        <Title value="Variant ListView" color="#EF5350"/>
-        <ListView
-          dataSource={this.state.dataSource}
-          renderRow={CustomListView.renderRow}
-        />
+  render() {
+    return (
+      <View>
+        <Title value="Variant ListView" color="#EF5350" />
+        <ListView dataSource={this.state.dataSource} renderRow={CustomListView.renderRow} />
       </View>
-		);
-	}
-}
+    );
+  }
+}

Example/src/RowItem.js

@@ -8,6 +8,10 @@ const styles = {
   },
 };
 
-export default function({ item }) {
-  return <View><Text style={styles.text}>{item.title}</Text></View>
+export default function ({ item }) {
+  return (
+    <View>
+      <Text style={styles.text}>{item.title}</Text>
+    </View>
+  );
 }

Example/src/Title.js

@@ -18,5 +18,5 @@ export default function ({ value, color }) {
     <View>
       <Text style={style}>{value}</Text>
     </View>
-  )
+  );
 }

Example/src/data.js

@@ -19,4 +19,4 @@ export default [
   'Shanti Solem',
   'Yuri Doughtie',
   'Wilford Meredith',
-];
+];

package.json

@@ -3,32 +3,37 @@
   "version": "0.0.1",
   "main": "index.js",
   "scripts": {
+    "format":
+      "prettier-eslint --write \"src/**/*.js\" && prettier-eslint --write \"Example/src/**/*.js\"",
     "lint": "eslint ./src",
-    "test": "./node_modules/.bin/jest --coverage",
+    "test": "./node_modules/.bin/jest --coverage --no-cache",
     "test:auto": "./node_modules/.bin/jest --watch"
   },
   "dependencies": {
-    "react": "16.0.0-alpha.6",
-    "react-native": "0.44.0"
+    "prop-types": "^15.5.10"
   },
   "devDependencies": {
-    "babel-jest": "19.0.0",
-    "babel-preset-react-native": "1.9.1",
+    "babel-jest": "21.2.0",
+    "babel-preset-react-native": "4.0.0",
     "coveralls": "^2.13.1",
-    "enzyme": "^2.9.1",
-    "eslint": "^3.19.0",
-    "eslint-config-airbnb": "^15.0.1",
-    "eslint-plugin-import": "^2.6.1",
-    "eslint-plugin-jsx-a11y": "^5.1.0",
+    "enzyme": "^3.1.0",
+    "enzyme-adapter-react-16": "^1.0.2",
+    "eslint": "^4.1.1",
+    "eslint-config-airbnb": "^15.0.2",
+    "eslint-plugin-import": "^2.2.0",
+    "eslint-plugin-jsx-a11y": "^6.0.2",
     "eslint-plugin-react": "^7.1.0",
-    "jest": "^20.0.0",
-    "react-native-mock": "^0.3.1",
-    "react-test-renderer": "16.0.0-alpha.12"
+    "jest": "21.2.1",
+    "prettier": "^1.6.1",
+    "prettier-eslint-cli": "^4.3.0",
+    "react": "16.0.0-beta.5",
+    "react-dom": "16.0.0-beta.5",
+    "react-native": "0.49.3",
+    "react-test-renderer": "16.0.0-beta.5"
   },
   "jest": {
     "preset": "react-native",
-    "setupFiles": [
-      "./setup/test.setup.js"
-    ]
+    "setupFiles": ["./setup/test.setup.js"],
+    "modulePathIgnorePatterns": ["<rootDir>/Example/"]
   }
 }

setup/test.setup.js

@@ -1,2 +1,4 @@
-/* eslint-disable */
-require('react-native-mock/mock');
+import Enzyme from 'enzyme';
+import Adapter from 'enzyme-adapter-react-16';
+
+Enzyme.configure({ adapter: new Adapter() });

src/__tests__/reactNativeAbHoc.test.js

@@ -15,11 +15,24 @@ describe('ReactNativeAbHoc', () => {
     const getItem = () => Promise.resolve('A');
     jest.setMock('AsyncStorage', { setItem: jest.fn(setItem), getItem: jest.fn(getItem) });
 
-    A = () => (<View><Text>A</Text></View>);
-    B = () => (<View><Text>B</Text></View>);
-    C = () => (<View><Text>C</Text></View>);
+    A = () => (
+      <View>
+        <Text>A</Text>
+      </View>
+    );
+    B = () => (
+      <View>
+        <Text>B</Text>
+      </View>
+    );
+    C = () => (
+      <View>
+        <Text>C</Text>
+      </View>
+    );
 
-    Component = ReactNativeAbHoc('Experiment',
+    Component = ReactNativeAbHoc(
+      'Experiment',
       { variant: 'A', component: A },
       { variant: 'B', component: B },
       { variant: 'C', component: C },
@@ -36,7 +49,9 @@ describe('ReactNativeAbHoc', () => {
     });
 
     it('should have thrown an error if the variant variant doesnt exist', () => {
-      expect(() => shallow(<Component variant="X" />)).toThrow('The variant named "X" doesn\'t exist in the current experient "Experiment');
+      expect(() => shallow(<Component variant="X" />)).toThrow(
+        'The variant named "X" doesn\'t exist in the current experient "Experiment',
+      );
     });
 
     it('should have returned the second component', () => {

src/reactNativeAbHoc.js

@@ -1,4 +1,5 @@
 import React from 'react';
+import PropTypes from 'prop-types';
 import { View, AsyncStorage } from 'react-native';
 
 const LOCAL_STORAGE_KEY = 'abhoc-variant';
@@ -8,7 +9,9 @@ export default (experiment, ...variants) => {
     static findVariant(variant) {
       const chosenVariant = variants.find(v => v.variant === variant);
       if (!chosenVariant) {
-        throw new Error(`The variant named "${variant}" doesn't exist in the current experient "${experiment}"`);
+        throw new Error(
+          `The variant named "${variant}" doesn't exist in the current experient "${experiment}"`,
+        );
       }
       return chosenVariant;
     }
@@ -56,7 +59,7 @@ export default (experiment, ...variants) => {
   }
 
   AbHoc.propTypes = {
-    variant: React.PropTypes.string,
+    variant: PropTypes.string,
   };
 
   AbHoc.defaultProps = {