Optimizely bug fix (#3677)

* country and state fix

* optimizely advanced audience targeting reject null id

* removing fb fapi changes

Author: joe-ayoub-segment

packages/destination-actions/src/destinations/optimizely-advanced-audience-targeting/syncAudience/__tests__/index.test.ts

@@ -64,6 +64,18 @@ describe('OptimizelyAdvancedAudienceTargeting.syncAudience', () => {
       ).resolves.not.toThrowError()
     })
 
+    it('should reject event when optimizelyUserId is null', async () => {
+      await expect(
+        testDestination.testAction('syncAudience', {
+          event: trackEvent,
+          useDefaultMappings: true,
+          mapping: {
+            optimizelyUserId: null
+          }
+        })
+      ).rejects.toThrowError()
+    })
+
     it('should handle errors response', async () => {
       nock('https://function.zaius.app/twilio_segment').post('/batch_sync_audience').reply(400)
 
@@ -153,6 +165,20 @@ describe('OptimizelyAdvancedAudienceTargeting.syncAudience', () => {
       ).resolves.not.toThrowError()
     })
 
+    it('should reject events when optimizelyUserId is null', async () => {
+      await testDestination.testBatchAction('syncAudience', {
+        events: identifyEvents,
+        useDefaultMappings: true,
+        mapping: {
+          optimizelyUserId: null
+        }
+      })
+
+      const multiStatus = testDestination.results[0].multistatus
+      expect(multiStatus).toBeDefined()
+      expect(multiStatus?.every((r) => r.status === 400)).toBe(true)
+    })
+
     it('should handle errors response', async () => {
       nock('https://function.zaius.app/twilio_segment').post('/batch_sync_audience').reply(400)
 

packages/destination-actions/src/destinations/optimizely-advanced-audience-targeting/syncAudience/index.ts

@@ -44,6 +44,7 @@ const action: ActionDefinition<Settings, Payload> = {
       description: 'The user identifier to sync to the Optimizely Audience',
       type: 'string',
       required: true,
+      allowNull: false,
       default: {
         '@if': {
           exists: { '@path': '$.userId' },