databricks-cli

Commit Graph

Author	SHA1	Message	Date
Pieter Noordhuis	61b0c59137	Move test helpers from internal to `acc` and `testutil` (#2008 ) ## Changes This change moves fixture helpers to `internal/acc/fixtures.go`. These helpers create an ephemeral path or resource for the duration of a test. Call sites are updated to use `acc.WorkspaceTest()` to construct a workspace-focused test wrapper as needed. This change also moves the `GetNodeTypeID()` function to `testutil`. ## Tests n/a	2024-12-12 21:28:04 +00:00
Ilia Babanov	157877a152	Fix bundle destroy integration test (#1435 ) I've updated the `deploy_then_remove_resources` test template in the previous PR, but didn't notice that it was used in the destroy test too. Now destroy test also checks deletion of jobs	2024-05-16 09:32:55 +00:00
shreyas-goenka	e008c2bd8c	Cleanup remote file path on bundle destroy (#1374 ) ## Changes The sync struct initialization would recreate the deleted `file_path`. This PR moves to not initializing the sync object to delete the snapshot, thus fixing the lingering `file_path` after `bundle destroy`. ## Tests Manually, and a integration test to prevent regression.	2024-04-19 11:48:04 +00:00

Author

SHA1

Message

Date

Pieter Noordhuis

61b0c59137

Move test helpers from internal to `acc` and `testutil` (#2008 )

## Changes

This change moves fixture helpers to `internal/acc/fixtures.go`. These
helpers create an ephemeral path or resource for the duration of a test.
Call sites are updated to use `acc.WorkspaceTest()` to construct a
workspace-focused test wrapper as needed.

This change also moves the `GetNodeTypeID()` function to `testutil`.

## Tests

n/a

2024-12-12 21:28:04 +00:00

Ilia Babanov

157877a152

Fix bundle destroy integration test (#1435 )

I've updated the `deploy_then_remove_resources` test template in the
previous PR, but didn't notice that it was used in the destroy test too.
Now destroy test also checks deletion of jobs

2024-05-16 09:32:55 +00:00

shreyas-goenka

e008c2bd8c

Cleanup remote file path on bundle destroy (#1374 )

## Changes
The sync struct initialization would recreate the deleted `file_path`.
This PR moves to not initializing the sync object to delete the
snapshot, thus fixing the lingering `file_path` after `bundle destroy`.

## Tests
Manually, and a integration test to prevent regression.

2024-04-19 11:48:04 +00:00

3 Commits