Renaming Tests Discussion
- Dominant language
- Rust
- Stars
- 9.3k
- Forks
- 2.4k
- Avg merge
- 3d 7h
- Merged PRs (30d)
- 344
Description
**Is your feature request related to a problem or challenge? Please describe what you are trying to do.**
I am starting the discussion on how we want to rename some of the tests post the refactoring of the tests in #1491
**Describe the solution you'd like**
Some of the tests have names that don't necessarily have anything to do with what they are testing. Most universally the tests that begin with `csv_*` come to mind. Does that prefix have any value? I think we should just universally remove them. Maybe another suggestion maybe to instead prefix tests with what functionality they are meant to actually test?
**Describe alternatives you've considered**
Not renaming them?
**Additional context**
Add any other context or screenshots about the feature request here.
Contributor guide
Research direction
Start by reviewing the test refactoring in #1491 and inventorying the tests with the `csv_*` prefix. The issue is a naming discussion rather than a defined edit; done requires an agreed naming convention and updated test names that clearly describe the functionality they cover.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- rust
- Domain
- testing
- Issue type
- Refactor
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 25/100