apache / apache/datafusion

Renaming Tests Discussion

Open
#1,505 8 comments 0 reactions 0 assignees View on GitHub
enhancement
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

Open the contributing 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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.