teraslice-client-js 1.3.0 → 1.3.2

package/package.json

@@ -1,7 +1,7 @@
 {
     "name": "teraslice-client-js",
     "displayName": "Teraslice Client (JavaScript)",
-    "version": "1.3.0",
+    "version": "1.3.2",
     "description": "A Node.js client for teraslice jobs, assets, and cluster references.",
     "keywords": [
         "elasticsearch",
@@ -33,7 +33,7 @@
     },
     "dependencies": {
         "@terascope/types": "^1.2.0",
-        "@terascope/utils": "^1.3.0",
+        "@terascope/utils": "^1.3.2",
         "auto-bind": "^5.0.1",
         "got": "^13.0.0"
     },

package/src/ex.ts

@@ -165,7 +165,7 @@ export default class Ex extends Client {
             // These are terminal states for a job so if we're not explicitly
             // watching for these then we need to stop waiting as the job
             // status won't change further.
-            if (terminal[result]) {
+            if (result in terminal) {
                 throw new TSError(
                     `Execution cannot reach the target status, "${target}", because it is in the terminal state, "${result}"`,
                     { context: { lastStatus: result } }

package/src/job.ts

@@ -143,7 +143,7 @@ export default class Job extends Client {
         const checkStatus = async (): Promise<Teraslice.ExecutionStatus> => {
             let result;
             try {
-                const ex = await this.get(`/jobs/${this._jobId}/ex`, options);
+                const ex = await this.get<Teraslice.ExecutionConfig>(`/jobs/${this._jobId}/ex`, options);
                 if (exId && ex.ex_id !== exId) {
                     console.warn(`[WARNING] the execution ${ex.ex_id} has changed from ${exId}`);
                 }
@@ -164,7 +164,7 @@ export default class Job extends Client {
             // These are terminal states for a job so if we're not explicitly
             // watching for these then we need to stop waiting as the job
             // status won't change further.
-            if (terminal[result]) {
+            if (result in terminal) {
                 throw new TSError(
                     `Job cannot reach the target status, "${target}", because it is in the terminal state, "${result}"`,
                     { context: { lastStatus: result } }

package/test/ex-spec.ts

@@ -142,12 +142,12 @@ describe('Teraslice Ex', () => {
         });
     });
 
-    const methodTestCases: string[] = [
+    const methodTestCases = [
         'stop',
         'pause',
         'resume'
-    ];
-    describe.each(methodTestCases)('->%s', (method: string) => {
+    ] as const;
+    describe.each(methodTestCases)('->%s', (method) => {
         describe('when called with nothing', () => {
             beforeEach(() => {
                 scope.post(`/ex/some-ex-id/_${method}`)

package/test/job-spec.ts

@@ -174,13 +174,13 @@ describe('Teraslice Job', () => {
         });
     });
 
-    const methodTestCases: string[] = [
+    const methodTestCases = [
         'start',
         'stop',
         'pause',
         'resume'
-    ];
-    describe.each(methodTestCases)('->%s', (method: string) => {
+    ] as const;
+    describe.each(methodTestCases)('->%s', (method) => {
         describe('when called with nothing', () => {
             beforeEach(() => {
                 scope.post(`/jobs/some-job-id/_${method}`)

package/tsconfig.json

@@ -2,7 +2,8 @@
     "extends": "../../tsconfig",
     "compilerOptions": {
         "outDir": "dist",
-        "rootDir": "."
+        "rootDir": ".",
+        "suppressImplicitAnyIndexErrors": false
     },
     "include": [
         "src",