fix(triggerEvents): fix response format for request with triggerEvents (

#2546)

* fix(triggerEvents): fix format of response when triggerEvents active, the response was formatted twice

* chore(test): fixed test since response modified with triggerEvents

lib/api/funnel.js

@@ -754,7 +754,8 @@ class Funnel {
   async executePluginRequest(request) {
     try {
       if (request.input.triggerEvents) {
-        return await this.processRequest(request);
+        const response = await this.processRequest(request);
+        return { ...response.result };
       }
       return await doAction(this.getController(request), request);
     } catch (e) {

test/api/funnel/executePluginRequest.test.js

@@ -104,26 +104,24 @@ describe("funnel.executePluginRequest", () => {
 
   it("should trigger pipes if triggerEvent is enabled", async () => {
     const request = new Request({
-      controller: "testme",
       action: "succeed",
+      controller: "testme",
       triggerEvents: true,
     });
 
-    return funnel.executePluginRequest(request).then((response) => {
-      should(response).be.exactly(request);
+    return funnel.executePluginRequest(request).then(() => {
       should(kuzzle.pipe).calledWith("testme:beforeSucceed");
       should(kuzzle.pipe).calledWith("testme:afterSucceed");
     });
   });
 
   it("should not trigger pipes if triggerEvent is disabled", async () => {
     const request = new Request({
-      controller: "testme",
       action: "succeed",
+      controller: "testme",
     });
 
-    return funnel.executePluginRequest(request).then((response) => {
-      should(response).be.exactly(request);
+    return funnel.executePluginRequest(request).then(() => {
       should(kuzzle.pipe).not.calledWith("testme:beforeSucceed");
       should(kuzzle.pipe).not.calledWith("testme:afterSucceed");
     });