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

WIP: Async test (ignore) #272

Draft
wants to merge 12 commits into
base: master
Choose a base branch
from
Draft

WIP: Async test (ignore) #272

wants to merge 12 commits into from

Conversation

tessus
Copy link
Collaborator

@tessus tessus commented Apr 3, 2024

Testing. Please ignore.

Rebased #246 on master

RealOrangeOne and others added 11 commits April 3, 2024 17:14
Except `Directory`, as there's no async-compatible `glob` implementation
This required a little refactor, but the code is now much cleaner, and correctly handles updating the config
This prevents tests from conflicting with eachother if they don't clean up after themselves correctly.
@codecov-commenter
Copy link

codecov-commenter commented Apr 3, 2024

Codecov Report

Attention: Patch coverage is 62.58993% with 52 lines in your changes are missing coverage. Please review.

Project coverage is 82.16%. Comparing base (15e3619) to head (a7a5977).

Files Patch % Lines
src/main.rs 0.00% 27 Missing ⚠️
src/server.rs 80.00% 12 Missing ⚠️
src/paste.rs 69.56% 7 Missing ⚠️
src/config.rs 0.00% 3 Missing ⚠️
src/util.rs 87.50% 2 Missing ⚠️
src/auth.rs 50.00% 1 Missing ⚠️

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #272      +/-   ##
==========================================
- Coverage   82.47%   82.16%   -0.32%     
==========================================
  Files          11       11              
  Lines        1204     1166      -38     
==========================================
- Hits          993      958      -35     
+ Misses        211      208       -3     
Flag Coverage Δ
unit-tests 82.16% <62.58%> (-0.32%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@tessus tessus mentioned this pull request Apr 3, 2024
12 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants