Adding test to handle empty files and fixing error message (#269)

Author: munozemilio

packages/lu/src/parser/qnafile/parseQnAFile.js

@@ -7,7 +7,7 @@ module.exports = {
         try {
             QnAJSON = JSON.parse(QnAFileContent);
         } catch (err) {
-            throw (new exception(retCode.errorCode.INVALID_INPUT_FILE, 'Sorry, error parsing file as QnA JSON: ' + file));
+            throw (new exception(retCode.errorCode.INVALID_INPUT_FILE, 'Sorry, error parsing QnA JSON content'))
         }
         return QnAJSON;
     }

packages/lu/test/commands/luis/convert.test.ts

@@ -516,3 +516,19 @@ describe('luis:convert file creation', () => {
   })
 })
 
+describe('luis:convert empty file handling', () => {
+  test
+  .stderr()
+  .command(['luis:convert', '--in', `${path.join(__dirname, './../../fixtures/empty.lu')}`])
+  .it('luis:convert errors out on empty lu file', async (ctx) => {
+    expect(ctx.stderr).to.contain('[ERROR] Cannot parse empty')
+  })
+
+  test
+  .stderr()
+  .command(['luis:convert', '--in', `${path.join(__dirname, './../../fixtures/empty.json')}`])
+  .it('luis:convert errors out on empty json file', async (ctx) => {
+    expect(ctx.stderr).to.contain('Sorry, error parsing content as LUIS JSON')
+  })
+})
+

packages/lu/test/commands/qnamaker/convert.test.ts

@@ -117,3 +117,19 @@ describe('qnamaker:convert file creation', () => {
     expect(ctx.stderr).to.contain('Path not found:')
   })
 })
+
+describe('qnamaker:convert empty file handling', () => {
+  test
+  .stderr()
+  .command(['qnamaker:convert', '--in', `${path.join(__dirname, './../../fixtures/empty.lu')}`])
+  .it('qnamaker:convert errors out on empty lu file', async (ctx) => {
+    expect(ctx.stderr).to.contain('[ERROR] Cannot parse empty')
+  })
+
+  test
+  .stderr()
+  .command(['qnamaker:convert', '--in', `${path.join(__dirname, './../../fixtures/empty.json')}`])
+  .it('qnamaker:convert errors out on empty json file', async (ctx) => {
+    expect(ctx.stderr).to.contain('Sorry, error parsing QnA JSON content')
+  })
+})