From dad78a04541b151fa5d7bb57c23ad4684a9ef73d Mon Sep 17 00:00:00 2001
From: melgenek <ymelnyk@liveintent.com>
Date: Mon, 15 Mar 2021 18:16:57 +0100
Subject: [PATCH 1/2] Don't parse response twice in LiveIntent Id submodule

---
 modules/liveIntentIdSystem.js                | 10 +---------
 test/spec/modules/liveIntentIdSystem_spec.js | 10 +++++++---
 2 files changed, 8 insertions(+), 12 deletions(-)

diff --git a/modules/liveIntentIdSystem.js b/modules/liveIntentIdSystem.js
index f87d67aae8e..5a955eefa92 100644
--- a/modules/liveIntentIdSystem.js
+++ b/modules/liveIntentIdSystem.js
@@ -166,15 +166,7 @@ export const liveIntentIdSubmodule = {
     const result = function(callback) {
       liveConnect.resolve(
         response => {
-          let responseObj = {};
-          if (response) {
-            try {
-              responseObj = JSON.parse(response);
-            } catch (error) {
-              utils.logError(error);
-            }
-          }
-          callback(responseObj);
+          callback(response);
         },
         error => {
           utils.logError(`${MODULE_NAME}: ID fetch encountered an error: `, error);
diff --git a/test/spec/modules/liveIntentIdSystem_spec.js b/test/spec/modules/liveIntentIdSystem_spec.js
index 3b4e4b9d9a7..b03638c5848 100644
--- a/test/spec/modules/liveIntentIdSystem_spec.js
+++ b/test/spec/modules/liveIntentIdSystem_spec.js
@@ -36,7 +36,7 @@ describe('LiveIntentId', function() {
     resetLiveIntentIdSubmodule();
   });
 
-  it('should initialize LiveConnect with a privacy string when getId, and include it in the resolution request', function() {
+  it('should initialize LiveConnect with a privacy string when getId, and include it in the resolution request', function () {
     uspConsentDataStub.returns('1YNY');
     gdprConsentDataStub.returns({
       gdprApplies: true,
@@ -47,12 +47,16 @@ describe('LiveIntentId', function() {
     submoduleCallback(callBackSpy);
     let request = server.requests[1];
     expect(request.url).to.match(/.*us_privacy=1YNY.*&gdpr=1&gdpr_consent=consentDataString.*/);
+    const response = {
+      unifiedId: 'a_unified_id',
+      segments: [123, 234]
+    }
     request.respond(
       200,
       responseHeader,
-      JSON.stringify({})
+      JSON.stringify(response)
     );
-    expect(callBackSpy.calledOnce).to.be.true;
+    expect(callBackSpy.calledOnceWith(response)).to.be.true;
   });
 
   it('should fire an event when getId', function() {

From b6ba432407e45b964591c787442f51fa1f827f7c Mon Sep 17 00:00:00 2001
From: melgenek <ymelnyk@liveintent.com>
Date: Mon, 15 Mar 2021 18:31:33 +0100
Subject: [PATCH 2/2] Update the liveintent module test to have the 204
 response

---
 test/spec/modules/liveIntentIdSystem_spec.js | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/test/spec/modules/liveIntentIdSystem_spec.js b/test/spec/modules/liveIntentIdSystem_spec.js
index b03638c5848..f1de2f3bf93 100644
--- a/test/spec/modules/liveIntentIdSystem_spec.js
+++ b/test/spec/modules/liveIntentIdSystem_spec.js
@@ -135,11 +135,10 @@ describe('LiveIntentId', function() {
     let request = server.requests[1];
     expect(request.url).to.be.eq('https://dummy.liveintent.com/idex/prebid/89899');
     request.respond(
-      200,
-      responseHeader,
-      JSON.stringify({})
+      204,
+      responseHeader
     );
-    expect(callBackSpy.calledOnce).to.be.true;
+    expect(callBackSpy.calledOnceWith({})).to.be.true;
   });
 
   it('should call the default url of the LiveIntent Identity Exchange endpoint, with a partner', function() {