apache / apache/shardingsphere
The integration test scenario empty_rules is confusing
Open
in: test
- Dominant language
- Java
- Stars
- 20.8k
- Forks
- 6.9k
- Avg merge
- 11h 38m
- Merged PRs (30d)
- 326
Description
## Question
Maybe we should rename the scenarios `empty_rules`.
https://github.com/apache/shardingsphere/blob/1c2f666c2f265d67e66b5e831441d0fdf881ff65/test/integration-test/test-suite/src/test/resources/env/scenario/empty_rules/proxy/conf/mysql/config-empty-rules.yaml#L211-L262
Contributor guide
Research direction
Start with test/integration-test/test-suite/src/test/resources/env/scenario/empty_rules/proxy/conf/mysql/config-empty-rules.yaml, especially the referenced lines, and inspect how the empty_rules scenario is named and referenced. The work is done when an agreed clearer scenario name is applied consistently without breaking the integration-test scenario.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- java
- Domain
- databases
- Issue type
- Refactor
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 35/100