Skip to content
This repository has been archived by the owner on Jun 20, 2023. It is now read-only.

"Exposure check failed" without internet connection despite previous successful check (EXPOSUREAPP-13383) #5411

Merged
merged 9 commits into from
Aug 4, 2022
Merged

"Exposure check failed" without internet connection despite previous successful check (EXPOSUREAPP-13383) #5411

merged 9 commits into from
Aug 4, 2022

Conversation

mtwalli
Copy link
Contributor

@mtwalli mtwalli commented Aug 3, 2022 •
edited

Testing

  • Follow the steps mentioned in the ticket.
  • Before it shows failed risk calculation
  • After it shows low risk

Ticket

@mtwalli mtwalli added bug Something isn't working maintainers Tag pull requests created by maintainers labels Aug 3, 2022
@mtwalli mtwalli added this to the 2.26.x milestone Aug 3, 2022
@mtwalli mtwalli requested review from a team August 3, 2022 07:29
@mtwalli mtwalli changed the base branch from main to release/2.26.x August 3, 2022 07:30
@mtwalli
Copy link
Contributor Author

mtwalli commented Aug 3, 2022

Fixes: corona-warn-app/cwa-documentation#800

@mtwalli mtwalli mentioned this pull request Aug 3, 2022
Closed
3 tasks
@github-actions
Copy link

github-actions bot commented Aug 3, 2022 •
edited

Firebase Test Results

    2 files      2 suites   3m 49s ⏱️
129 tests 129 ✔️ 0 💤 0 ❌
258 runs  258 ✔️ 0 💤 0 ❌

Results for commit 24b796c.

♻️ This comment has been updated with latest results.

CV113
CV113 previously approved these changes Aug 3, 2022
View reviewed changes
Copy link
Contributor

@CV113 CV113 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

UA approved

@github-actions
Copy link

github-actions bot commented Aug 3, 2022 •
edited

Unit Test Results - Device For Tester Release

   563 files  ±0     563 suites  ±0   5m 56s ⏱️ +12s
2 775 tests ±0  2 775 ✔️ ±0  0 💤 ±0  0 ❌ ±0 
2 824 runs  ±0  2 824 ✔️ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit 24b796c. ± Comparison against base commit 56a0728.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actions bot commented Aug 3, 2022 •
edited

Unit Test Results - Device Release

   560 files     560 suites   5m 59s ⏱️
2 769 tests 2 769 ✔️ 0 💤 0 ❌
2 818 runs  2 818 ✔️ 0 💤 0 ❌

Results for commit 24b796c.

♻️ This comment has been updated with latest results.

@sonarcloud
Copy link

sonarcloud bot commented Aug 3, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

84.2% 84.2% Coverage
0.0% 0.0% Duplication

@mtwalli mtwalli merged commit 125c56d into release/2.26.x Aug 4, 2022
@mtwalli mtwalli deleted the fix/13383-pt-exposure-failure branch August 4, 2022 08:43
@mtwalli mtwalli mentioned this pull request Aug 9, 2022
Closed
3 tasks
@mtwalli mtwalli linked an issue Aug 9, 2022 that may be closed by this pull request
Exposure Check Failed #4740
Closed
3 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@schauersbergern schauersbergern schauersbergern approved these changes

@CV113 CV113 CV113 left review comments

Assignees

@schauersbergern schauersbergern

Labels
bug Something isn't working maintainers Tag pull requests created by maintainers
Projects
None yet
Milestone
2.26.x
Development

Successfully merging this pull request may close these issues.

Exposure Check Failed
3 participants
@mtwalli @schauersbergern @CV113