fix(Flowtype): Resolve Flow warnings

Author: nodkz

.flowconfig

@@ -1,6 +1,4 @@
 [ignore]
-.*/__tests__/.*
-.*/__mocks__/.*
 .*/coverage/.*
 .*/resources/.*
 <PROJECT_ROOT>/lib/.*
@@ -36,7 +34,6 @@
 .*/node_modules/uglify.*
 .*/node_modules/yargs.*
 
-
 [include]
 
 [libs]

package.json

@@ -22,7 +22,7 @@
   },
   "homepage": "https://github.com/nodkz/graphql-compose-relay",
   "peerDependencies": {
-    "graphql-compose": ">=1.19.3"
+    "graphql-compose": ">=1.20.3"
   },
   "devDependencies": {
     "babel-cli": "^6.24.1",

src/__tests__/composeWithRelay-test.js

@@ -20,6 +20,7 @@ describe('composeWithRelay', () => {
     });
 
     it('should throw error if got a not TypeComposer', () => {
+      // $FlowFixMe
       expect(() => composeWithRelay(123)).toThrowError('should provide TypeComposer instance');
     });
 
@@ -41,19 +42,23 @@ describe('composeWithRelay', () => {
   describe('when pass RootQuery type composer', () => {
     it('should add `node` field to RootQuery', () => {
       const nodeField = rootQueryComposer.getField('node');
+      // $FlowFixMe
       expect(nodeField.type).toBeInstanceOf(GraphQLInterfaceType);
+      // $FlowFixMe
       expect(nodeField.type.name).toBe('Node');
     });
   });
 
   describe('when pass User type composer (not RootQuery)', () => {
     it('should add or override id field', () => {
       const idField = userComposer.getField('id');
+      // $FlowFixMe
       expect(idField.description).toContain('globally unique ID');
     });
 
     it('should make id field NonNull', () => {
       const idField = userComposer.getField('id');
+      // $FlowFixMe
       expect(idField.type).toBeInstanceOf(GraphQLNonNull);
     });
 
@@ -72,7 +77,9 @@ describe('composeWithRelay', () => {
         }
       }`;
       const result = await graphql.graphql(schema, query);
+      // $FlowFixMe
       expect(result.data.user.id).toBe(toGlobalId('User', 1));
+      // $FlowFixMe
       expect(result.data.user.name).toBe('Pavel');
     });
 
@@ -94,7 +101,9 @@ describe('composeWithRelay', () => {
         name
       }`;
       const result = await graphql.graphql(schema, query);
+      // $FlowFixMe
       expect(result.data.node.id).toBe(toGlobalId('User', 1));
+      // $FlowFixMe
       expect(result.data.node.name).toBe('Pavel');
     });
 
@@ -116,7 +125,9 @@ describe('composeWithRelay', () => {
         }
       }`;
       const result = await graphql.graphql(schema, query);
+      // $FlowFixMe
       expect(result.data.createUser.record.name).toBe('Ok');
+      // $FlowFixMe
       expect(result.data.createUser.clientMutationId).toBe('123');
     });
   });

src/__tests__/nodeFieldConfig-test.js

@@ -33,19 +33,36 @@ describe('nodeFieldConfig', () => {
   });
 
   it('should return null if args.id not defined', () => {
-    expect(config.resolve({}, {})).toBeNull();
+    const source = {};
+    const args = {};
+    const context = {};
+    const info = ({}: any);
+    expect(config.resolve(source, args, context, info)).toBeNull();
   });
 
   it('should return null if findById not defined for type', () => {
-    expect(config.resolve({}, { id: toGlobalId('UnexistedType', 1) })).toBeNull();
+    const source = {};
+    const args = { id: toGlobalId('UnexistedType', 1) };
+    const context = {};
+    const info = ({}: any);
+    expect(config.resolve(source, args, context, info)).toBeNull();
   });
 
   it('should return Promise if type exists, but id not exist', () => {
-    expect(config.resolve({}, { id: toGlobalId('User', 666) })).toBeInstanceOf(Promise);
+    const source = {};
+    const args = { id: toGlobalId('User', 666) };
+    const context = {};
+    const info = ({}: any);
+    expect(config.resolve(source, args, context, info)).toBeInstanceOf(Promise);
   });
 
   it('should return Promise with user data', async () => {
-    const res = await config.resolve({}, { id: toGlobalId('User', 1) });
+    const source = {};
+    const args = { id: toGlobalId('User', 1) };
+    const context = {};
+    const info = ({}: any);
+    const res = await config.resolve(source, args, context, info);
+    // $FlowFixMe
     expect(res.name).toBe('Pavel');
   });
 });

src/__tests__/wrapMutationResolver.js

@@ -36,6 +36,7 @@ describe('wrapMutationResolver', () => {
       );
 
       const type = getNamedType(fieldConfigManyArgsWithoutInput.args.input.type);
+      // $FlowFixMe
       const itc = new InputTypeComposer(type);
       expect(itc.hasField('sort')).toBe(true);
       expect(itc.hasField('limit')).toBe(true);
@@ -48,6 +49,7 @@ describe('wrapMutationResolver', () => {
       );
 
       const type = getNamedType(fieldConfigManyArgsWithInput.args.input.type);
+      // $FlowFixMe
       const itc = new InputTypeComposer(type);
       expect(itc.hasField('sort')).toBe(false);
       expect(itc.hasField('limit')).toBe(false);