Confirm prompt had bogus RegEx expressions. Missed adding a Unittest to the mocha test suite.

Author: Stevenic

Node/lib/dialogs/EntityRecognizer.js

@@ -193,8 +193,8 @@ var EntityRecognizer = (function () {
         }
     };
     EntityRecognizer.dateExp = /^\d{4}-\d{2}-\d{2}/i;
-    EntityRecognizer.yesExp = /^(1|y|yes|yep|sure|ok|true)\z/i;
-    EntityRecognizer.noExp = /^(0|n|no|nope|not|false)\z/i;
+    EntityRecognizer.yesExp = /^(1|y|yes|yep|sure|ok|true)/i;
+    EntityRecognizer.noExp = /^(0|n|no|nope|not|false)/i;
     EntityRecognizer.numberExp = /[+-]?(?:\d+\.?\d*|\d*\.?\d+)/;
     return EntityRecognizer;
 })();

Node/package.json

@@ -2,7 +2,7 @@
   "name": "botbuilder",
   "author": "Microsoft Corp.",
   "description": "Bot Builder is a dialog system for building rich bots on virtually any platform.",
-  "version": "0.6.4",
+  "version": "0.6.5",
   "license": "MIT",
   "keywords": [
     "bots",

Node/src/dialogs/EntityRecognizer.ts

@@ -62,8 +62,8 @@ export interface IFindMatchResult {
 
 export class EntityRecognizer {
     static dateExp = /^\d{4}-\d{2}-\d{2}/i;
-    static yesExp = /^(1|y|yes|yep|sure|ok|true)\z/i;
-    static noExp = /^(0|n|no|nope|not|false)\z/i;
+    static yesExp = /^(1|y|yes|yep|sure|ok|true)/i;
+    static noExp = /^(0|n|no|nope|not|false)/i;
     static numberExp = /[+-]?(?:\d+\.?\d*|\d*\.?\d+)/;
 
     static findEntity(entities: IEntity[], type: string): IEntity {

Node/tests/Dialogs.js

@@ -43,6 +43,10 @@ describe('Dialogs', function() {
             },
             function (session, results) {
                 assert(results.response && results.response.entity === 'green');
+                builder.Prompts.confirm(session, 'Is green your choice?');
+            },
+            function (session, results) {
+                assert(results.response && results.response === true);
                 builder.Prompts.time(session, 'enter a time');
             },
             function (session, results) {
@@ -65,9 +69,12 @@ describe('Dialogs', function() {
                     bot.processMessage({ text: 'green' });
                     break;
                 case 4:
-                    bot.processMessage({ text: 'in 5 minutes' });
+                    bot.processMessage({ text: 'yes' });
                     break;
                 case 5:
+                    bot.processMessage({ text: 'in 5 minutes' });
+                    break;
+                case 6:
                     assert(message.text == 'done');
                     done();
                     break;