Various qa fixes (#427)

Author: JSpru

packages/luis/src/commands/luis/init.ts …s/config/src/commands/config/set/luis.tspackages/luis/src/commands/luis/init.ts renamed to packages/config/src/commands/config/set/luis.ts packages/luis/src/commands/luis/init.ts renamed to packages/config/src/commands/config/set/luis.ts

@@ -3,12 +3,11 @@
  * Licensed under the MIT License.
  */
 
-import {CLIError, Command, flags} from '@microsoft/bf-cli-command'
-import { throws } from 'assert'
+import {Command, flags} from '@microsoft/bf-cli-command'
 const {cli} = require('cli-ux')
-const utils = require('../../utils/index')
+import {getConfigFile, writeConfigFile, Config} from '../../../utils/configfilehandler'
 
-export default class LuisInit extends Command {
+export default class ConfigSetLuis extends Command {
   static description = 'Stores default LUIS application values in global config.'
 
   static examples = [`
@@ -24,7 +23,7 @@ export default class LuisInit extends Command {
   }
 
   async run() {
-    const {flags} = this.parse(LuisInit)
+    const {flags} = this.parse(ConfigSetLuis)
     const configDir = this.config.configDir
 
     if (Object.entries(flags).length === 0 && flags.constructor === Object) {
@@ -37,24 +36,20 @@ export default class LuisInit extends Command {
       return this.log('Unable to save config settings')
     } catch (err) {
       this.log(`Unable to save config settings: ${err}`)
+      this._help()
     }
   }
 
   async promptSaveConfig(flags: any, configPath: string) {
     const configPrefix = 'luis__'
-    let userConfig = await utils.getUserConfig(configPath)
-    if (userConfig === null) {
-      await utils.createConfigFile(configPath)
-      userConfig = {}
-    }
+    let userConfig: Config = await getConfigFile(configPath)
     const saveConfigOptIn = await cli.confirm('Would you like to save the provided values to your global config file? (Y/N)')
     if (saveConfigOptIn) {
-      // save config
       const flagLabels = Object.keys(flags)
       flagLabels.map(label => {
         userConfig[`${configPrefix}${label}`] = flags[label]
       })
-      await utils.writeUserConfig(userConfig, configPath)
+      await writeConfigFile(configPath, userConfig)
       return true
     }
     return false

packages/config/src/commands/config/set/qnamaker.ts

@@ -29,7 +29,6 @@ export default class ConfigSetQnamaker extends Command {
   async run() {
     const {flags} = this.parse(ConfigSetQnamaker)
     let userConfig: Config = await getConfigFile(this.config.configDir)
-
     if (flags.subscriptionKey) {
       this.setValue('subscriptionKey', flags.subscriptionKey, userConfig)
     }

packages/config/src/commands/config/show/luis.ts

@@ -0,0 +1,26 @@
+/*!
+ * Copyright (c) Microsoft Corporation. All rights reserved.
+ * Licensed under the MIT License.
+ */
+
+import {Command, flags} from '@microsoft/bf-cli-command'
+import {getConfigFile, Config} from '../../../utils/configfilehandler'
+
+export default class ConfigShowLuis extends Command {
+  static description = 'Display LUIS settings'
+
+  static flags: any = {
+    help: flags.help({char: 'h', description: 'config:show:luis help'})
+  }
+
+  async run() {
+    const userConfig: Config = await getConfigFile(this.config.configDir)
+    let luis: any = {}
+    Object.keys(userConfig).forEach((key: string) => {
+      if (key.startsWith('luis__')) {
+        luis[key] = userConfig[key]
+      }
+    })
+    this.log(JSON.stringify(luis, null, 2))
+  }
+}

packages/config/test/commands/config/set/luis.test.ts

@@ -0,0 +1,38 @@
+import {expect, test} from '@oclif/test'
+import ConfigSetLuis from '../../../../src/commands/config/set/luis'
+const sinon = require('sinon')
+    
+describe('config:set:luis', () => {
+    
+    beforeEach(() => {
+        sinon.stub(ConfigSetLuis.prototype, 'promptSaveConfig').returns(true)
+    })
+    
+    afterEach(() => {
+        sinon.restore();
+    });
+    
+    test
+    .stdout()
+    .command(['config:set:luis', '--help'])
+    .it('should print the help contents when --help is passed as an argument', ctx => {
+        expect(ctx.stdout).to.contain('Stores default LUIS application values in global config.')
+    })
+    
+    test
+    .stdout()
+    .stderr()
+    .command(['config:set:luis'])
+    .it('displays an message indication nothing saved if no config values passed', ctx => {
+        expect(ctx.stdout).to.contain('No config settings specified')
+    })
+    
+    test
+    .stdout()
+    .stderr()
+    .command(['config:set:luis', '--appId', '9999'])
+    .it('displays an message indication values saved successfully', ctx => {
+        expect(ctx.stdout).to.contain('Config settings saved')
+    })
+    
+})

packages/config/test/commands/config/show/luis.test.ts

@@ -0,0 +1,19 @@
+import {expect, test} from '@oclif/test'
+import {initTestConfigFile, deleteTestConfigFile, getConfigFile} from './../../../configfilehelper'
+
+describe('config:show:luis', () => {
+    before(async function() {
+      await initTestConfigFile()
+    });
+  
+    after(async function() {
+      await deleteTestConfigFile()
+    });
+  
+    test
+      .stdout()
+      .command(['config:show:luis'])
+      .it('Displays config file luis data', ctx => {
+        expect(ctx.stdout).to.contain('"luis__appId": "xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx"')
+    })
+})

packages/config/test/configfilehelper.ts

@@ -27,7 +27,8 @@ export async function initTestConfigFile() {
     qnamaker__subscriptionKey: "222222cccccctttttth223kk3k33",
     qnamaker__hostname: "https://somehost.net",
     qnamaker__endpointKey: "xxxxxxxxxxxxxxxxxxx",
-    qnamaker__kbId: "xxxxxxxxxxxxxxxxxxxxxxx"
+    qnamaker__kbId: "xxxxxxxxxxxxxxxxxxxxxxx",
+    luis__appId: "xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx"
   }
 
   await fs.mkdirp(pathToConfigJson)

packages/lu/src/commands/luis/index.ts

@@ -24,15 +24,15 @@ export default class LuisApplicationCreate extends Command {
     description: flags.string({description: 'Description of LUIS application'}),
     versionId: flags.string({description: 'LUIS version Id. (mandatory, defaults to config:LUIS:versionId)'}),
     tokenizerVersion: flags.string({description: 'Version specifies how sentences are tokenized (optional). See also: https://aka.ms/luistokens'}),
-    save: flags.string({description: 'Save configuration settings from imported app (appId & endpoint)'}),
+    save: flags.boolean({description: 'Save configuration settings from imported app (appId & endpoint)'}),
   }
 
   async run() {
     const {flags} = this.parse(LuisApplicationCreate)
     const flagLabels = Object.keys(LuisApplicationCreate.flags)
     const configDir = this.config.configDir
 
-    const {
+    let {
       endpoint,
       subscriptionKey,
       name,
@@ -45,6 +45,8 @@ export default class LuisApplicationCreate extends Command {
 
     const usageScenario = 'Bot Framework'
 
+    if (!culture) culture = 'en-us'
+
     const requiredProps = {endpoint, subscriptionKey, name}
     utils.validateRequiredProps(requiredProps)
 
@@ -59,7 +61,8 @@ export default class LuisApplicationCreate extends Command {
       if (save) {
         const config = {
           appId: newAppId,
-          endpoint
+          endpoint,
+          subscriptionKey
         }
         const response = await this.saveImportedConfig(config, configDir)
         if (response) this.log('Config settings saved')

packages/luis/src/commands/luis/application/create.ts

@@ -5,6 +5,7 @@
 
 import {CLIError, Command, flags} from '@microsoft/bf-cli-command'
 
+const {cli} = require('cli-ux')
 const utils = require('../../../utils/index')
 
 export default class LuisApplicationDelete extends Command {
@@ -37,6 +38,13 @@ export default class LuisApplicationDelete extends Command {
 
     const client = utils.getLUISClient(subscriptionKey, endpoint)
 
+    if(!flags.appId) {
+      const deleteAppConfirmation = await cli.confirm(`Are you sure you would like to delete app with id: ${appId}? (Y/N)`)
+      if (!deleteAppConfirmation) {
+        return this.log('No action taken')
+      }
+    }
+
     try {
       const result = await client.apps.deleteMethod(appId)
       if (result.code === 'Success') {

packages/luis/src/commands/luis/application/delete.ts

@@ -31,7 +31,7 @@ export default class LuisApplicationImport extends Command {
 
     let {endpoint, subscriptionKey, name, inVal} = await utils.processInputs(flags, flagLabels, configDir)
 
-    const requiredProps = {endpoint, subscriptionKey, name}
+    const requiredProps = {endpoint, subscriptionKey}
     utils.validateRequiredProps(requiredProps)
 
     inVal = inVal ? inVal.trim() : flags.in