New typescript linting fixes

Author: XavierGeerinck

jest.config.js

@@ -1,4 +1,5 @@
+/** @type {import('ts-jest/dist/types').InitialOptionsTsJest} */
 module.exports = {
   preset: 'ts-jest',
-  testEnvironment: 'node'
+  testEnvironment: 'node',
 };

package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "dapr-client",
-  "version": "0.3.1",
+  "version": "1.0.0",
   "description": "The official Dapr (https://dapr.io) SDK for Node.js",
   "types": "http/index.d.ts",
   "scripts": {
@@ -19,31 +19,31 @@
   "author": "Xavier Geerinck",
   "license": "ISC",
   "dependencies": {
-    "@grpc/grpc-js": "^1.3.2",
-    "@js-temporal/polyfill": "^0.1.0",
+    "@grpc/grpc-js": "^1.3.7",
+    "@js-temporal/polyfill": "^0.2.0",
     "body-parser": "^1.19.0",
-    "google-protobuf": "^3.15.8",
-    "grpc": "^1.24.10",
+    "google-protobuf": "^3.18.0",
     "node-fetch": "^2.6.1",
+    "grpc": "^1.24.10",
     "restana": "^4.9.1",
     "uuid": "^8.3.2"
   },
   "devDependencies": {
-    "@types/body-parser": "^1.19.0",
-    "@types/google-protobuf": "^3.15.2",
+    "@types/body-parser": "^1.19.1",
+    "@types/google-protobuf": "^3.15.5",
     "@types/jest": "^27.0.1",
-    "@types/node": "^15.0.3",
+    "@types/node": "^16.9.1",
     "@types/node-fetch": "^2.5.8",
     "@types/uuid": "^8.3.1",
-    "grpc-tools": "^1.11.1",
-    "grpc_tools_node_protoc_ts": "^5.2.2",
-    "jest": "^27.0.6",
-    "nodemon": "^2.0.7",
-    "prettier": "^2.2.1",
-    "ts-jest": "^27.0.4",
+    "grpc-tools": "^1.11.2",
+    "grpc_tools_node_protoc_ts": "^5.3.2",
+    "jest": "^27.2.0",
+    "nodemon": "^2.0.12",
+    "prettier": "^2.4.0",
+    "ts-jest": "^27.0.5",
     "tslint": "^6.1.3",
     "tslint-config-prettier": "^1.18.0",
-    "typescript": "^4.2.3"
+    "typescript": "^4.4.3"
   },
   "repository": {
     "type": "git",

package.json

@@ -169,7 +169,7 @@ export default class ActorStateManager<T> {
       return val;
     }
 
-    const changeKind = this.isStateMarkedForRemove(stateName) ? StateChangeKind.UPDATE : StateChangeKind.ADD;
+    const changeKind = await this.isStateMarkedForRemove(stateName) ? StateChangeKind.UPDATE : StateChangeKind.ADD;
     stateChangeTracker.set(stateName, new StateMetadata(value, changeKind));
 
     return value;

src/actors/runtime/ActorStateManager.ts

@@ -37,9 +37,11 @@ export default class HTTPServerBinding implements IServerBinding {
       } catch (e) {
         res.statusCode = HttpStatusCode.INTERNAL_SERVER_ERROR;
 
+        console.debug(e);
+
         return res.end(JSON.stringify({
           error: "COULD_NOT_PROCESS_CALLBACK",
-          error_msg: `Something happened while processing the input binding callback - ${e.message}`
+          error_msg: `Something happened while processing the input binding callback`
         }));
       }
     });

src/implementation/Server/HTTPServer/binding.ts

@@ -109,7 +109,8 @@ describe('ActorRuntime', () => {
     try {
       await runtime.invoke(DemoActorCounterImpl.name, actorId, "someRandomMethod");
     } catch (e) {
-      expect(e.message).toEqual(`{"error":"ACTOR_METHOD_DOES_NOT_EXIST","errorMsg":"The actor method 'someRandomMethod' does not exist on ${DemoActorCounterImpl.name}"}`);
+      let msg = (e as Error).message;
+      expect(msg).toEqual(`{"error":"ACTOR_METHOD_DOES_NOT_EXIST","errorMsg":"The actor method 'someRandomMethod' does not exist on ${DemoActorCounterImpl.name}"}`);
     }
   });
 

test/unit/actor/actorRuntime.test.ts

@@ -14,7 +14,8 @@ describe('DaprClient', () => {
     try {
       const client = new DaprClient(host, host);
     } catch (e) {
-      expect(e.message).toEqual("DAPR_CLIENT_INCORRECT_PORT");
+      let msg = (e as Error).message;
+      expect(msg).toEqual("DAPR_CLIENT_INCORRECT_PORT");
     }
   });
 });

test/unit/main/DaprClient.test.ts

@@ -14,15 +14,17 @@ describe('DaprServer', () => {
     try {
       const server = new DaprServer(host, host);
     } catch (e) {
-      expect(e.message).toEqual("DAPR_SERVER_INCORRECT_SERVER_PORT");
+      let msg = (e as Error).message;
+      expect(msg).toEqual("DAPR_SERVER_INCORRECT_SERVER_PORT");
     }
   });
 
   it('should throw an error on a wrong port for client', () => {
     try {
       const server = new DaprServer(host, port, host, host);
     } catch (e) {
-      expect(e.message).toEqual("DAPR_SERVER_INCORRECT_SIDECAR_PORT");
+      let msg = (e as Error).message;
+      expect(msg).toEqual("DAPR_SERVER_INCORRECT_SIDECAR_PORT");
     }
   });
 });