Skip to content

Comments

Test Improvements: Move last tests to new scenarios#10100

Merged
potatoqualitee merged 5 commits intodevelopmentfrom
move_last_tests_to_new_scenarios
Jan 16, 2026
Merged

Test Improvements: Move last tests to new scenarios#10100
potatoqualitee merged 5 commits intodevelopmentfrom
move_last_tests_to_new_scenarios

Conversation

@andreasjordan
Copy link
Collaborator

@andreasjordan andreasjordan commented Jan 16, 2026

  • Changed all remaining usages of instance1 and instance2 to InstanceSingle
  • Split scenario SINGLE into three parts to not hit the 1 hour limit
  • Using version 2016 for SINGLE
  • Using versions 2016 and 2017 for MULTI and COPY
  • Using version 2017 for HADR and RESTART
  • Using no sql server for scenario default as there are only tests that do not need a running instance
  • enabled all tests ("appveyor_disabled" = @())

If needed, I can later test SINGLE against 2008R2 Express to see how many tests work against that version.

If this PR is too big to review, I can split it and only change instance2 to InstanceSingle in the first round.

Situation before this PR (total runtime: 1 hr 32 min):
image

Situation after this PR (total runtime: 1 hr 36 min):
image

@potatoqualitee
Copy link
Member

Thank you 🙏🏼 looks so good!

@potatoqualitee potatoqualitee merged commit dc3e796 into development Jan 16, 2026
15 checks passed
@potatoqualitee potatoqualitee deleted the move_last_tests_to_new_scenarios branch January 16, 2026 14:37
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