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
Pieter Noordhuis	dd3b7ec450	Define and use `testutil.TestingT` interface (#2003 ) ## Changes Using an interface instead of a concrete type means we can pass `testing.T` directly or any wrapper type that implements a superset of this interface. It prepares for more broad use of `acc.WorkspaceT`, which enhances the testing object with helper functions for using a Databricks workspace. This eliminates the need to dereference a `testing.T` field on a wrapper type. ## Tests n/a	2024-12-12 14:42:15 +00:00
Andrew Nester	fab3e8f168	Added integration test to deploy bundle to /Shared root path (#1914 ) ## Changes Added integration test to deploy bundle to /Shared root path ## Tests ``` --- PASS: TestAccDeployBasicToSharedWorkspace (24.58s) PASS coverage: 31.2% of statements in ./... ok github.com/databricks/cli/internal/bundle 25.572s coverage: 31.2% of statements in ./... ``` --------- Co-authored-by: shreyas-goenka <88374338+shreyas-goenka@users.noreply.github.com>	2024-11-20 12:20:39 +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

Pieter Noordhuis

dd3b7ec450

Define and use `testutil.TestingT` interface (#2003 )

## Changes

Using an interface instead of a concrete type means we can pass
`*testing.T` directly or any wrapper type that implements a superset of
this interface. It prepares for more broad use of `acc.WorkspaceT`,
which enhances the testing object with helper functions for using a
Databricks workspace.

This eliminates the need to dereference a `*testing.T` field on a
wrapper type.

## Tests

n/a

2024-12-12 14:42:15 +00:00

Andrew Nester

fab3e8f168

Added integration test to deploy bundle to /Shared root path (#1914 )

## Changes
Added integration test to deploy bundle to /Shared root path

## Tests
```
--- PASS: TestAccDeployBasicToSharedWorkspace (24.58s)
PASS
coverage: 31.2% of statements in ./...
ok  	github.com/databricks/cli/internal/bundle	25.572s	coverage: 31.2% of statements in ./...
```

---------

Co-authored-by: shreyas-goenka <88374338+shreyas-goenka@users.noreply.github.com>

2024-11-20 12:20:39 +00:00

3 Commits