apache / apache/gravitino

[Improvement] Misleading `IT` suffix in test class

Open
#2,599 2 comments 0 reactions 0 assignees View on GitHub
improvement
Dominant language
Java
Stars
3.2k
Forks
935
Avg merge
1d 15h
Merged PRs (30d)
315

Description

### What would you like to be improved?

Some test class has the `IT` suffix, e.g. `XXXTableOperationsIT`, `TestMysqlDatabaseOperations`, `TestPostgreSqlSchemaOperations`. But integration test means end-to-end testing.
Should we remove the misleading suffix?

### How should we improve?

Maybe we can revisit all the test classes in test folder to see which of them are end to end testing and remove the suffix from all other classes.

Contributor guide

Open the contributing guide

Research direction

Start by inventorying the test folder, including XXXTableOperationsIT, TestMysqlDatabaseOperations, and TestPostgreSqlSchemaOperations, and determine which classes are end-to-end tests. Define the naming rule before making changes; done means non-end-to-end classes no longer carry the misleading IT suffix and the renamed tests remain runnable.

Written by the indexing model from the issue text.

Assessment

Tech stack
java
Domain
testing
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
30/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.