Skip to content

test(Spanner): Bulk Migration: add SQL Server integration testing support to SourceDbToSpanner template - #4202

Draft
pratickchokhani wants to merge 3 commits into
GoogleCloudPlatform:mainfrom
pratickchokhani:sqlserver-bulk-migration-integration
Draft

test(Spanner): Bulk Migration: add SQL Server integration testing support to SourceDbToSpanner template#4202
pratickchokhani wants to merge 3 commits into
GoogleCloudPlatform:mainfrom
pratickchokhani:sqlserver-bulk-migration-integration

Conversation

@pratickchokhani

Copy link
Copy Markdown
Contributor

No description provided.

@codecov

codecov Bot commented Sep 1, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 96.39066% with 17 lines in your changes missing coverage. Please review.
✅ Project coverage is 63.62%. Comparing base (ab8f188) to head (a8d56bb).
⚠️ Report is 2 commits behind head on main.

Files with missing lines Patch % Lines
...lectadapter/sqlserver/SqlServerDialectAdapter.java 92.78% 5 Missing and 9 partials ⚠️
...rowmapper/provider/SqlServerJdbcValueMappings.java 97.19% 1 Missing and 2 partials ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##               main    #4202      +/-   ##
============================================
+ Coverage     55.94%   63.62%   +7.68%     
+ Complexity     7484     2878    -4606     
============================================
  Files          1134      566     -568     
  Lines         70175    33133   -37042     
  Branches       8023     3707    -4316     
============================================
- Hits          39258    21081   -18177     
+ Misses        28370    10992   -17378     
+ Partials       2547     1060    -1487     
Components Coverage Δ
spanner-templates 85.00% <96.39%> (+0.29%) ⬆️
spanner-import-export ∅ <ø> (∅)
spanner-live-forward-migration 88.79% <100.00%> (+0.10%) ⬆️
spanner-live-reverse-replication 81.54% <100.00%> (+0.13%) ⬆️
spanner-bulk-migration 89.34% <96.39%> (+0.26%) ⬆️
gcs-spanner-dv 88.05% <100.00%> (+0.13%) ⬆️
Files with missing lines Coverage Δ
...v2/reader/io/jdbc/iowrapper/config/SQLDialect.java 100.00% <100.00%> (ø)
...oud/teleport/v2/source/SourceConnectorFactory.java 100.00% <100.00%> (ø)
...rce/sqlserver/SqlServerSrcToSpSourceConnector.java 100.00% <100.00%> (ø)
...apper/config/defaults/SqlServerConfigDefaults.java 100.00% <100.00%> (ø)
.../v2/spanner/migrations/avro/AvroToValueMapper.java 96.01% <100.00%> (+1.61%) ⬆️
...ort/v2/spanner/migrations/constants/Constants.java 0.00% <ø> (ø)
...r/migrations/source/config/SourceConfigParser.java 79.54% <ø> (ø)
...2/spanner/migrations/source/config/SourceType.java 100.00% <100.00%> (ø)
...rowmapper/provider/SqlServerJdbcValueMappings.java 97.19% <97.19%> (ø)
...lectadapter/sqlserver/SqlServerDialectAdapter.java 92.78% <92.78%> (ø)

... and 594 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@pratickchokhani pratickchokhani added the addition New feature or request label Sep 1, 2026
@pratickchokhani
pratickchokhani force-pushed the sqlserver-bulk-migration-integration branch from 587925f to f8c2a12 Compare September 1, 2026 13:29
@pratickchokhani pratickchokhani changed the title feat(Spanner): Bulk Migration: add SQL Server integration testing support to SourceDbToSpanner template test(Spanner): Bulk Migration: add SQL Server integration testing support to SourceDbToSpanner template Sep 2, 2026
@pratickchokhani
pratickchokhani force-pushed the sqlserver-bulk-migration-integration branch 9 times, most recently from d665459 to b0b17d5 Compare September 4, 2026 10:10
…g connector implementation, dialect adapter, and configuration mappings
@pratickchokhani
pratickchokhani force-pushed the sqlserver-bulk-migration-integration branch 7 times, most recently from 26c3c14 to 859cb55 Compare September 6, 2026 07:56
@pratickchokhani
pratickchokhani force-pushed the sqlserver-bulk-migration-integration branch from 859cb55 to a8d56bb Compare September 6, 2026 13:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

addition New feature or request size/XXL

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant