Missing presence tracing (EXPOSUREAPP-14413)

[schauersbergern]

Ticket

Problem:
Current shareSelectedCheckIns() fun in CheckInsConsentViewModel produced following log:

2022-12-13T14:12:17.022Z D/CheckInsConsentViewModel: SelectedCheckIns=[2]
2022-12-13T14:12:19.900Z D/CheckInsConsentViewModel$shareSelectedCheckIns: Navigate to shareSelectedCheckIns
2022-12-13T14:12:19.900Z D/CheckInRepository: updateSubmissionConsents(checkInIds=[2], consent=true)
2022-12-13T14:12:19.901Z D/CheckInsConsentViewModel$resetPreviousSubmissionConsents: Trying to reset submission consents
2022-12-13T14:12:19.902Z D/CheckInsConsentViewModel$shareSelectedCheckIns: Navigate to SubmissionTestResultConsentGivenFragment
2022-12-13T14:12:19.909Z D/CheckInRepository: updateSubmissionConsents(checkInIds=[2], consent=false)

It seems like first launched resetPreviousSubmissionConsents() in this case is processed later in the task queue than updateSubmissionConsents. This sets the consent to true and then to false, resulting to behaviour that the checkin is not sent on submission: (see empty checkins)

2022-12-13T14:12:31.548Z D/SubmissionTask: Submitting SubmissionData(registrationToken=########-####-####-####-########39db, authCode=########-####-####-####-########92ab, temporaryExposureKeys=[# de.rki.coronawarnapp.server.protocols.external.exposurenotification.TemporaryExposureKeyExportOuterClass$TemporaryExposureKey@69cf4e69], consentToFederation=true, visitedCountries=[DE, IE, LV, HR, SI], unencryptedCheckIns=[], encryptedCheckIns=[], submissionType=SUBMISSION_TYPE_PCR_TEST)

Solution:
Changing resetPreviousSubmissionConsents() to a suspending function should ensure that checkInRepository.updateSubmissionConsents is not called until resetPreviousSubmissionConsents finished

[sonarqubecloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

100.0% Coverage
0.0% Duplication