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

Add unit test for PresenceTracingRiskRepository (EXPOSUREAPP-6332) #2814

Merged
merged 7 commits into from
Apr 13, 2021
Merged

Add unit test for PresenceTracingRiskRepository (EXPOSUREAPP-6332) #2814

merged 7 commits into from
Apr 13, 2021

Conversation

chiljamgossow
Copy link
Contributor

Unit tests
Refactoring/ clean up
Ensures only data from last 14 days are used
Correct assignment of latest matches to risk result

@chiljamgossow chiljamgossow requested review from a team and d4rken April 13, 2021 05:46
@chiljamgossow chiljamgossow added the maintainers Tag pull requests created by maintainers label Apr 13, 2021
@chiljamgossow chiljamgossow added this to the 2.0.0 milestone Apr 13, 2021
d4rken
d4rken approved these changes Apr 13, 2021
View reviewed changes
Copy link
Member

@d4rken d4rken left a comment

Choose a reason for hiding this comment

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

lgtm

@BMItr BMItr self-assigned this Apr 13, 2021
@sonarcloud
Copy link

sonarcloud bot commented Apr 13, 2021

Kudos, SonarCloud Quality Gate passed!

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

48.8% 48.8% Coverage
0.0% 0.0% Duplication

@chiljamgossow chiljamgossow merged commit b3e8472 into release/2.0.x Apr 13, 2021
@chiljamgossow chiljamgossow deleted the feature/6332-test-repo branch April 13, 2021 14:10
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@BMItr BMItr BMItr approved these changes

@d4rken d4rken d4rken approved these changes

Assignees

@d4rken d4rken

@BMItr BMItr

Labels
maintainers Tag pull requests created by maintainers
Projects
None yet
Milestone
2.0.0
Development

Successfully merging this pull request may close these issues.
4 participants
@chiljamgossow @d4rken @BMItr @harambasicluka