apache / apache/seatunnel

[Feature][connector-jdbc] Optimize unit testing of jdbc connector code

Open
#10,210 3 comments 1 reaction 0 assignees View on GitHub
good first issue help wanted
Dominant language
Java
Stars
9.7k
Forks
2.4k
Avg merge
3d 9h
Merged PRs (30d)
204

Description

### Search before asking

- [x] I had searched in the [feature](https://github.com/apache/seatunnel/issues?q=is%3Aissue+label%3A%22Feature%22) and found no similar feature requirement.

### Description

### Problem
1. At present, the unit tests of connector-jdbc code basically rely on the local environment and will be turned off during the actual running of ci, which will lead to functional risks. We hope to introduce docker containers and run unit tests in the actual ci process.
2. The unit test coverage of connector-jdbc code is relatively low, such as the jdbcCatalog test and jdbcDialect test. Additionally, there is a lack of boundary condition test cases. We hope to add relevant test cases.
3. After completing points 1 and 2, you can transfer the unit tests in the jdbc-e2e module to connector-jdbc.

### Implementation scope
| type | status |Assignees |
| -------------- | ---- | ---- |
| mysql | | |
| oracle | | |
| pgsql | | |
| sqlserver | | |
| dameng | | |
| db2 | | |
| kingbase | | |
### Usage Scenario

_No response_

### Related issues

_No response_

### Are you willing to submit a PR?

- [ ] Yes I am willing to submit a PR!

### Code of Conduct

- [x] I agree to follow this project's [Code of Conduct](https://www.apache.org/foundation/policies/conduct)

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by reviewing the connector-jdbc unit tests, especially jdbcCatalog and jdbcDialect, along with the jdbc-e2e module and the CI configuration. Check how tests currently depend on the local environment and which database types are covered. Done means boundary cases are added, Docker-backed tests run in CI, and applicable jdbc-e2e tests are moved into connector-jdbc.

Written by the indexing model from the issue text.

Assessment

Tech stack
docker, java
Domain
ci-cd, databases, testing
Issue type
Feature
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.