no ab group in browserblacklist

modules/intentIqIdSystem.ts

@@ -227,7 +227,7 @@
 
 function verifyIdType(value: number): number {
   if (value === 0 || value === 1 || value === 3 || value === 4) return value;
   return -1;
 }
 
 function appendPartnersFirstParty(url: string, configParams: any): string {
@@ -273,7 +273,7 @@
       +d[3]
     );
   } catch (e) {
     return NaN;
   }
 }
 
@@ -474,11 +474,16 @@
     const cmpData = getCmpData();
     const gdprDetected = cmpData.gdprString;
     firstPartyData = tryParse(readData(FIRST_PARTY_KEY_FINAL, allowedStorage) as string);
-    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);
+
+    if (!isBlacklisted) {
+      actualABGroup = defineABTestingGroup(configParams, partnerData?.terminationCause);
+      if (groupChanged) groupChanged(actualABGroup, partnerData?.terminationCause);
+    } else {
+      actualABGroup = undefined;
+    }
     let newUser = false;
 
     setGamReporting(gamObjectReference, gamParameterName, actualABGroup, isBlacklisted);
@@ -539,7 +544,7 @@
     if (chSupported) {
       chPromise = fetchAndHandleCH();
       chPromise.catch((err: any) => {
         logError('fetchAndHandleCH failed', err);
       });
     } else {
       clientHints = '';
@@ -555,8 +560,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) {
@@ -623,7 +628,7 @@
             .then((ch: any) => buildAndSendPixel(ch || ''));
         }
       } else {
         buildAndSendPixel('');
       }
       return;
     }
@@ -686,7 +691,7 @@
             };
             if (callbackTimeoutID) clearTimeout(callbackTimeoutID);
             if ('cttl' in respJson) {
               partnerData.cttl = respJson.cttl;
             } else partnerData.cttl = HOURS_72;
 
             if ('tc' in respJson) {
@@ -727,7 +732,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') {
@@ -742,7 +747,7 @@
             }
 
             if ('sid' in respJson) {
               partnerData.siteId = respJson.sid;
             }
 
             if ('spd' in respJson) {
@@ -751,14 +756,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,79 @@
     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 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 = {