Move integration tests to integration package (#2009)

## Changes

Objectives:
* A dedicated directory for integration tests
* It is not picked up by `go test ./...`
* No need for a `TestAcc` test name prefix
* More granular packages to improve test selection (future)

The tree structure generally mirrors the source code tree structure.

Requirements for new files in this directory:
* Every package **must** be named after its directory with `_test` appended
* Requiring a different package name for integration tests avoids
aliasing with the main package.
* Every integration test package **must** include a `main_test.go` file.

These requirements are enforced by a unit test in the `integration` package.

## Tests

Integration tests pass.

The total run time regresses by about 10%. A follow-up change that
increases the degree of test parallelism will address this.

Author: pietern

Makefile

@@ -37,6 +37,6 @@ vendor:
 	@go mod vendor
 
 integration:
-	gotestsum --format github-actions --rerun-fails --jsonfile output.json --packages "./internal/..." -- -run "TestAcc.*" -parallel 4 -timeout=2h
+	gotestsum --format github-actions --rerun-fails --jsonfile output.json --packages "./integration/..." -- -parallel 4 -timeout=2h
 
 .PHONY: fmt lint lintcheck test testonly coverage build snapshot vendor integration

integration/README.md

@@ -1,4 +1,4 @@
-package internal
+package assumptions_test
 
 import (
 	"encoding/base64"

internal/dashboard_assumptions_test.go …ssumptions/dashboard_assumptions_test.gointernal/dashboard_assumptions_test.go renamed to integration/assumptions/dashboard_assumptions_test.go internal/dashboard_assumptions_test.go renamed to integration/assumptions/dashboard_assumptions_test.go

@@ -0,0 +1,13 @@
+package assumptions_test
+
+import (
+	"testing"
+
+	"github.com/databricks/cli/integration/internal"
+)
+
+// TestMain is the entrypoint executed by the test runner.
+// See [internal.Main] for prerequisites for running integration tests.
+func TestMain(m *testing.M) {
+	internal.Main(m)
+}

integration/assumptions/main_test.go

@@ -1,4 +1,4 @@
-package bundle
+package bundle_test
 
 import (
 	"fmt"

internal/bundle/artifacts_test.go integration/bundle/artifacts_test.gointernal/bundle/artifacts_test.go renamed to integration/bundle/artifacts_test.go

@@ -1,4 +1,4 @@
-package bundle
+package bundle_test
 
 import (
 	"os"

internal/bundle/basic_test.go integration/bundle/basic_test.gointernal/bundle/basic_test.go renamed to integration/bundle/basic_test.go

@@ -1,4 +1,4 @@
-package bundle
+package bundle_test
 
 import (
 	"fmt"