Skip to content

Update connection timeout in test configs to match pipeline values#3133

Merged
Aniruddh25 merged 2 commits intodev/anmunde/increaseTimeoutfrom
copilot/sub-pr-3132
Feb 15, 2026
Merged

Update connection timeout in test configs to match pipeline values#3133
Aniruddh25 merged 2 commits intodev/anmunde/increaseTimeoutfrom
copilot/sub-pr-3132

Conversation

Copy link
Contributor

Copilot AI commented Feb 15, 2026

Why make this change?

Addresses review feedback from #3132 requesting consistency between pipeline and test configuration connection timeout values. Pipeline configs were updated to use 30-second timeouts, but test configs still used 5 seconds.

What is this change?

Updated Connection Timeout parameter from 5 to 30 seconds in three test configuration files:

  • src/Service.Tests/dab-config.MsSql.json
  • src/Service.Tests/dab-config.DwSql.json
  • src/Service.Tests/Multidab-config.MsSql.json

This aligns test configurations with pipeline configurations (.pipelines/mssql-pipelines.yml, .pipelines/dwsql-pipelines.yml) to ensure consistent timeout behavior across test and CI environments.

How was this tested?

  • Integration Tests
  • Unit Tests

Configuration changes only - no functional code changes requiring new tests.

Sample Request(s)

N/A - Configuration changes only.


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Co-authored-by: Aniruddh25 <3513779+Aniruddh25@users.noreply.github.com>
Copilot AI changed the title [WIP] Update connection timeout in DW/MSSQL pipelines based on feedback Update connection timeout in test configs to match pipeline values Feb 15, 2026
Copilot AI requested a review from Aniruddh25 February 15, 2026 03:11
@Aniruddh25 Aniruddh25 marked this pull request as ready for review February 15, 2026 03:13
@Aniruddh25 Aniruddh25 merged commit c92b1d4 into dev/anmunde/increaseTimeout Feb 15, 2026
1 check passed
@Aniruddh25 Aniruddh25 deleted the copilot/sub-pr-3132 branch February 15, 2026 03:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants