mirror of https://github.com/databricks/cli.git
This commit is contained in:
parent
c5a02ef8fb
commit
eb94cd6717
|
@ -26,9 +26,12 @@ import (
|
||||||
// Returns an error.
|
// Returns an error.
|
||||||
func filerForVolume(ctx context.Context, b *bundle.Bundle) (filer.Filer, string, diag.Diagnostics) {
|
func filerForVolume(ctx context.Context, b *bundle.Bundle) (filer.Filer, string, diag.Diagnostics) {
|
||||||
artifactPath := b.Config.Workspace.ArtifactPath
|
artifactPath := b.Config.Workspace.ArtifactPath
|
||||||
|
|
||||||
w := b.WorkspaceClient()
|
w := b.WorkspaceClient()
|
||||||
|
|
||||||
|
if !strings.HasPrefix(artifactPath, "/Volumes/") {
|
||||||
|
return nil, "", diag.Errorf("expected artifact_path to start with /Volumes/, got %s", artifactPath)
|
||||||
|
}
|
||||||
|
|
||||||
parts := strings.Split(artifactPath, "/")
|
parts := strings.Split(artifactPath, "/")
|
||||||
volumeFormatErr := fmt.Errorf("expected UC volume path to be in the format /Volumes/<catalog>/<schema>/<volume>/..., got %s", artifactPath)
|
volumeFormatErr := fmt.Errorf("expected UC volume path to be in the format /Volumes/<catalog>/<schema>/<volume>/..., got %s", artifactPath)
|
||||||
|
|
||||||
|
|
|
@ -174,6 +174,19 @@ func TestFilerForVolumeWithInvalidVolumePaths(t *testing.T) {
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
func TestFilerForVolumeWithInvalidPrefix(t *testing.T) {
|
||||||
|
b := &bundle.Bundle{
|
||||||
|
Config: config.Root{
|
||||||
|
Workspace: config.Workspace{
|
||||||
|
ArtifactPath: "/Volume/main/my_schema/my_volume",
|
||||||
|
},
|
||||||
|
},
|
||||||
|
}
|
||||||
|
|
||||||
|
_, _, diags := filerForVolume(context.Background(), b)
|
||||||
|
require.EqualError(t, diags.Error(), "expected artifact_path to start with /Volumes/, got /Volume/main/my_schema/my_volume")
|
||||||
|
}
|
||||||
|
|
||||||
func TestFilerForVolumeWithValidlVolumePaths(t *testing.T) {
|
func TestFilerForVolumeWithValidlVolumePaths(t *testing.T) {
|
||||||
validPaths := []string{
|
validPaths := []string{
|
||||||
"/Volumes/main/my_schema/my_volume",
|
"/Volumes/main/my_schema/my_volume",
|
||||||
|
|
Loading…
Reference in New Issue