[Facebook Conversions] Adding Test Event Code to all mappings.

[seg-leonelsanches]

Customers don't want to set the Test Event Code in Settings for all mapping. Suppose a customer with 10+ mappings: is it a good approach setting a Test Event Code when the intention is to test one mapping?

Testing

• Added unit tests for new functionality
• Tested end-to-end using the local server
• [If destination is already live] Tested for backward compatibility of destination. Note: New required fields are a breaking change.
• [Segmenters] Tested in the staging environment
• [Segmenters] [If applicable for this change] Tested for regression with Hadron.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 78.01%. Comparing base (2574714) to head (bc32a01).
Report is 29 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #2516      +/-   ##
==========================================
+ Coverage   77.99%   78.01%   +0.01%     
==========================================
  Files         991      991              
  Lines       17383    17398      +15     
  Branches     3281     3295      +14     
==========================================
+ Hits        13558    13573      +15     
  Misses       2737     2737              
  Partials     1088     1088              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[joe-ayoub-segment]

hi @seg-leonelsanches thanks for the PR.
The code looks good to me, but someone from Stratconn will need to review.
Could you also add a unit test for this please? and also show proof of testing.

hi @brennan - another SA enhancement raised by @seg-leonelsanches . Could someone from the Stratconn team please own reviewing this?

[marinhero]

Well done!

[joe-ayoub-segment]

Will deploy next Tuesday. No deploys this week due to election.

[joe-ayoub-segment]

Hi @seg-leonelsanches PR deployed