remove 'trigger(...)' wrap from addDialogTrigger (microsoft#2797)

Author: pcostantini

Node/core/lib/dialogs/ActionSet.js

@@ -18,7 +18,7 @@ var ActionSet = (function () {
     ActionSet.prototype.addDialogTrigger = function (actions, dialogId) {
         if (this.trigger) {
             this.trigger.localizationNamespace = dialogId.split(':')[0];
-            actions.beginDialogAction('trigger(' + dialogId + ')', dialogId, this.trigger);
+            actions.beginDialogAction(dialogId, dialogId, this.trigger);
         }
     };
     ActionSet.prototype.findActionRoutes = function (context, callback) {

Node/core/src/dialogs/ActionSet.ts

@@ -91,7 +91,7 @@ export class ActionSet {
     public addDialogTrigger(actions: ActionSet, dialogId: string): void {
         if (this.trigger) {
             this.trigger.localizationNamespace = dialogId.split(':')[0];
-            actions.beginDialogAction('trigger(' + dialogId + ')', dialogId, this.trigger);
+            actions.beginDialogAction(dialogId, dialogId, this.trigger);
         }
     }
 

Node/core/tests/actions.js

@@ -64,6 +64,7 @@ describe('actions', function() {
             onSelectAction: function (session, args, next) {
                 assert(session !== null);
                 assert(args !== null);
+                assert(args.action === '/');
                 assert(next !== null);
                 done();
             }