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: unit: explicitly ask for dvc repo #9628

Merged
merged 1 commit into from
Jun 19, 2023
Merged

Conversation

efiop
Copy link
Contributor

@efiop efiop commented Jun 19, 2023

Currently, these rely on the root of the repo being in the dvc repo.

Leftover from #9627

@efiop efiop added refactoring Factoring and re-factoring skip-changelog Skips changelog labels Jun 19, 2023
Currently these rely on the root of the repo being in the dvc repo.
@codecov
Copy link

codecov bot commented Jun 19, 2023

Codecov Report

Patch coverage: 100.00% and project coverage change: -0.28 ⚠️

Comparison is base (92dc1f3) 90.52% compared to head (74a91a8) 90.25%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #9628      +/-   ##
==========================================
- Coverage   90.52%   90.25%   -0.28%     
==========================================
  Files         471      471              
  Lines       36058    36058              
  Branches     5185     5185              
==========================================
- Hits        32643    32544      -99     
- Misses       2822     2897      +75     
- Partials      593      617      +24     
Impacted Files Coverage Δ
tests/unit/command/test_add.py 100.00% <100.00%> (ø)
tests/unit/command/test_data_sync.py 100.00% <100.00%> (ø)
tests/unit/command/test_diff.py 100.00% <100.00%> (ø)
tests/unit/command/test_experiments.py 100.00% <100.00%> (ø)
tests/unit/command/test_imp.py 100.00% <100.00%> (ø)
tests/unit/command/test_imp_url.py 100.00% <100.00%> (ø)

... and 25 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@efiop efiop merged commit 1835c8c into iterative:main Jun 19, 2023
@efiop efiop added testing Related to the tests and the testing infrastructure and removed refactoring Factoring and re-factoring labels Jun 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
skip-changelog Skips changelog testing Related to the tests and the testing infrastructure
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant