Skip to content

Commit 82bbfed

Browse files
committed
refactor Resolvers: add typeComposer as first required argument for Resolver.constructor.
Needs for Resolver Middlewares, that should have access to typeComposer settings. Eg. graphql-compose-rela's middlewares should have access to typeName.
1 parent 08014f4 commit 82bbfed

File tree

14 files changed

+14
-14
lines changed

14 files changed

+14
-14
lines changed

src/modelConverter.js

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,6 @@ import type {
99
MongooseModelT,
1010
typeConverterOpts,
1111
typeConverterResolversOpts,
12-
GraphQLObjectType,
1312
typeConverterInputTypeOpts,
1413
} from './definition';
1514

src/resolvers/__tests__/findByIds-test.js

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -72,7 +72,8 @@ describe('findByIds() ->', () => {
7272
});
7373

7474
it('should return empty array if args._ids is not valid objectIds', async () => {
75-
const result = await findByIds(UserModel, UserTypeComposer).resolve({ args: { _ids: ['d', 'e'] } });
75+
const result = await findByIds(UserModel, UserTypeComposer)
76+
.resolve({ args: { _ids: ['d', 'e'] } });
7677
expect(result).to.be.instanceOf(Array);
7778
expect(result).to.be.empty;
7879
});

src/resolvers/count.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ export default function count(
2727
throw new Error('Second arg for Resolver count() should be instance of TypeComposer.');
2828
}
2929

30-
return new Resolver({
30+
return new Resolver(typeComposer, {
3131
outputType: GraphQLInt,
3232
name: 'count',
3333
kind: 'query',

src/resolvers/createOne.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ export default function createOne(
2727
throw new Error('Second arg for Resolver createOne() should be instance of TypeComposer.');
2828
}
2929

30-
const resolver = new Resolver({
30+
const resolver = new Resolver(typeComposer, {
3131
name: 'createOne',
3232
kind: 'mutation',
3333
description: 'Create one document with mongoose defaults, setters, hooks and validation',

src/resolvers/findById.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ export default function findById(
2727
throw new Error('Second arg for Resolver findById() should be instance of TypeComposer.');
2828
}
2929

30-
return new Resolver({
30+
return new Resolver(typeComposer, {
3131
outputType: typeComposer.getType(),
3232
name: 'findById',
3333
kind: 'query',

src/resolvers/findByIds.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,7 @@ export default function findByIds(
3030
throw new Error('Second arg for Resolver findByIds() should be instance of TypeComposer.');
3131
}
3232

33-
return new Resolver({
33+
return new Resolver(typeComposer, {
3434
outputType: new GraphQLList(typeComposer.getType()),
3535
name: 'findByIds',
3636
kind: 'query',

src/resolvers/findMany.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@ export default function findMany(
3131
throw new Error('Second arg for Resolver findMany() should be instance of TypeComposer.');
3232
}
3333

34-
return new Resolver({
34+
return new Resolver(typeComposer, {
3535
outputType: new GraphQLList(typeComposer.getType()),
3636
name: 'findMany',
3737
kind: 'query',

src/resolvers/findOne.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,7 @@ export default function findOne(
3030
throw new Error('Second arg for Resolver findOne() should be instance of TypeComposer.');
3131
}
3232

33-
return new Resolver({
33+
return new Resolver(typeComposer, {
3434
outputType: typeComposer.getType(),
3535
name: 'findOne',
3636
kind: 'query',

src/resolvers/removeById.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@ export default function removeById(
3232
);
3333
}
3434

35-
const resolver = new Resolver({
35+
const resolver = new Resolver(typeComposer, {
3636
name: 'removeById',
3737
kind: 'mutation',
3838
description: 'Remove one document: '

src/resolvers/removeMany.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ export default function removeMany(
2828
);
2929
}
3030

31-
const resolver = new Resolver({
31+
const resolver = new Resolver(typeComposer, {
3232
name: 'removeMany',
3333
kind: 'mutation',
3434
description: 'Remove many documents without returning them: '

0 commit comments

Comments
 (0)