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

Split tests in CI #8319

Merged
merged 1 commit into from
Jul 24, 2024
Merged

Split tests in CI #8319

merged 1 commit into from
Jul 24, 2024

Conversation

yabirgb
Copy link
Member

@yabirgb yabirgb commented Jul 24, 2024

Closes #(issue_number)

Checklist

  • The PR modified the frontend, and updated the user guide to reflect the changes.

@yabirgb yabirgb force-pushed the mk8 branch 4 times, most recently from fb8a1eb to 3a014d3 Compare July 24, 2024 11:48
Copy link

codecov bot commented Jul 24, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 54.66%. Comparing base (9923d23) to head (7fa9cad).
Report is 15 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #8319       /-   ##
===========================================
  Coverage    51.56%   54.66%    3.10%     
===========================================
  Files         1728     1729        1     
  Lines       124855   124816      -39     
  Branches     16561    16562        1     
===========================================
  Hits         64384    68234     3850     
  Misses       58458    54256    -4202     
- Partials      2013     2326      313     
Flag Coverage Δ
backend 69.25% <ø> ( 6.73%) ⬆️

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.

@yabirgb yabirgb added the ready for final review Backend PR ready to be reviewed by great Lefteris label Jul 24, 2024
rotkehlchen/tests/fixtures/accounting.py Outdated Show resolved Hide resolved
"""Create and return the user data directory. Clean up old directory if existing"""
user_data_dir = data_dir / USERSDIR_NAME / username
if worker_id != 'master':
Copy link
Member

Choose a reason for hiding this comment

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

same here. Would need a comment to explain why this is here and what it does. Best in the docstring.

@yabirgb yabirgb force-pushed the mk8 branch 2 times, most recently from d799dc9 to 8b3cc63 Compare July 24, 2024 17:50
Copy link
Member

@LefterisJP LefterisJP left a comment

Choose a reason for hiding this comment

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

LGTM

@LefterisJP LefterisJP merged commit 2b8ae61 into rotki:develop Jul 24, 2024
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready for final review Backend PR ready to be reviewed by great Lefteris
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants