chore: bump Flow and ESLint preset (#241)

* chore: bump Flow and ESLint preset

* fix name of waitForElement

Author: thymikee

.eslintrc

@@ -3,6 +3,7 @@
   "rules": {
     "flowtype/no-weak-types": 0,
     "react-native/no-raw-text": 0,
-    "no-console": 1
+    "no-console": 1,
+    "react/no-multi-comp": 0
   }
 }

docs/API.md

@@ -318,14 +318,14 @@ expect(onEndReached).toHaveBeenCalled();
 Defined as:
 
 ```jsx
-function waitForExpect<T: *>(
+function waitForElement<T>(
   expectation: () => T,
   timeout: number = 4500,
   interval: number = 50
 ): Promise<T> {}
 ```
 
-Waits for non-deterministic periods of time until your element appears or times out. `waitForExpect` periodically calls `expectation` every `interval` milliseconds to determine whether the element appeared or not.
+Waits for non-deterministic periods of time until your element appears or times out. `waitForElement` periodically calls `expectation` every `interval` milliseconds to determine whether the element appeared or not.
 
 ```jsx
 import { render, waitForElement } from 'react-testing-library';

package.json

@@ -17,17 +17,15 @@
   "devDependencies": {
     "@babel/cli": "^7.1.2",
     "@babel/core": "^7.1.2",
-    "@callstack/eslint-config": "^7.0.0",
+    "@callstack/eslint-config": "^8.0.0",
     "@release-it/conventional-changelog": "^1.1.0",
     "@types/react": "^16.7.11",
     "@types/react-test-renderer": "^16.0.3",
-    "@typescript-eslint/eslint-plugin": "^1.10.2",
     "babel-jest": "^24.7.1",
-    "chalk": "^2.4.1",
     "conventional-changelog-cli": "^2.0.11",
     "dedent": "^0.7.0",
-    "eslint": "^6.0.1",
-    "flow-bin": "^0.104.0",
+    "eslint": "^6.7.2",
+    "flow-bin": "^0.107.0",
     "flow-copy-source": "^2.0.6",
     "jest": "^24.7.1",
     "metro-react-native-babel-preset": "^0.52.0",
@@ -36,14 +34,14 @@
     "react-test-renderer": "16.8.6",
     "release-it": "^12.3.3",
     "strip-ansi": "^5.2.0",
-    "typescript": "^3.5.3"
+    "typescript": "^3.7.3"
   },
   "dependencies": {
     "pretty-format": "^24.0.0"
   },
   "peerDependencies": {
     "react": ">=16.0.0",
-    "react-test-renderer": ">= 16.0.0"
+    "react-test-renderer": ">=16.0.0"
   },
   "scripts": {
     "test": "jest",
@@ -71,8 +69,5 @@
   },
   "publishConfig": {
     "registry": "https://registry.npmjs.org/"
-  },
-  "resolutions": {
-    "eslint-plugin-import": "2.18.0"
   }
 }

src/__tests__/a11yAPI.test.js

@@ -10,15 +10,14 @@ const TEXT_LABEL = 'cool text';
 const TEXT_HINT = 'static text';
 const TEXT_ROLE = 'link';
 const NO_MATCHES_TEXT = 'not-existent-element';
-
 const NO_INSTANCES_FOUND = 'No instances found';
 const FOUND_TWO_INSTANCES = 'Expected 1 but found 2 instances';
 
 const Typography = ({ children, ...rest }) => {
   return <Text {...rest}>{children}</Text>;
 };
 
-class Button extends React.Component<*> {
+class Button extends React.Component<any> {
   render() {
     return (
       <TouchableOpacity

src/__tests__/debug.test.js

@@ -12,7 +12,7 @@ function TextComponent({ text }) {
   return <Text>{text}</Text>;
 }
 
-class Button extends React.Component<*, *> {
+class Button extends React.Component<any, any> {
   state = { counter: 0 };
 
   onPress = async () => {

src/__tests__/render.test.js

@@ -1,5 +1,4 @@
 // @flow
-/* eslint-disable react/no-multi-comp */
 import React from 'react';
 import {
   View,
@@ -18,7 +17,7 @@ const PLACEHOLDER_CHEF = 'Who inspected freshness?';
 const INPUT_FRESHNESS = 'Custom Freshie';
 const INPUT_CHEF = 'I inspected freshie';
 
-class Button extends React.Component<*> {
+class Button extends React.Component<any> {
   render() {
     return (
       <TouchableOpacity onPress={this.props.onPress}>
@@ -28,7 +27,7 @@ class Button extends React.Component<*> {
   }
 }
 
-class Banana extends React.Component<*, *> {
+class Banana extends React.Component<any, any> {
   state = {
     fresh: false,
   };

src/__tests__/waitForElement.test.js

@@ -1,10 +1,9 @@
 // @flow
-/* eslint-disable react/no-multi-comp */
 import React from 'react';
 import { View, Text, TouchableOpacity } from 'react-native';
 import { render, fireEvent, waitForElement } from '..';
 
-class Banana extends React.Component<*, *> {
+class Banana extends React.Component<any> {
   changeFresh = () => {
     this.props.onChangeFresh();
   };
@@ -21,7 +20,7 @@ class Banana extends React.Component<*, *> {
   }
 }
 
-class BananaContainer extends React.Component<*, *> {
+class BananaContainer extends React.Component<{}, any> {
   state = { fresh: false };
 
   onChangeFresh = async () => {

src/helpers/a11yAPI.js

@@ -12,7 +12,7 @@ type ArrayQueryAllFn = (
 type ArrayGetFn = (string | Array<string>) => ReactTestInstance;
 type ArrayGetAllFn = (string | Array<string>) => Array<ReactTestInstance>;
 
-type A11yAPI = {
+type A11yAPI = {|
   getByA11yLabel: GetFn,
   getAllByA11yLabel: GetAllFn,
   queryByA11yLabel: QueryFn,
@@ -29,7 +29,7 @@ type A11yAPI = {
   getAllByA11yStates: ArrayGetAllFn,
   queryByA11yStates: ArrayQueryFn,
   queryAllByA11yStates: ArrayQueryAllFn,
-};
+|};
 
 export function matchStringValue(prop?: string, matcher: string | RegExp) {
   if (!prop) {
@@ -55,7 +55,6 @@ export function matchArrayValue(
     return prop.includes(matcher);
   }
 
-  // $FlowFixMe - callback is sync hence prop exists
   return !matcher.some(e => !prop.includes(e));
 }
 

src/helpers/getByAPI.js

@@ -69,7 +69,7 @@ const getTextInputNodeByDisplayValue = (node, value) => {
 };
 
 export const getByName = (instance: ReactTestInstance) =>
-  function getByNameFn(name: string | React.ComponentType<*>) {
+  function getByNameFn(name: string | React.ComponentType<any>) {
     logDeprecationWarning('getByName', 'getByType');
     try {
       return typeof name === 'string'
@@ -81,7 +81,7 @@ export const getByName = (instance: ReactTestInstance) =>
   };
 
 export const getByType = (instance: ReactTestInstance) =>
-  function getByTypeFn(type: React.ComponentType<*>) {
+  function getByTypeFn(type: React.ComponentType<any>) {
     try {
       return instance.findByType(type);
     } catch (error) {
@@ -139,7 +139,7 @@ export const getByTestId = (instance: ReactTestInstance) =>
   };
 
 export const getAllByName = (instance: ReactTestInstance) =>
-  function getAllByNameFn(name: string | React.ComponentType<*>) {
+  function getAllByNameFn(name: string | React.ComponentType<any>) {
     logDeprecationWarning('getAllByName', 'getAllByType');
     const results =
       typeof name === 'string'
@@ -152,7 +152,7 @@ export const getAllByName = (instance: ReactTestInstance) =>
   };
 
 export const getAllByType = (instance: ReactTestInstance) =>
-  function getAllByTypeFn(type: React.ComponentType<*>) {
+  function getAllByTypeFn(type: React.ComponentType<any>) {
     const results = instance.findAllByType(type);
     if (results.length === 0) {
       throw new ErrorWithStack('No instances found', getAllByTypeFn);

src/helpers/queryByAPI.js

@@ -19,7 +19,7 @@ import {
 import { logDeprecationWarning, createQueryByError } from './errors';
 
 export const queryByName = (instance: ReactTestInstance) =>
-  function queryByNameFn(name: string | React.ComponentType<*>) {
+  function queryByNameFn(name: string | React.ComponentType<any>) {
     logDeprecationWarning('queryByName', 'getByName');
     try {
       return getByName(instance)(name);
@@ -29,7 +29,7 @@ export const queryByName = (instance: ReactTestInstance) =>
   };
 
 export const queryByType = (instance: ReactTestInstance) =>
-  function queryByTypeFn(type: React.ComponentType<*>) {
+  function queryByTypeFn(type: React.ComponentType<any>) {
     try {
       return getByType(instance)(type);
     } catch (error) {
@@ -83,7 +83,7 @@ export const queryByTestId = (instance: ReactTestInstance) =>
   };
 
 export const queryAllByName = (instance: ReactTestInstance) => (
-  name: string | React.ComponentType<*>
+  name: string | React.ComponentType<any>
 ) => {
   logDeprecationWarning('queryAllByName', 'getAllByName');
   try {
@@ -94,7 +94,7 @@ export const queryAllByName = (instance: ReactTestInstance) => (
 };
 
 export const queryAllByType = (instance: ReactTestInstance) => (
-  type: React.ComponentType<*>
+  type: React.ComponentType<any>
 ) => {
   try {
     return getAllByType(instance)(type);