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

Fixes issues and tests on indexes #555

Merged
merged 5 commits into from
Jul 29, 2022
Merged

Conversation

brichet
Copy link
Collaborator

@brichet brichet commented Jul 29, 2022

This PR :

  • updates indexes after each way to upload or delete a package
  • adds some tests on update indexes function
  • adds tests on duplicate upload, with and without force flag
  • adds some tests on repodata.json file after deleting a package

@brichet brichet added bug Something isn't working maintenance labels Jul 29, 2022
@codecov-commenter
Copy link

codecov-commenter commented Jul 29, 2022

Codecov Report

Merging #555 (6a41176) into main (bee4a48) will decrease coverage by 0.82%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #555      +/-   ##
==========================================
- Coverage   82.87%   82.04%   -0.83%     
==========================================
  Files          78       78              
  Lines        5973     5977       +4     
==========================================
- Hits         4950     4904      -46     
- Misses       1023     1073      +50     
Impacted Files Coverage Δ
quetz/main.py 86.70% <100.00%> (+0.50%) ⬆️
quetz/pkgstores.py 42.45% <0.00%> (-11.60%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update bee4a48...6a41176. Read the comment docs.

@wolfv
Copy link
Member

wolfv commented Jul 29, 2022

Thank you @brichet this looks great!

@wolfv wolfv merged commit 5d4eb9f into mamba-org:main Jul 29, 2022
@brichet brichet deleted the fix/repodata branch August 17, 2022 06:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working maintenance
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants