chore(typescript): fix ts errors in tests

Author: Michael Wittwer

jest.config.js

@@ -3,7 +3,7 @@ module.exports = {
   preset: 'ts-jest',
   testEnvironment: 'node',
   transform: {
-    '^.+\\.[tj]sx?$': [
+     '^.+\\.tsx?$': [
       'ts-jest',
       {
         tsconfig: './tsconfig.jest.json'

src/decorator/metadata/property-metadata.spec.ts

@@ -15,6 +15,7 @@ class NestedModel {
   createdAt: Date
 }
 
+// tslint:disable-next-line:max-classes-per-file
 @Model()
 class TestModel {
   @CollectionProperty()

src/dynamo/batchget/batch-get.request.spec.ts

@@ -12,7 +12,7 @@ describe('batch get', () => {
 
   describe('constructor', () => {
     it('use provided DynamoDB instance', () => {
-      const dynamoDB = new DynamoDB()
+      const dynamoDB = new DynamoDB.default()
       const batchGetRequest = new BatchGetRequest(dynamoDB)
       expect(batchGetRequest.dynamoDB).toBe(dynamoDB)
 
@@ -87,8 +87,10 @@ describe('batch get', () => {
     })
 
     it('should throw when more than 100 items are added', () => {
-      const items55: Array<Partial<SimpleWithPartitionKeyModel>> = [...new Array(55)].map((x, i) => ({ id: `id-${i}` }))
-      const items60: Array<Partial<Organization>> = [...new Array(60)].map((x, i) => ({
+      const items55: Array<Partial<SimpleWithPartitionKeyModel>> = [...new Array(55)].map((_x, i) => ({
+        id: `id-${i}`,
+      }))
+      const items60: Array<Partial<Organization>> = [...new Array(60)].map((_x, i) => ({
         id: `id-${i}`,
         createdAtDate: new Date(),
       }))

src/dynamo/batchwrite/batch-write.request.spec.ts

@@ -14,7 +14,7 @@ describe('batchWriteRequest', () => {
     })
 
     describe('use provided DynamoDB instance', () => {
-      const dynamoDB = new DynamoDB()
+      const dynamoDB = new DynamoDB.default()
       const batchWriteRequest = new BatchWriteRequest(dynamoDB)
       expect(batchWriteRequest.dynamoDB).toBe(dynamoDB)
 

src/dynamo/dynamo-db-wrapper.spec.ts

@@ -142,7 +142,7 @@ describe('dynamo rx', () => {
   })
 
   it('should use given dynamoDB client', () => {
-    const dynamoDB = new DynamoDB()
+    const dynamoDB = new DynamoDB.default()
     const dynamoDBWrapper = new DynamoDbWrapper(dynamoDB)
     expect(dynamoDBWrapper.dynamoDB).toBe(dynamoDB)
 

src/dynamo/dynamo-db-wrapper.ts

@@ -14,8 +14,7 @@ export class DynamoDbWrapper {
 
   constructor(dynamoDB?: DynamoDB) {
     // create the actual dynamoDB client
-    // @ts-ignore
-    this.dynamoDB = dynamoDB || new DynamoDB()
+    this.dynamoDB = dynamoDB || new DynamoDB.default()
   }
 
   /*

src/dynamo/dynamo-store.spec.ts

@@ -98,7 +98,7 @@ describe('dynamo store', () => {
   })
 
   describe('use provided dynamoDB instance', () => {
-    const dynamoDB = new DynamoDB()
+    const dynamoDB = new DynamoDB.default()
     const store = new DynamoStore(SimpleWithPartitionKeyModel, dynamoDB)
     expect(store.dynamoDB).toBe(dynamoDB)
 

src/dynamo/expression/request-expression-builder.spec.ts

@@ -12,7 +12,7 @@ import {
 } from './request-expression-builder'
 
 const DYNAMO_RX_MOCK: DynamoDbWrapper = <DynamoDbWrapper>{
-  query(params: DynamoDB.QueryInput): Promise<DynamoDB.QueryOutput> {
+  query(_params: DynamoDB.QueryInput): Promise<DynamoDB.QueryOutput> {
     return Promise.resolve({})
   },
 }

src/dynamo/request/read-many.request.spec.ts

@@ -22,7 +22,7 @@ class TestRequest<T> extends ReadManyRequest<T, T, any, any, any, any, any> {
 
   protected readonly logger: Logger
 
-  protected doRequest(params: any): Promise<any> {
+  protected doRequest(_params: any): Promise<any> {
     return Promise.resolve({})
   }
 }

src/dynamo/request/write.request.spec.ts

@@ -16,7 +16,7 @@ describe('write request', () => {
       this.logger = createLogger('dynamo.request.PutRequest', modelClazz)
     }
 
-    protected doRequest(params: any): Promise<any> {
+    protected doRequest(_params: any): Promise<any> {
       return Promise.resolve(null)
     }
   }