Skip to content

acc: drop tests that only assert the terraform removal error - #6955

Merged
denik merged 2 commits into
mainfrom
denik/tf-cleanup-removal-error-tests
Oct 7, 2026
Merged

denik merged 2 commits into
mainfrom
denik/tf-cleanup-removal-error-tests

Conversation

@denik

@denik denik commented Oct 6, 2026 •

Copy link
Copy Markdown
Contributor

Deletes 9 acceptance tests that used to exercise terraform-engine behaviour and now only print the engine removal error. Kept: validate/engine-config-valid, migrate/command/engine-config-terraform (config variant) and deploy/readplan/terraform-error (the single env-var variant). No behaviour change.

This pull request and its description were written by Isaac.

Base automatically changed from denik/engine-env-message to main October 6, 2026 23:11
@eng-dev-ecosystem-bot

eng-dev-ecosystem-bot commented Oct 6, 2026 •

Copy link
Copy Markdown
Collaborator

Integration test report

Commit: 49e004a

Run: 37545638916

Env 🔄​flaky ✅​pass 🙈​skip Time
✅​ aws linux 281 17 6:00
✅​ aws windows 283 15 3:46
✅​ azure linux 280 17 5:38
✅​ azure windows 282 15 3:50
✅​ gcp linux 281 17 7:17
🔄​ gcp windows 2 281 15 4:34
Test Name gcp windows
🔄​ TestFsCpFileToNonExistentDir 🔄​f
🔄​ TestFsCpFileToNonExistentDir/uc-volumes_to_uc-volumes 🔄​f
Top 6 slowest tests (at least 2 minutes):
duration env testname
3:55 aws linux TestAccept
3:51 azure linux TestAccept
3:49 gcp linux TestAccept
3:13 azure windows TestAccept
3:06 aws windows TestAccept
3:04 gcp windows TestAccept

denik and others added 2 commits October 7, 2026 01:14
Co-authored-by: Isaac <no-reply@databricks.com>
Co-authored-by: Isaac <no-reply@databricks.com>
@denik
denik force-pushed the denik/tf-cleanup-removal-error-tests branch from 54c8136 to 49e004a Compare October 6, 2026 23:15
@github-actions github-actions Bot added the DABs DABs related issues label Oct 6, 2026
@denik
denik marked this pull request as ready for review October 7, 2026 06:35
@denik
denik enabled auto-merge October 7, 2026 06:36
@denik
denik added this pull request to the merge queue Oct 7, 2026
Merged via the queue into main with commit d59d953 Oct 7, 2026
27 checks passed
@denik
denik deleted the denik/tf-cleanup-removal-error-tests branch October 7, 2026 07:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

DABs DABs related issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants