@webex/internal-plugin-conversation 3.7.0-next.1 → 3.7.0-next.3

package/dist/share-activity.js

@@ -391,7 +391,7 @@ var ShareActivity = _webexCore.WebexPlugin.extend({
       return res.body.spaceUrl;
     });
   },
-  version: "3.7.0-next.1"
+  version: "3.7.0-next.3"
 });
 
 /**

package/package.json

@@ -19,12 +19,12 @@
     ]
   },
   "dependencies": {
-    "@webex/common": "3.7.0-next.1",
-    "@webex/helper-html": "3.7.0-next.1",
-    "@webex/helper-image": "3.7.0-next.1",
-    "@webex/internal-plugin-encryption": "3.7.0-next.1",
-    "@webex/internal-plugin-user": "3.7.0-next.1",
-    "@webex/webex-core": "3.7.0-next.1",
+    "@webex/common": "3.7.0-next.2",
+    "@webex/helper-html": "3.7.0-next.2",
+    "@webex/helper-image": "3.7.0-next.2",
+    "@webex/internal-plugin-encryption": "3.7.0-next.2",
+    "@webex/internal-plugin-user": "3.7.0-next.2",
+    "@webex/webex-core": "3.7.0-next.2",
     "crypto-js": "^4.1.1",
     "lodash": "^4.17.21",
     "node-scr": "^0.3.0",
@@ -36,13 +36,13 @@
     "@webex/eslint-config-legacy": "0.0.0",
     "@webex/jest-config-legacy": "0.0.0",
     "@webex/legacy-tools": "0.0.0",
-    "@webex/test-helper-chai": "3.7.0-next.1",
-    "@webex/test-helper-file": "3.7.0-next.1",
-    "@webex/test-helper-make-local-url": "3.7.0-next.1",
-    "@webex/test-helper-mocha": "3.7.0-next.1",
-    "@webex/test-helper-mock-webex": "3.7.0-next.1",
-    "@webex/test-helper-retry": "3.7.0-next.1",
-    "@webex/test-helper-test-users": "3.7.0-next.1",
+    "@webex/test-helper-chai": "3.7.0-next.2",
+    "@webex/test-helper-file": "3.7.0-next.2",
+    "@webex/test-helper-make-local-url": "3.7.0-next.2",
+    "@webex/test-helper-mocha": "3.7.0-next.2",
+    "@webex/test-helper-mock-webex": "3.7.0-next.2",
+    "@webex/test-helper-retry": "3.7.0-next.2",
+    "@webex/test-helper-test-users": "3.7.0-next.2",
     "eslint": "^8.24.0",
     "prettier": "^2.7.1",
     "sinon": "^9.2.4"
@@ -56,5 +56,5 @@
     "test:style": "eslint ./src/**/*.*",
     "test:unit": "webex-legacy-tools test --unit --runner jest"
   },
-  "version": "3.7.0-next.1"
+  "version": "3.7.0-next.3"
 }

package/src/conversation.js

@@ -340,33 +340,36 @@ const Conversation = WebexPlugin.extend({
    * @param {String} recipientId,
    * @returns {Promise<Activity>}
    */
-  addReaction(conversation, displayName, activity, recipientId) {
-    return this.createReactionHmac(displayName, activity).then((hmac) => {
-      const addReactionPayload = {
-        actor: {objectType: 'person', id: this.webex.internal.device.userId},
-        target: {
-          id: conversation.id,
-          objectType: 'conversation',
-        },
-        verb: 'add',
-        objectType: 'activity',
-        parent: {
-          type: 'reaction',
-          id: activity.id,
-        },
-        object: {
-          objectType: 'reaction2',
-          displayName,
-          hmac,
-        },
-      };
+  async addReaction(conversation, displayName, activity, recipientId) {
+    let hmac;
+    if (this.config.includeEncryptionTransforms) {
+      hmac = await this.createReactionHmac(displayName, activity);
+    }
 
-      if (recipientId) {
-        addReactionPayload.recipients = {items: [{id: recipientId, objectType: 'person'}]};
-      }
+    const addReactionPayload = {
+      actor: {objectType: 'person', id: this.webex.internal.device.userId},
+      target: {
+        id: conversation.id,
+        objectType: 'conversation',
+      },
+      verb: 'add',
+      objectType: 'activity',
+      parent: {
+        type: 'reaction',
+        id: activity.id,
+      },
+      object: {
+        objectType: 'reaction2',
+        displayName,
+        hmac,
+      },
+    };
 
-      return this.sendReaction(conversation, addReactionPayload);
-    });
+    if (recipientId) {
+      addReactionPayload.recipients = {items: [{id: recipientId, objectType: 'person'}]};
+    }
+
+    return this.sendReaction(conversation, addReactionPayload);
   },
 
   /**

package/test/unit/spec/conversation.js

@@ -41,49 +41,27 @@ describe('plugin-conversation', () => {
         const {conversation} = webex.internal;
         const recipientId = 'example-recipient-id';
         const expected = {items: [{id: recipientId, objectType: 'person'}]};
+        conversation.config.includeEncryptionTransforms = true;
         conversation.sendReaction = sinon.stub().returns(Promise.resolve());
         conversation.createReactionHmac = sinon.stub().returns(Promise.resolve('hmac'));
 
         return conversation.addReaction({}, 'example-display-name', {}, recipientId).then(() => {
+          assert.called(conversation.createReactionHmac);
           assert.deepEqual(conversation.sendReaction.args[0][1].recipients, expected);
         });
       });
-    });
-
-    describe('deleteReaction()', () => {
-      it('should add recipients to the payload if provided', () => {
-        const {conversation} = webex.internal;
-        const recipientId = 'example-recipient-id';
-        const expected = {items: [{id: recipientId, objectType: 'person'}]};
-        conversation.sendReaction = sinon.stub().returns(Promise.resolve());
-
-        return conversation.deleteReaction({}, 'example-reaction-id', recipientId).then(() => {
-          assert.deepEqual(conversation.sendReaction.args[0][1].recipients, expected);
-        });
-      });
-    });
 
-    describe('prepare()', () => {
-      it('should ammend activity recipients to the returned object', () => {
-        const {conversation} = webex.internal;
-        const activity = {recipients: 'example-recipients'};
-
-        return conversation.prepare(activity).then((results) => {
-          assert.deepEqual(results.recipients, activity.recipients);
-        });
-      });
-    });
-
-    describe('addReaction()', () => {
-      it('should add recipients to the payload if provided', () => {
+      it('will not call createReactionHmac if config prohibits', () => {
         const {conversation} = webex.internal;
         const recipientId = 'example-recipient-id';
         const expected = {items: [{id: recipientId, objectType: 'person'}]};
+        conversation.config.includeEncryptionTransforms = false;
         conversation.sendReaction = sinon.stub().returns(Promise.resolve());
-        conversation.createReactionHmac = sinon.stub().returns(Promise.resolve('hmac'));
+        conversation.createReactionHmac = sinon.stub();
 
         return conversation.addReaction({}, 'example-display-name', {}, recipientId).then(() => {
           assert.deepEqual(conversation.sendReaction.args[0][1].recipients, expected);
+          assert.notCalled(conversation.createReactionHmac);
         });
       });
     });