Stratconn 6646/other trait groups support (#3668)

* added other traits group support

* single dropdown for traits

* fix traitGroups response

* added includeTraits

* fix snapshots

* change to profile identifiers

* added test

* update comment

* check incorrect format key

* reject profiles with invalid trait format

* update desc

* multiStatusResponse all invalid profiles

* removed sent+body during validation

* handle perform function

* fix ExecuteInput

* perform err handling

* fix responses

* remove redundant tests

Author: monutwilio

packages/destination-actions/src/destinations/memora-internal/upsertProfile.types.ts

@@ -14,9 +14,9 @@ export interface Payload {
    */
   memora_store: string
   /**
-   * Contact identifiers (email and/or phone). At least one identifier is required.
+   * Profile identifiers (email and/or phone). At least one identifier is required. These identifiers are stored in the Contact trait group.
    */
-  contact_identifiers: {
+  profile_identifiers: {
     /**
      * User email address
      */
@@ -27,9 +27,9 @@ export interface Payload {
     phone?: string
   }
   /**
-   * Contact traits for the profile. At least one trait is required. These fields are dynamically loaded from the selected Memora Store.
+   * Traits for the profile from all trait groups. At least one trait is required. These fields are dynamically loaded from the selected Memora Store. When manually entering keys, use the format "TraitGroupName.$.traitName" (e.g., "Contact.$.firstName", "PurchaseHistory.$.lastPurchaseDate").
    */
-  contact_traits: {
+  profile_traits: {
     [k: string]: unknown
   }
 }

packages/destination-actions/src/destinations/memora/__tests__/snapshot.test.ts

@@ -35,11 +35,11 @@ describe(`Testing snapshot for ${destinationSlug} destination:`, () => {
       const mapping = {
         ...event.properties,
         memora_store: 'test-store-id',
-        contact_identifiers: {
+        profile_identifiers: {
           email: '[email protected]'
         },
-        contact_traits: {
-          firstName: 'Test'
+        profile_traits: {
+          'Contact.$.firstName': 'Test'
         }
       }
 
@@ -80,12 +80,12 @@ describe(`Testing snapshot for ${destinationSlug} destination:`, () => {
       const mapping = {
         ...event.properties,
         memora_store: 'test-store-id',
-        contact_identifiers: {
+        profile_identifiers: {
           email: '[email protected]'
         },
-        contact_traits: {
-          firstName: 'Test',
-          lastName: 'User'
+        profile_traits: {
+          'Contact.$.firstName': 'Test',
+          'Contact.$.lastName': 'User'
         }
       }
 
@@ -107,64 +107,5 @@ describe(`Testing snapshot for ${destinationSlug} destination:`, () => {
         expect(rawBody).toMatchSnapshot()
       }
     })
-
-    it(`${actionSlug} action - should throw error when memora_store is missing`, async () => {
-      const seedName = `${destinationSlug}#${actionSlug}`
-      const action = destination.actions[actionSlug]
-      const [eventData, settingsData] = generateTestData(seedName, destination, action, true)
-
-      const event = createTestEvent({
-        properties: eventData
-      })
-
-      const mapping = {
-        ...event.properties,
-        contact_identifiers: {
-          email: '[email protected]'
-        },
-        contact_traits: {
-          firstName: 'Test'
-        }
-      }
-
-      await expect(
-        testDestination.testAction(actionSlug, {
-          event: event,
-          mapping: mapping,
-          settings: settingsData,
-          auth: undefined
-        })
-      ).rejects.toThrow()
-    })
-
-    it(`${actionSlug} action - should throw error when profile has no traits`, async () => {
-      const seedName = `${destinationSlug}#${actionSlug}`
-      const action = destination.actions[actionSlug]
-      const [eventData, settingsData] = generateTestData(seedName, destination, action, true)
-
-      nock(/.*/).persist().get(/.*/).reply(200)
-      nock(/.*/).persist().post(/.*/).reply(202)
-      nock(/.*/).persist().put(/.*/).reply(202)
-
-      const event = createTestEvent({
-        properties: eventData
-      })
-
-      const mapping = {
-        ...event.properties,
-        memora_store: 'test-store-id',
-        contact_identifiers: {},
-        contact_traits: {}
-      }
-
-      await expect(
-        testDestination.testAction(actionSlug, {
-          event: event,
-          mapping: mapping,
-          settings: settingsData,
-          auth: undefined
-        })
-      ).rejects.toThrow('No valid profiles found for import')
-    })
   }
 })