Fixed issue where BOM wasn't being stripped from transcripts opened v…

…ia the File menu.
microsoft · Apr 12, 2019 · 12c9710 · 12c9710
1 parent 731978d
commit 12c9710
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 3 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -11,6 +11,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - [client] Fixed secret prompt dialog's opaque background so that it is now transparent in PR [1407](https://github.com/Microsoft/BotFramework-Emulator/pull/1407)
 - [build / client] Fixed ipc issue that was breaking the command service in PR [1418](https://github.com/Microsoft/BotFramework-Emulator/pull/1418)
 - [build] Bumped electron version to v4.1.1 and updated .dmg installer background image in PR [1419](https://github.com/Microsoft/BotFramework-Emulator/pull/1419)
+- [client] Fixed issue where BOM wasn't being stripped from transcripts opened via the File menu in PR [1425](https://github.com/Microsoft/BotFramework-Emulator/pull/1425)
 
 ## v4.3.3 - 2019 - 03 - 14
 ## Fixed

diff --git a/packages/app/main/src/commands/emulatorCommands.spec.ts b/packages/app/main/src/commands/emulatorCommands.spec.ts
@@ -71,7 +71,6 @@ jest.mock('chokidar', () => ({
 
 jest.mock('fs-extra', () => ({
   stat: async () => ({ isFile: () => true }),
-  readFile: async () => JSON.stringify((mockConversation as any).transcript),
 }));
 
 jest.mock('mkdirp', () => ({
@@ -94,6 +93,7 @@ jest.mock('../utils', () => ({
   writeFile: async () => true,
   loadSettings: () => ({ windowState: {} }),
   getThemes: async () => [],
+  readFileSync: () => JSON.stringify((mockConversation as any).transcript),
 }));
 
 jest.mock('../utils/ensureStoragePath', () => ({

diff --git a/packages/app/main/src/commands/emulatorCommands.ts b/packages/app/main/src/commands/emulatorCommands.ts
@@ -45,7 +45,7 @@ import { getActiveBot, getBotInfoByPath, patchBotsJson, toSavableBot } from '../
 import { emulator } from '../emulator';
 import { mainWindow } from '../main';
 import { dispatch, getStore as getSettingsStore } from '../settingsData/store';
-import { parseActivitiesFromChatFile, showSaveDialog, writeFile } from '../utils';
+import { parseActivitiesFromChatFile, showSaveDialog, writeFile, readFileSync } from '../utils';
 import { cleanupId as cleanupActivityChannelAccountId, CustomActivity } from '../utils/conversation';
 import { botProjectFileWatcher } from '../watchers';
 import { TelemetryService } from '../telemetry';
@@ -125,7 +125,7 @@ export function registerCommands(commandRegistry: CommandRegistryImpl) {
         throw new Error(`${Commands.FeedTranscriptFromDisk}: File ${filePath} not found.`);
       }
 
-      const activities = JSON.parse(await fs.readFile(transcriptPath, 'utf-8'));
+      const activities = JSON.parse(readFileSync(transcriptPath));
 
       await mainWindow.commandService.call(
         Commands.FeedTranscriptFromMemory,