Minor fixes

Author: Batta32

tools/botskills/src/utils/authenticationUtils.ts

@@ -243,7 +243,7 @@ export class AuthenticationUtils {
         } catch (err) {
             logger.warning(`Could not configure authentication connection automatically.`);
             if (currentCommand.length > 0) {
-                logger.warning(`There was an error while executing the following command:\n\t${currentCommand.join(' ')}\\${err.message}`);
+                logger.warning(`There was an error while executing the following command:\n\t${currentCommand.join(' ')}\n${err.message}`);
                 logger.warning(`You must configure one of the following connection types MANUALLY in the Azure Portal:
         ${manifest.authenticationConnections.map((authConn: IAuthenticationConnection) => authConn.serviceProviderId)
                     .join(', ')}`);

tools/botskills/test/authentication.test.js

@@ -8,7 +8,7 @@ const { writeFileSync } = require("fs");
 const { join, resolve } = require("path");
 const sandbox = require("sinon").createSandbox();
 const { TestLogger } = require("./helpers/testLogger");
-const { AuthenticationUtils } = require("../lib/utils/index");
+const { AuthenticationUtils } = require("../lib/utils");
 const authenticationUtils = new AuthenticationUtils();
 const emptyAzureAuthSettings = JSON.stringify(require(resolve(__dirname, join("mocks", "azureAuthSettings", "emptyAuthSettings.json"))));
 const filledAzureAuthSettings = JSON.stringify(require(resolve(__dirname, join("mocks", "azureAuthSettings", "filledAuthSettings.json"))));
@@ -141,7 +141,7 @@ https://github.com/microsoft/botframework-solutions/blob/master/docs/howto/assis
 https://github.com/microsoft/botframework-solutions/blob/master/docs/howto/assistant/linkedaccounts.md#authentication-configuration`);
             strictEqual(warningList[warningList.length - 2], `You must configure one of the following connection types MANUALLY in the Azure Portal:
         Azure Active Directory v2`);
-            strictEqual(warningList[warningList.length - 3], `There was an error while executing the following command:\n\taz ad app show --id \\Mocked function throws an Error`)
+            strictEqual(warningList[warningList.length - 3], `There was an error while executing the following command:\n\taz ad app show --id \nMocked function throws an Error`)
         });
 
         it("when the scopes are not configured automatically", async function() {

tools/botskills/test/childProcess.test.js

@@ -4,8 +4,7 @@
  */
 
 const { strictEqual } = require("assert");
-const { ChildProcessUtils } = require("../lib/utils/childProcessUtils");
-const sandbox = require("sinon").createSandbox();
+const { ChildProcessUtils } = require("../lib/utils");
 const childProcessUtils = new ChildProcessUtils();
 const validOutput = "testing childProcess util";
 const unrecognizeCommand = "unrecognizeCommand";

tools/botskills/test/sanitization.test.js

@@ -4,7 +4,7 @@
  */
 
 const { strictEqual } = require("assert");
-const { sanitizePath } = require("../lib/utils/index");
+const { sanitizePath } = require("../lib/utils");
 
 describe("The sanitization path util", function () {
     describe("should return a path without trailing backslash", function () {

tools/botskills/test/validation.test.js

@@ -4,7 +4,7 @@
  */
 
 const { strictEqual } = require("assert");
-const { validatePairOfArgs } = require("../lib/utils/index");
+const { validatePairOfArgs } = require("../lib/utils");
 
 describe("The validation util", function() {
     describe("should return a message", function() {