[Test] Fix async test (#1707)

This commit fixes async tests and adds a related exception throwing.

ONE-vscode-DCO-1.0-Signed-off-by: Dayoung Lee <[email protected]>

src/Tests/MPQEditor/MPQEditor.test.ts

@@ -151,7 +151,7 @@ suite("MPQEditor", function () {
 
         await MPQEditorProvider.updateDocumentBy(document, newJson);
 
-        document.save();
+        await document.save();
         const newJsonText: string = document.getText();
         const newCont = JSON.parse(newJsonText);
         assert.strictEqual(newCont["default_quantization_dtype"], "int16");

src/Tests/Utils/Hash.test.ts

@@ -46,7 +46,6 @@ suite("Utils", function () {
         const notExistingPath = testBuilder.getPath("non-existing");
         generateHash(vscode.Uri.file(notExistingPath)).catch(() => {
           pass();
-          assert.ok(true);
         });
       });
     });

src/Utils/Hash.ts

@@ -15,10 +15,14 @@
  */
 
 import * as crypto from "crypto";
+import fs from "fs";
 import vscode from "vscode";
 
-export async function generateHash(uri: vscode.Uri) {
-  // TODO: Error handling
+export async function generateHash(uri: vscode.Uri): Promise<string> {
+  if (!fs.existsSync(uri.fsPath)) {
+    throw new Error(`File does not exists at ${uri.fsPath}`);
+  }
+
   return crypto
     .createHash("sha256")
     .update(Buffer.from(await vscode.workspace.fs.readFile(uri)).toString())