chore: upgrade contentful's eslint to 6.x

move dev dependencies to root folder

Author: dpinol

packages/botonic-plugin-contentful/.eslintrc.js

@@ -43,9 +43,9 @@ module.exports = {
     // special for TYPESCRIPT
     "@typescript-eslint/explicit-function-return-type": "off", // annoying for tests
     "@typescript-eslint/explicit-member-accessibility": "off", //we think defaulting to public is a good default
+    "@typescript-eslint/no-empty-function": "warn",
     "@typescript-eslint/no-namespace": ["error", { allowDeclarations: true }], // to encapsulate types in namespace with same name as Class
     "@typescript-eslint/no-non-null-assertion" : "warn", // specially useful in tests, and "when you know what you're doing"
-    "@typescript-eslint/no-object-literal-type-assertion" : [ "error", {allowAsParameter: false}], //useful to pass options to plugins
     "@typescript-eslint/no-parameter-properties": "off", // opinionated: parameter properties make data classes shorter
     // allow public functions/classes to call private functions/classes declared below.
     // otoh, variables (typically constants) should be declared at the top
@@ -63,9 +63,6 @@ module.exports = {
       "files": [
         "tests/**/*.ts" // to be able to skip required fields when not used in a particular test
       ],
-      "rules": {
-        "@typescript-eslint/no-object-literal-type-assertion" : "off",
-      }
     }
   ],
   env: {

packages/botonic-plugin-contentful/package-lock.json

@@ -2,7 +2,7 @@
   "scripts": {
     "build": "rm -rf lib && tsc",
     "build_with_tests": "tsc -b tests/tsconfig.json",
-    "test": "jest",
+    "test": "../../node_modules/.bin/jest",
     "lint": "npm run lint_core -- --fix",
     "lint_ci": "npm run lint_core -- -c .eslintrc_slow.js",
     "lint_core": "node_modules/.bin/eslint_d --cache --quiet 'src/**/*.ts*' 'tests/**/*.ts*'",
@@ -32,27 +32,11 @@
     "node-nlp": "^4.0.2"
   },
   "devDependencies": {
-    "@types/jest": "^25.1.0",
-    "@types/jest-each": "^24.3.0",
     "@types/memoizee": "^0.4.3",
     "@types/moment-timezone": "^0.5.12",
-    "@types/node": "^13.1.6",
-    "@typescript-eslint/eslint-plugin": "^1.13.0",
-    "@typescript-eslint/parser": "^1.13.0",
-    "eslint": "^5.16.0",
-    "eslint-config-prettier": "^5.1.0",
-    "eslint-plugin-filenames": "^1.3.2",
-    "eslint-plugin-jest": "^23.1.1",
-    "eslint-plugin-no-null": "^1.0.2",
-    "eslint-plugin-prettier": "^3.1.1",
-    "eslint_d": "^8.0.0",
+    "@types/node": "^13.7.2",
     "identity-obj-proxy": "^3.0.0",
-    "jest": "^25.1.0",
-    "jest-extended": "^0.11.4",
-    "prettier": "^1.19.1",
-    "ts-jest": "^25.0.0",
-    "ts-mockito": "^2.5.0",
-    "typescript": "^3.7.5"
+    "jest-extended": "^0.11.4"
   },
   "engines": {
     "node": ">=12.0.0"

packages/botonic-plugin-contentful/package.json

@@ -66,9 +66,7 @@ export default class BotonicPluginContentful {
     }
   }
 
-  // @ts-ignore
-  pre({ input, session, lastRoutePath }) {}
+  pre(r: { input: any; session: any; lastRoutePath: any }) {}
 
-  // @ts-ignore
-  post({ input, session, lastRoutePath, response }) {}
+  post(r: { input: any; session: any; lastRoutePath: any; response: any }) {}
 }

packages/botonic-plugin-contentful/src/plugin.ts

@@ -26,8 +26,7 @@ export interface BotonicText extends BotonicMsg {
 export class BotonicMsgConverter {
   constructor(readonly options = new RenderOptions()) {}
 
-  carousel(carousel: cms.Carousel, delayS: number = 0): BotonicMsgs {
-    // eslint-disable-next-line @typescript-eslint/no-object-literal-type-assertion
+  carousel(carousel: cms.Carousel, delayS = 0): BotonicMsgs {
     return {
       type: 'carousel',
       delay: delayS,
@@ -69,7 +68,7 @@ export class BotonicMsgConverter {
     })
   }
 
-  text(text: cms.Text, delayS: number = 0): BotonicMsgs {
+  text(text: cms.Text, delayS = 0): BotonicMsgs {
     const msg: any = {
       type: 'text',
       delay: delayS,

packages/botonic-plugin-contentful/src/render/botonic-converter.ts

@@ -36,6 +36,7 @@ export class SearchByKeywords {
     const results = kws.findCandidatesWithKeywordsAt(inputText)
     return results.map(res => {
       const candidate = res.candidate as SearchResult
+      // eslint-disable-next-line @typescript-eslint/ban-ts-ignore
       // @ts-ignore
       candidate.match = res.match
       return candidate

packages/botonic-plugin-contentful/src/search/search-by-keywords.ts

@@ -21,7 +21,7 @@ export class Schedule {
     this.zone = zone
   }
 
-  createHourAndMinute(hour: number, minute: number = 0): HourAndMinute {
+  createHourAndMinute(hour: number, minute = 0): HourAndMinute {
     return new HourAndMinute(this.zone, hour, minute)
   }
 

packages/botonic-plugin-contentful/src/time/schedule.ts

@@ -6,8 +6,8 @@ function europeDate(
   year: number,
   month: number,
   day: number,
-  hour: number = 0,
-  minute: number = 0
+  hour = 0,
+  minute = 0
 ): Date {
   const date = new Date(Date.UTC(year, month, day, hour, minute, 0))
 

packages/botonic-plugin-contentful/tests/contentful/date-range.test.ts

@@ -21,8 +21,8 @@ function europeDate(
   year: number,
   month: number,
   day: number,
-  hour: number = 0,
-  minute: number = 0
+  hour = 0,
+  minute = 0
 ): Date {
   const date = new Date(Date.UTC(year, month, day, hour, minute, 0))
 

packages/botonic-plugin-contentful/tests/time/schedule.test.ts

@@ -32,5 +32,11 @@
     "forceConsistentCasingInFileNames": true,
     "noImplicitReturns": true,
     "noUnusedLocals": true
-  }
+  },
+  // test options
+  "types": [
+    // @types/node must be a direct dependency. If only defined in a parent package.json, we get TSC errors for
+    // symbols from these libs: "ES2016.Array.Include", "ES2017.object"
+    "@types/jest"
+  ]
 }

packages/botonic-plugin-contentful/tsconfig.json

@@ -27,7 +27,6 @@ test('TEST: ErrorReportingCMS write rejected', async (done: DoneCallback) => {
       expect(trackException.reason).toBe(error)
       done()
     })
-    
 })
 
 test('TEST: ErrorReportingCMS write success', async () => {