fix: Attribute null object reference

[jamesnrokt]

Instructions

• Using SetUserTag would cause Attempt to invoke virtual method 'java.lang.String java.lang.Object.toString()' on a null object reference

Summary

• Perform null check before adding to final attributes

Testing Plan

• Was this tested locally? If not, explain why.
• Tested as part of the Maui integration

Reference Issue (For mParticle employees only. Ignore if you are an outside contributor)

• Closes N/A

[rmi22186]

Good catch @jamesnrokt . One question - In MP, we can set an attribute directly to be null using these user tags. In the example on our docs, https://docs.mparticle.com/developers/client-sdks/android/users/#set-user-tags, it uses platinum_member: NULL as the example.

In Rokt, is null not a valid value, and so we skip the setting of those attributes, as opposed to having the option to set them as null?

[rmi22186]

Can you also add a test for passing a null value?

[jamesnrokt]

Added a test for passing in a null value

I believe Rokt doesn't use null values. For something like platinum_member we'd do key value "membershipLevel": "platinum"

[rmi22186]

LGTM - just failing a check due to the workflow not being updated to permit PR-ing to main, but no big deal.

[jamesnrokt]

Thanks Rob

LGTM - just failing a check due to the workflow not being updated to permit PR-ing to main, but no big deal.

Already have a PR in place for that check: mParticle/mparticle-workflows#88