using node.js API breaks when running in browser; Fixes #68 (#77)

microsoft · Sep 13, 2022 · cc3141e · cc3141e
1 parent 5524e7a
commit cc3141e
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 1 deletion.
diff --git a/src/activateMockDebug.ts b/src/activateMockDebug.ts
@@ -183,6 +183,7 @@ class MockConfigurationProvider implements vscode.DebugConfigurationProvider {
 }
 
 export const workspaceFileAccessor: FileAccessor = {
+	isWindows: false,
 	async readFile(path: string): Promise<Uint8Array> {
 		let uri: vscode.Uri;
 		try {

diff --git a/src/debugAdapter.ts b/src/debugAdapter.ts
@@ -17,6 +17,7 @@ import { FileAccessor } from './mockRuntime';
  * So we can only use node.js API for accessing files.
  */
 const fsAccessor:  FileAccessor = {
+	isWindows: process.platform === 'win32',
 	readFile(path: string): Promise<Uint8Array> {
 		return fs.readFile(path);
 	},

diff --git a/src/mockRuntime.ts b/src/mockRuntime.ts
@@ -5,6 +5,7 @@
 import { EventEmitter } from 'events';
 
 export interface FileAccessor {
+	isWindows: boolean;
 	readFile(path: string): Promise<Uint8Array>;
 	writeFile(path: string, contents: Uint8Array): Promise<void>;
 }
@@ -674,7 +675,7 @@ export class MockRuntime extends EventEmitter {
 	}
 
 	private normalizePathAndCasing(path: string) {
-		if (process.platform === 'win32') {
+		if (this.fileAccessor.isWindows) {
 			return path.replace(/\//g, '\\').toLowerCase();
 		} else {
 			return path.replace(/\\/g, '/');