Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[tests-only] [full-ci] Test quota improvements #39903

Closed
wants to merge 4 commits into from

Conversation

phil-davis
Copy link
Contributor

@phil-davis phil-davis commented Mar 21, 2022

WIP on tests for PR #39895

There are no fails of the existing tests.

* In case there are problems writing the contents of the file, try to
figure out the size of the content in several ways in order to provide a
more meaningful message
* Ensure the locks are properly released in case an error happens
* Handle quota while copying files. In addition, take into account the
space used by the file we're trying to overwrite (in case there is
already a file there)
* Part files will also be considered for quota while they're being
uploaded
@phil-davis phil-davis self-assigned this Mar 21, 2022
@sonarcloud
Copy link

sonarcloud bot commented Mar 21, 2022

SonarCloud Quality Gate failed.    Quality Gate failed

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

41.1% 41.1% Coverage
0.0% 0.0% Duplication

@phil-davis
Copy link
Contributor Author

I added some new test scenarios directly to #39895

@phil-davis phil-davis closed this Mar 21, 2022
@phil-davis phil-davis deleted the test-quota_improvements branch March 21, 2022 07:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants