fix: fixed lint issues

worldscapes · Mar 14, 2022 · dac66ec · dac66ec
1 parent 9a5cf91
commit dac66ec
Show file tree

Hide file tree

Showing 4 changed files with 20 additions and 19 deletions.
diff --git a/.eslintrc b/.eslintrc
@@ -13,6 +13,7 @@
   "rules": {
     "@typescript-eslint/no-explicit-any": "error",
 
+    "@typescript-eslint/ban-types": "warn",
     "@typescript-eslint/no-empty-function": "warn",
     "@typescript-eslint/explicit-module-boundary-types": "warn",
 

diff --git a/src/ecr/ecr/ecr.api.tools.ts b/src/ecr/ecr/ecr.api.tools.ts
@@ -87,7 +87,7 @@ export namespace ECRApiTools {
                         },
                         commands: [],
                     });
-                })
+                });
 
             });
 
@@ -234,7 +234,7 @@ export namespace ECRApiTools {
 
                 test("Should call custom command handlers", () => {
                    const handlerEffect = jest.fn((command, store) => {});
-                   const handler = createCommandHandler(CreateEntityWithComponentsCommand, handlerEffect)
+                   const handler = createCommandHandler(CreateEntityWithComponentsCommand, handlerEffect);
                    ecr.addCustomCommandHandler(handler);
                    const command = new CreateEntityWithComponentsCommand([]);
                    ecr.injectCommands([ command ]);
@@ -254,7 +254,7 @@ export namespace ECRApiTools {
                         removePrototype(new TestComponent(2)),
                         removePrototype(new TestComponent(3)),
                         removePrototype(new TestComponent(4)),
-                    ]
+                    ];
                     ecr.injectCommands([ new CreateEntityWithComponentsCommand(components) ]);
                     const snapshot = ecr.runSimulationTick().snapshot;
                     expect(snapshot.entities).toHaveLength(1);
@@ -320,10 +320,10 @@ export namespace ECRApiTools {
                             'testResource1': removePrototype(new TestResource(1)),
                             'testResource2': removePrototype(new TestResource2(2)),
                         }
-                    }
+                    };
 
                     ecr.injectCommands([ new LoadSnapshotCommand(testSnapshot) ]);
-                })
+                });
 
                 test("Should check custom rule condition", () => {
                     const condition = jest.fn(() => true);
@@ -332,10 +332,10 @@ export namespace ECRApiTools {
                         query: emptyQuery,
                         condition,
                         body
-                    })
+                    });
                     ecr.runSimulationTick();
                     expect(condition).toBeCalledTimes(1);
-                })
+                });
 
                 test("Should call custom rule body when condition returns true", () => {
                     const condition = jest.fn(() => true);
@@ -344,7 +344,7 @@ export namespace ECRApiTools {
                         query: emptyQuery,
                         condition,
                         body
-                    })
+                    });
                     ecr.runSimulationTick();
                     expect(body).toBeCalledTimes(1);
                 });
@@ -355,7 +355,7 @@ export namespace ECRApiTools {
                         condition: jest.fn(() => false),
                         body: jest.fn(() => {})
                     });
-                    ecr.addRule(rule)
+                    ecr.addRule(rule);
                     ecr.runSimulationTick();
                     expect(rule.body).not.toBeCalled();
                 });
@@ -445,7 +445,7 @@ export namespace ECRApiTools {
 
             });
 
-        })
+        });
 
     }
 }
diff --git a/src/engine/client/simulation/implementations/simple.client-simulation.ts b/src/engine/client/simulation/implementations/simple.client-simulation.ts
@@ -13,6 +13,6 @@ export class SimpleClientSimulation extends ClientSimulationApi {
   }
 
   public applyServerUpdate(snapshot: WorldStateSnapshot): void {
-    this.ecr.injectCommands([ new LoadSnapshotCommand(snapshot) ])
+    this.ecr.injectCommands([ new LoadSnapshotCommand(snapshot) ]);
   }
 }
diff --git a/src/typing/WSCStructure.test.ts b/src/typing/WSCStructure.test.ts
@@ -24,7 +24,7 @@ describe("Typing tools", () => {
             expect(getTypeName(SomeClass)).toBe("SomeClass");
         });
 
-    })
+    });
 
 
     describe("getObjectType", () => {
@@ -34,33 +34,33 @@ describe("Typing tools", () => {
             expect(getObjectType(obj)).toBe("SomeClass");
         });
 
-    })
+    });
 
     describe("isTypeOf", () => {
 
         test("Should return true when the given type is in object's typeChain", () => {
             const obj = removePrototype(new SomeClass2());
             expect(isTypeOf(obj, SomeClass)).toBeTruthy();
-        })
+        });
 
         test("Should return false when the given type is not in object's typeChain", () => {
             const obj = removePrototype(new SomeClass3());
             expect(isTypeOf(obj, SomeClass2)).toBeFalsy();
-        })
+        });
 
-    })
+    });
 
     describe("isSameType", () => {
         test("Should return true when objects are of same type", () => {
             const obj = removePrototype(new SomeClass2());
             const obj2 = removePrototype(new SomeClass2());
             expect(isSameType(obj, obj2)).toBeTruthy();
-        })
+        });
 
         test("Should return false when objects are of different type", () => {
             const obj = removePrototype(new SomeClass());
             const obj2 = removePrototype(new SomeClass2());
             expect(isSameType(obj, obj2)).toBeFalsy();
-        })
-    })
+        });
+    });
 });