Don't use group for listed browser

libraries/intentIqConstants/intentIqConstants.js

@@ -8,7 +8,7 @@ export const DEFAULT_PERCENTAGE = 95;
 export const CLIENT_HINTS_KEY = "_iiq_ch";
 export const EMPTY = "EMPTY";
 export const GVLID = "1323";
-export const VERSION = 0.37;
+export const VERSION = 0.371;
 export const PREBID = "pbjs";
 export const HOURS_24 = 86400000;
 export const HOURS_72 = HOURS_24 * 3;

modules/intentIqIdSystem.js

@@ -102,7 +102,7 @@
 
 function verifyIdType(value) {
   if (value === 0 || value === 1 || value === 3 || value === 4) return value;
   return -1;
 }
 
 function appendPartnersFirstParty(url, configParams) {
@@ -148,7 +148,7 @@
       +d[3]
     );
   } catch (e) {
     return NaN;
   }
 }
 
@@ -356,13 +356,18 @@
     const cmpData = getCmpData();
     const gdprDetected = cmpData.gdprString;
     firstPartyData = tryParse(readData(FIRST_PARTY_KEY_FINAL, allowedStorage));
-    actualABGroup = defineABTestingGroup(configParams, partnerData?.terminationCause);
-    if (groupChanged) groupChanged(actualABGroup, partnerData?.terminationCause);
     const currentBrowserLowerCase = detectBrowser();
     const browserBlackList = typeof configParams.browserBlackList === 'string' ? configParams.browserBlackList.toLowerCase() : '';
     const isBlacklisted = browserBlackList?.includes(currentBrowserLowerCase);
     let newUser = false;
 
+    if (!isBlacklisted) {
+      actualABGroup = defineABTestingGroup(configParams, partnerData?.terminationCause);
+      if (groupChanged) groupChanged(actualABGroup, partnerData?.terminationCause);
+    } else {
+      actualABGroup = undefined;
+    }
+
     setGamReporting(gamObjectReference, gamParameterName, actualABGroup, isBlacklisted);
 
     callbackTimeoutID = setTimeout(() => {
@@ -421,7 +426,7 @@
     if (chSupported) {
       chPromise = fetchAndHandleCH();
       chPromise.catch(err => {
         logError('fetchAndHandleCH failed', err);
       });
     } else {
       clientHints = '';
@@ -437,8 +442,8 @@
     if (typeof partnerData.failCount === 'number') failCount = partnerData.failCount;
     if (typeof partnerData.noDataCounter === 'number') noDataCount = partnerData.noDataCounter;
     if (partnerData.wsrvcll) {
       partnerData.wsrvcll = false;
       storeData(PARTNER_DATA_KEY, JSON.stringify(partnerData), allowedStorage, firstPartyData);
     }
 
     if (partnerData.data) {
@@ -505,7 +510,7 @@
             .then(ch => buildAndSendPixel(ch || ''));
         }
       } else {
         buildAndSendPixel('');
       }
       return;
     }
@@ -568,7 +573,7 @@
             };
             if (callbackTimeoutID) clearTimeout(callbackTimeoutID);
             if ('cttl' in respJson) {
               partnerData.cttl = respJson.cttl;
             } else partnerData.cttl = HOURS_72;
 
             if ('tc' in respJson) {
@@ -609,7 +614,7 @@
               }
               // If data is empty, means we should save as INVALID_ID
               if (respJson.data === '') {
                 respJson.data = INVALID_ID;
               } else {
                 // If data is a single string, assume it is an id with source intentiq.com
                 if (respJson.data && typeof respJson.data === 'string') {
@@ -624,7 +629,7 @@
             }
 
             if ('sid' in respJson) {
               partnerData.siteId = respJson.sid;
             }
 
             if ('spd' in respJson) {
@@ -633,14 +638,14 @@
             }
 
             if ('abTestUuid' in respJson) {
               if ('ls' in respJson && respJson.ls === true) {
                 partnerData.abTestUuid = respJson.abTestUuid;
               }
             }
 
             if ('gpr' in respJson) {
               // GAM prediction reporting
               partnerData.gpr = respJson.gpr;
             } else {
               delete partnerData.gpr; // remove prediction flag in case server doesn't provide it
             }

test/spec/modules/intentIqIdSystem_spec.js

@@ -2029,6 +2029,42 @@ describe('IntentIQ tests', function () {
     expect(groupChangedSpy.calledWith(usedGroup)).to.be.true;
   });
 
+  it('should NOT call groupChanged when the current browser is blacklisted', async function () {
+    const groupChangedSpy = sinon.spy();
+    const blk = detectBrowser();
+    const configParams = {
+      params: {
+        ...defaultConfigParams.params,
+        browserBlackList: blk,
+        groupChanged: groupChangedSpy
+      }
+    };
+
+    intentIqIdSubmodule.getId(configParams);
+    await waitForClientHints();
+
+    expect(groupChangedSpy.called).to.be.false;
+  });
+
+  it('should not mark a test group on the sync pixel when the current browser is blacklisted', async function () {
+    const blk = detectBrowser();
+    const configParams = {
+      params: {
+        ...defaultConfigParams.params,
+        browserBlackList: blk
+      }
+    };
+
+    intentIqIdSubmodule.getId(configParams);
+    await waitForClientHints();
+
+    const pixelRequest = server.requests[0];
+    expect(pixelRequest).to.exist;
+    expect(pixelRequest.url).to.include('at=20');
+    expect(pixelRequest.url).to.not.include('testGroup=');
+    expect(pixelRequest.url).to.include('isInTestGroup=false');
+  });
+
   it('should include testPercentage with configured abPercentage in AT=39 URL', async function () {
     const callBackSpy = sinon.spy();
     const configParams = {