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

Fix crash when the UI queues multiple test removals while a slow network request is on-going (DEV) #2999

Merged
merged 2 commits into from Apr 28, 2021

Conversation

d4rken
Copy link
Member

@d4rken d4rken commented Apr 28, 2021

For data consistency, the test repo processes actions sequentially.
If we have a blocking operations (network refresh), the UI could queue up multiple test removals.
The first one would work, the following would throw an exception.
We should handle this silently for users of SubmissionRepository.

If we have a blocking operations (network refresh), the UI could queue up multiple test removals.
The first one would work, the following would throw an exception.
We should handle this silently for users of `SubmissionRepository`
@d4rken d4rken added bug Something isn't working maintainers Tag pull requests created by maintainers labels Apr 28, 2021
@d4rken d4rken added this to the 2.1.0 milestone Apr 28, 2021
@d4rken d4rken requested a review from a team April 28, 2021 16:44
@d4rken d4rken changed the title Fix crash when the queues multiple test removals while a slow network request is on-going (DEV) Fix crash when the UI queues multiple test removals while a slow network request is on-going (DEV) Apr 28, 2021
@chris-cwa chris-cwa self-assigned this Apr 28, 2021
@axelherbstreith axelherbstreith self-assigned this Apr 28, 2021
@sonarcloud
Copy link

sonarcloud bot commented Apr 28, 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

No Coverage information No Coverage information
0.0% 0.0% Duplication

@d4rken d4rken merged commit 9058112 into release/2.1.x Apr 28, 2021
@d4rken d4rken deleted the fix/DEV-corona-test-remova-network-delay branch April 28, 2021 17:06
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working maintainers Tag pull requests created by maintainers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants