fix(t-rules): delete-tablewithoutindex

[everleene]

PR Description

This pull request removes the "TableWithoutIndex" rule (T002) from the codebase, configuration, and tests, and introduces a new test fixture to ensure database cleanup after each test. It also makes minor improvements to the code formatting workflow and configuration.

Rule removal and related cleanup:

• Removed the "TableWithoutIndex" rule (T002) from all configuration files (dblinter.cfg, dblinter/default_config.yaml, tests/data/good_config.yaml), the implementation (dblinter/rules/T002/TableWithoutIndex.py), and the associated tests (tests/rules/T002/test_TableWithoutIndex.py). Test assertions in tests/test_main.py that referenced this rule were also updated or removed. [1] [2] [3] [4] [5] [6] [7] [8]

Test infrastructure improvements:

• Added a new cleanup_database fixture in tests/conftest.py to automatically clean up the database after each test, preventing test pollution and ensuring isolation between tests.

Workflow and configuration updates:

• Updated the isort command in GitHub Actions workflows to only check the dblinter/ and tests/ directories, improving performance and relevance. (.github/workflows/build-main.yaml, .github/workflows/build-pr.yaml) [1] [2]
• Added a black configuration to exclude dblinter.cfg from formatting. (pyproject.toml)

Test assertion fixes:

• Adjusted test assertions in tests/test_main.py to reflect the removal of the "TableWithoutIndex" rule, ensuring tests only check for remaining relevant messages. [1] [2] [3] [4]# PR Description

Known limitations / corner cases (if any)

N/A

[CLAassistant]

All committers have signed the CLA.

[vincentmoreaudba]

LGTM

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[pmpetit]

ok