fix: solve the functional test failure caused by app-config

cr https://code.amazon.com/reviews/CR-20222194
alexa · Feb 24, 2020 · 5d3053e · 5d3053e
1 parent e4e72e2
commit 5d3053e
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/lib/model/abstract-config-file.js b/lib/model/abstract-config-file.js
@@ -13,7 +13,7 @@ const FILE_TYPE_YAML = 'YAML';
 const READ_METHOD_BY_FILE_TYPE = {
     JSON: (filePath) => {
         try {
-            return JSON.parse(fs.readFileSync(filePath));
+            return JSON.parse(fs.readFileSync(filePath, 'utf-8'));
         } catch (error) {
             throw new Error(`Failed to parse JSON file ${filePath}.\n${error}`);
         }

diff --git a/test/functional/commands/api/index.js b/test/functional/commands/api/index.js
@@ -181,6 +181,8 @@ class ApiCommandBasicTest {
         fs.existsSync.callThrough();
         sinon.stub(fs, 'readFileSync').withArgs(askCliConfig, 'utf-8').returns(JSON.stringify(this._TEST_DATA.ASK_CONFIG));
         fs.readFileSync.callThrough();
+        sinon.stub(fs, 'accessSync').withArgs(askCliConfig, fs.constants.R_OK | fs.constants.W_OK).returns(true);
+        fs.accessSync.callThrough();
 
         // Mock http server
         // According to input httpClienConfig,