wingbot 3.37.0 → 3.37.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "wingbot",
-  "version": "3.37.0",
+  "version": "3.37.1",
   "description": "Enterprise Messaging Bot Conversation Engine",
   "main": "index.js",
   "scripts": {

package/src/ConversationTester.js

@@ -69,11 +69,15 @@ const DEFAULT_TEXT_THRESHOLD = 0.8;
  * @prop {List[]} lists
  */
 
+/** @typedef {import('./ReducerWrapper')} ReducerWrapper */
+/** @typedef {import('./Router')} Router */
+/** @typedef {import('./BuildRouter')} BuildRouter */
+
 /**
  * Callback for getting a tester
  *
  * @callback testerFactory
- * @param {Router|ReducerWrapper} bot - the chatbot itself
+ * @param {Router|ReducerWrapper|BuildRouter} bot - the chatbot itself
  * @param {TestsGroup} test - the chatbot itself
  * @returns {Tester}
  */
@@ -90,6 +94,12 @@ const DEFAULT_TEXT_THRESHOLD = 0.8;
  * @prop {number} stepCount
  */
 
+/**
+ * @typedef {object} Logger
+ * @prop {Function} log
+ * @prop {Function} error
+ */
+
 /**
  * Automated Conversation tests runner
  */
@@ -109,6 +119,7 @@ class ConversationTester {
      * @param {number} [options.textCasesPerStep]
      * @param {number} [options.textCaseParallel]
      * @param {boolean} [options.allowEmptyResponse]
+     * @param {Logger} [options.log]
      * @param {testerFactory} [options.testerFactory]
      */
     constructor (testsSource, botFactory, options = {}) {
@@ -122,6 +133,7 @@ class ConversationTester {
         };
         this._output = '';
         this._cachedBot = null;
+        this._log = options.log || console;
     }
 
     /**
@@ -159,18 +171,21 @@ class ConversationTester {
             this._cachedBot = null;
         }
         this._output = '';
-        const testCases = await this._getTestCases(lang);
-        const groups = this._getGroups(testCases);
-        const testsGroups = this._getTestsGroups(groups, step);
-        const stepCount = Number.isInteger(step) ? groups.length : 1;
 
         const botconfig = validationRequestBody;
         let failed = 0;
         let passed = 0;
         let skipped = 0;
         let summaryOutput = '';
+        let stepCount = 0;
+        let testsGroups = [];
 
         try {
+            const testCases = await this._getTestCases(lang);
+            const groups = this._getGroups(testCases);
+            testsGroups = this._getTestsGroups(groups, step);
+            stepCount = Number.isInteger(step) ? groups.length : 1;
+
             const results = [];
             for (const testsGroup of testsGroups) {
                 let stepResult;
@@ -224,6 +239,7 @@ class ConversationTester {
             }
 
         } catch (e) {
+            this._log.error('#Tester failed', e, { output: this._output, passed });
             this._output += `\nBot test failed: ${e.message}\n`;
         }
         return {

package/src/graphApi/conversationTestApi.js

@@ -11,6 +11,8 @@ const ConversationTester = require('../ConversationTester');
  * @prop {Function} getTestCases
  */
 
+/** @typedef {import('../ConversationTester').Logger} Logger */
+
 /**
  * Returns API for conversations testing
  *
@@ -26,6 +28,8 @@ const ConversationTester = require('../ConversationTester');
  * @param {number} [options.textCasesPerStep]
  * @param {number} [options.textCaseParallel]
  * @param {boolean} [options.allowEmptyResponse]
+ * @param {Logger} [options.log]
+ *
  * @param {string[]|Function} [acl] - limit api to array of groups or use auth function
  */
 function conversationTestApi (testsSource, botFactory, options, acl) {