-
Notifications
You must be signed in to change notification settings - Fork 499
Adjust test polling behavior 21 days after registration (EXPOSUREAPP-6882) #3016
Conversation
…ata set). Stop test polling after 21 days if we are in a final state (*_REDEEMED).
import org.joda.time.Instant | ||
|
||
fun CoronaTest.isOlderThan21Days(nowUTC: Instant): Boolean { | ||
return Duration(registeredAt, nowUTC).standardDays > 21 |
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.
Personal preference, but i would use a default param here
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.
NEVER!
(sry, wrong PR 🥇 )
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.
lgtm!
Kudos, SonarCloud Quality Gate passed! |
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.
Looks correct to me
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.
LGTM, verified for both PCR and RAT.
Map HTTP 400 errors to *_REDEEMED after 21 days (server has deleted data set).
Stop test polling after 21 days if we are in a final state (*_REDEEMED).
See ticket and related tech spec changes. 2.1.x hotfix target.
Check test cases, right PCR/RAT results?
Testable via mockserver and timetravel