From 920e7ab8e21270310ba0a90aa0299d46cd7bdb65 Mon Sep 17 00:00:00 2001 From: nodkz Date: Thu, 29 Apr 2021 11:31:45 +0600 Subject: [PATCH] test: remove redundant `any` coercion --- src/__tests__/ObjectTypeComposer-test.ts | 16 ++++++++-------- src/__tests__/Resolver-test.ts | 14 +++++++------- 2 files changed, 15 insertions(+), 15 deletions(-) diff --git a/src/__tests__/ObjectTypeComposer-test.ts b/src/__tests__/ObjectTypeComposer-test.ts index 76f2414..b539b73 100644 --- a/src/__tests__/ObjectTypeComposer-test.ts +++ b/src/__tests__/ObjectTypeComposer-test.ts @@ -1109,13 +1109,13 @@ describe('ObjectTypeComposer', () => { name: 'findById', resolve: () => '123', }); - expect(await tc.getResolver('findById').resolve({} as any)).toBe('123'); + expect(await tc.getResolver('findById').resolve({})).toBe('123'); tc.wrapResolverResolve('findById', (next) => async (rp) => { const prev = await next(rp); return `${prev}456`; }); - expect(await tc.getResolver('findById').resolve({} as any)).toBe('123456'); + expect(await tc.getResolver('findById').resolve({})).toBe('123456'); }); it('wrapResolver() should wrap resolver via callback', async () => { @@ -1123,14 +1123,14 @@ describe('ObjectTypeComposer', () => { name: 'update', resolve: () => '123', }); - expect(await tc.getResolver('update').resolve({} as any)).toBe('123'); + expect(await tc.getResolver('update').resolve({})).toBe('123'); const prevResolver = tc.getResolver('update'); tc.wrapResolver('update', (resolver) => { resolver.resolve = () => '456'; // eslint-disable-line return resolver; }); - expect(await tc.getResolver('update').resolve({} as any)).toBe('456'); + expect(await tc.getResolver('update').resolve({})).toBe('456'); expect(tc.getResolver('update')).not.toBe(prevResolver); expect((prevResolver as any).resolve(undefined)).toBe('123'); }); @@ -1144,15 +1144,15 @@ describe('ObjectTypeComposer', () => { }, resolve: () => '123', }); - expect(await tc.getResolver('update').resolve({} as any)).toBe('123'); + expect(await tc.getResolver('update').resolve({})).toBe('123'); tc.wrapResolverAs('updateExt', 'update', (resolver) => { resolver.resolve = () => '456'; // eslint-disable-line resolver.addArgs({ c: 'Boolean' }); return resolver; }); - expect(await tc.getResolver('updateExt').resolve({} as any)).toBe('456'); - expect(await tc.getResolver('update').resolve({} as any)).toBe('123'); + expect(await tc.getResolver('updateExt').resolve({})).toBe('456'); + expect(await tc.getResolver('update').resolve({})).toBe('123'); expect(tc.getResolver('update').getArgNames()).toEqual(['a', 'b']); expect(tc.getResolver('updateExt').getArgNames()).toEqual(['a', 'b', 'c']); @@ -1180,7 +1180,7 @@ describe('ObjectTypeComposer', () => { return '123'; }, }); - expect(await tc.getResolver('update', [mw1, mw2]).resolve({} as any)).toBe('123'); + expect(await tc.getResolver('update', [mw1, mw2]).resolve({})).toBe('123'); expect(log).toEqual(['m1.before', 'm2.before', 'call update', 'm2.after', 'm1.after']); }); }); diff --git a/src/__tests__/Resolver-test.ts b/src/__tests__/Resolver-test.ts index ac43530..d0d5e82 100644 --- a/src/__tests__/Resolver-test.ts +++ b/src/__tests__/Resolver-test.ts @@ -693,7 +693,7 @@ describe('Resolver', () => { await newResolver.resolve({ args: { filter: { age: 15, isActive: false } }, someKey: 16, - } as any); + }); expect(rpSnap.rawQuery).toEqual({ age: { $gt: 15 }, @@ -830,7 +830,7 @@ describe('Resolver', () => { sortTypeNameFallback: 'SortEnum', }); - newResolver.resolve({ args: { sort: 'PRICE_ASC' }, query } as any); + newResolver.resolve({ args: { sort: 'PRICE_ASC' }, query }); expect(rpSnap.args.sort).toEqual({ price: 1 }); expect(whereSnap).toEqual({ price: { $gt: 0 } }); }); @@ -934,8 +934,8 @@ describe('Resolver', () => { source: { id: 1 }, args: { limit: 1 }, context: { isAdmin: true, db: {} }, - info: { fieldName: 'a', otherAstFields: {} }, - } as any); + info: { fieldName: 'a', otherAstFields: {} } as any, + }); expect(console.log.mock.calls[0]).toEqual(['ResolverResolveParams for User.find():']); expect(console.dir.mock.calls[0]).toEqual([ @@ -963,7 +963,7 @@ describe('Resolver', () => { r1.debugParams('args, args.sort, source.name').resolve({ source: { id: 1, name: 'Pavel' }, args: { limit: 1, sort: 'id' }, - } as any); + }); expect(console.log.mock.calls[0]).toEqual(['ResolverResolveParams for User.find():']); expect(console.dir.mock.calls[0]).toEqual([ @@ -1052,7 +1052,7 @@ describe('Resolver', () => { .resolve({ source: { id: 1, name: 'Pavel' }, args: { limit: 1, sort: 'id' }, - } as any); + }); expect(console.time.mock.calls[0]).toEqual(['Execution time for User.find()']); expect(console.timeEnd.mock.calls[0]).toEqual(['Execution time for User.find()']); @@ -1210,7 +1210,7 @@ describe('Resolver', () => { return res; }; - const res = await r.withMiddlewares([mw1, mw2]).resolve({} as any); + const res = await r.withMiddlewares([mw1, mw2]).resolve({}); expect(res).toBe('users result'); expect(log).toEqual(['m1.before', 'm2.before', 'call User.find()', 'm2.after', 'm1.after']); });