nextest icon indicating copy to clipboard operation
nextest copied to clipboard

[WIP] option for store dir relative to the target dir

Open sunshowers opened this issue 10 months ago • 5 comments

Still to figure out:

  • [ ] Write tests for parsing configuration.
  • [ ] Write tests which set configuration and ensure that output gets written to the right spot.
  • [ ] How should remapped archive directories be handled? Archive target directories are typically temporary directories which will be deleted at the end of the run. Using that as the store directory seems wrong. In that case, we may wish to use another location instead.

sunshowers avatar Feb 16 '25 05:02 sunshowers

Codecov Report

:x: Patch coverage is 90.62500% with 12 lines in your changes missing coverage. Please review. :white_check_mark: Project coverage is 80.20%. Comparing base (44e109c) to head (75d27f4).

Files with missing lines Patch % Lines
nextest-runner/src/config/elements/store.rs 89.36% 5 Missing :warning:
nextest-runner/src/reporter/aggregator/junit.rs 68.75% 5 Missing :warning:
cargo-nextest/src/errors.rs 0.00% 2 Missing :warning:
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2154      +/-   ##
==========================================
+ Coverage   80.17%   80.20%   +0.03%     
==========================================
  Files         113      114       +1     
  Lines       26312    26362      +50     
==========================================
+ Hits        21096    21144      +48     
- Misses       5216     5218       +2     

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

:rocket: New features to boost your workflow:
  • :snowflake: Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

codecov[bot] avatar Feb 16 '25 05:02 codecov[bot]

Static BadgeStatic BadgeStatic BadgeStatic Badge

View Full Report ↗︎Docs

trunk-io[bot] avatar Feb 16 '25 06:02 trunk-io[bot]

Hi! I've been writing some tests for this in https://github.com/eduardorittner/nextest/tree/store-dir-relative. I'm not sure how to best integrate these changes? Open a PR from my branch to upstream/store-dir-relative? You just copy the commits over?

Also, regarding your third point, do you mean that when store.dir's full path is a temporary directory we should choose another directory? But how can nextest detect when it's a temporary directory? Is there a mechanism for this?

eduardorittner avatar Dec 01 '25 12:12 eduardorittner

Thanks for picking this up. Let me rebase on to main and then you can submit your work as dependent on it.

But how can nextest detect when it's a temporary directory?

Only concerned about temp dirs nextest itself creates in this context, which can be tracked internally.

sunshowers avatar Dec 01 '25 20:12 sunshowers

Rebased this on main.

sunshowers avatar Dec 01 '25 21:12 sunshowers