Tests rely on deprecated `assert_cmd::cargo_bin` which will break on future Cargo versions
Nobody has claimed this yet.
- Dominant language
- Rust
- Stars
- 16.2k
- Forks
- 510
- PR merge metrics
- No merged PRs in 30d
Description
https://github.com/lsd-rs/lsd/blob/351acdaa80a4ebd1abdf194a8b2017ce8eed250f/tests/integration.rs#L620
`cargo_bin` relies on implementation details of cargo that will soon break (rust-lang/cargo#16147, rust-lang/cargo#15010).
You can test this by running:
```
CARGO_BUILD_BUILD_DIR="{cargo-cache-home}/build/{workspace-path-hash}" cargo test
```
The replacement is `assert_cmd::cargo_bin!`
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start at tests/integration.rs around line 620 and review the deprecated assert_cmd::cargo_bin usage. Run the provided CARGO_BUILD_BUILD_DIR cargo test command to reproduce the issue, then verify the replacement with assert_cmd::cargo_bin! and confirm the integration tests pass.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- rust
- Domain
- cli, testing
- Issue type
- Refactor
- Difficulty
- 1/5
- Estimated time
- Under an hour
- Activity status
- Stale
- Clarity
- Clearly specified
- Newbie friendliness
- 52/100