babelfish-for-postgresql / babelfish_extensions

Babelfish for PostgreSQL provides the capability for PostgreSQL to work with applications written for Microsoft SQL Server. Babelfish understands the SQL Server wire-protocol and T-SQL, the Microsoft SQL Server query and procedural language, so you don’t have to switch database drivers or rewrite all of your application queries.
https://babelfishpg.org/
Apache License 2.0
274 stars 93 forks source link

jTDS: support running JDBC tests with jTDS driver #2861

Open staticlibs opened 1 month ago

staticlibs commented 1 month ago

Description

This PR is a part of a change originally implemented in #2320.

This change to JDBC test harness introduces the jtds_jdbc_schedule list file. To run the test from this new file instead of jdbc_schedule file it is necessary to set useJTDSInsteadOfMSSQLJDBC environment variable to true:

useJTDSInsteadOfMSSQLJDBC=true mvn test

Changes from #2860 PR are required to run jTDS tests successfully.

For jtds_jdbc_schedule file the intention is to use original tests unmodified where possible. Test for which modifications were required (mostly due to data types and error handling differences) are copied with jtds- prefix.

Github workfow file jdbc-tests-with-jtds.yml is included, but tests there are failing until #2860 is integrated. It is based on jdbc-tests-with-parallel-query.yml workflow file.

Issues Resolved

2137

Test Scenarios Covered

The following tests from the original list are included:

Check List

By submitting this pull request, I confirm that my contribution is under the terms of the Apache 2.0 and PostgreSQL licenses, and grant any person obtaining a copy of the contribution permission to relicense all or a portion of my contribution to the PostgreSQL License solely to contribute all or a portion of my contribution to the PostgreSQL open source project.

For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Signed-off-by: Alex Kasko alex@staticlibs.net

coveralls commented 1 month ago

Pull Request Test Coverage Report for Build 10410968113

Details


Totals Coverage Status
Change from base Build 10389490935: 0.0%
Covered Lines: 44427
Relevant Lines: 59998

💛 - Coveralls