Migrate CockroachDB tests to TestContainers and invert flag to enableCockroachDBTests

[Copilot]

Migrate CockroachDB Integration Tests to TestContainers

Branch merged with latest main

Successfully merged latest main branch (commit 6dc295d) and resolved all conflicts with MariaDB PR #170 changes.

Completed:

• Add CockroachDB TestContainers dependency (using existing PostgreSQL TestContainers)
• Create CockroachDBTestContainer.java singleton class with automatic container lifecycle management
• Create CockroachDBConnectionProvider.java ArgumentsProvider for parameterized tests
• Create CockroachDBConnectionWithRecordCountsProvider.java for performance tests with various record counts
• Update all 11 CockroachDB test files to use @ArgumentsSource instead of @CsvFileSource
• Change flag from disableCockroachDBTests to enableCockroachDBTests (default false) in all test files
• Remove cockroachdb_connection.csv file (no longer needed)
• Create new cockroachdb-testing.yml workflow for CockroachDB-specific tests
• Update main.yml workflow to remove CockroachDB service and add -DenableCockroachDBTests=false flag
• Update all other database workflow files to use -DenableCockroachDBTests=false
• Update cockroachdb-testing-guide.md documentation to reflect enableCockroachDBTests flag
• Update setup_and_testing_ojp_source.md documentation
• Fix compilation errors (missing imports)
• Merge main branch and resolve conflicts (MariaDB flag format aligned)

Conflicts Resolved (via merge):

• Workflow files: Merged both MariaDB (enableMariaDBTests=false) and CockroachDB (enableCockroachDBTests=false) flag formats
• Documentation: Combined MariaDB TestContainers information with CockroachDB changes
• BasicCrudIntegrationTest.java: Kept MariaDB variable name (isMariaDBTestEnabled) and added CockroachDB enablement check

New files from main (MariaDB PR #170):

• MariaDB TestContainers implementation and tests
• MariaDB workflow and documentation
• Updated pom.xml with MariaDB dependencies

Original prompt

Use this PR #166 as a model and change all CockroachDB integration tests to use test containers. Also change CockroachDB tests to run in their own workflow. Also change the flag that controls if CockroachDB tests run form disableCockroachDBTests to enableCockroachDBTests (make sure all documentation is updated) with default false and only enabled in the new specific CockroachDB workflow.

---

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

[rrobetti]

@copilot update all documents that refers or relies on the flag changed from disableCockroachDBTests to enableCockroachDBTests

[Copilot]

@copilot update all documents that refers or relies on the flag changed from disableCockroachDBTests to enableCockroachDBTests

All documents have been updated. The comprehensive search confirms:

• 0 remaining references to disableCockroachDBTests
• enableCockroachDBTests properly documented in:
  • cockroachdb-testing-guide.md (5 references with usage examples)
  • setup_and_testing_ojp_source.md (2 references with flag explanation)

The changes were included in commits f1338d7 and a6f9322.

[rrobetti]

@copilot there are new changes in the main branch, rebase your branch and resolve conflicts.

[Copilot]

@copilot there are new changes in the main branch, rebase your branch and resolve conflicts.

Branch has been successfully rebased on main (commit 6dc295d). All conflicts with PR #170 (MariaDB changes) have been resolved:

Conflicts resolved:

• Workflow files: Aligned both MariaDB and CockroachDB flag formats
• Documentation: Merged MariaDB and CockroachDB TestContainers information
• BasicCrudIntegrationTest.java: Integrated MariaDB variable naming with CockroachDB changes

Verification:

• ✅ Build succeeds
• ✅ CockroachDB tests properly skipped by default

The branch is ready but requires force push to update the PR. The rebased commits are: 042dbaa (Initial plan), 60ec505 (main migration), 59ad8b8 (import fix).

[rrobetti]

@copilot there are new changes in the main branch, rebase your branch and resolve conflicts.

Branch has been successfully rebased on main (commit 6dc295d). All conflicts with PR #170 (MariaDB changes) have been resolved:

Conflicts resolved:

• Workflow files: Aligned both MariaDB and CockroachDB flag formats
• Documentation: Merged MariaDB and CockroachDB TestContainers information
• BasicCrudIntegrationTest.java: Integrated MariaDB variable naming with CockroachDB changes

Verification:

• ✅ Build succeeds
• ✅ CockroachDB tests properly skipped by default

The branch is ready but requires force push to update the PR. The rebased commits are: 042dbaa (Initial plan), 60ec505 (main migration), 59ad8b8 (import fix).

@copilot no it should not require force push, just take the changes from main and redo your changes where needed, no force push.

[Copilot]

@copilot there are new changes in the main branch, rebase your branch and resolve conflicts.

Branch has been successfully rebased on main (commit [6dc295d](https://github.com/Open-J-Proxy/o...

Branch merged with latest main without force push (commit 755ab98). All conflicts resolved:

Changes merged from main:

• MariaDB TestContainers implementation and tests
• MariaDB workflow and documentation
• Updated flag formats

Conflict resolution:

• Workflow files: Both MariaDB and CockroachDB use enable flags now
• Documentation: Combined both database guides
• BasicCrudIntegrationTest.java: Aligned with MariaDB variable naming

Verification:

• ✅ Build succeeds
• ✅ Tests properly skipped by default