New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[SDK-416] Fix missing disqualification event after global opt-out #4606
Conversation
The disqualification events were not being recorded when toggling the global user participation.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks good.
Does this need a Swift version too?
assertEquals("Experiment branch must match", "test-branch", enrollmentEventExtras["branch"]) | ||
assertNotNull("Experiment enrollment-id must not be null", enrollmentEventExtras["enrollment_id"]) | ||
} | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice!
@@ -490,18 +490,25 @@ open class Nimbus( | |||
internal fun setGlobalUserParticipationOnThisThread(active: Boolean) = withCatchAll { | |||
val enrolmentChanges = nimbusClient.setGlobalUserParticipation(active) | |||
if (enrolmentChanges.isNotEmpty()) { | |||
recordExperimentTelemetryEvents(enrolmentChanges) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok.
No, Swift already handles this correctly. I could add tests in Swift to match the tests I added here, if you like. |
65a0909
to
24b5aa3
Compare
Codecov Report
@@ Coverage Diff @@
## main #4606 +/- ##
=======================================
Coverage 80.69% 80.69%
=======================================
Files 48 48
Lines 5217 5217
=======================================
Hits 4210 4210
Misses 1007 1007 Continue to review full report at Codecov.
|
The disqualification events were not being recorded when toggling the global user participation.