Skip to content

Commit

Permalink
Fix failing fs mkdir test on azure (#627)
Browse files Browse the repository at this point in the history
Regex check was missing a "." character and adding it in fixes the test.
The test now passes on all three cloud providers
  • Loading branch information
shreyas-goenka authored Jul 31, 2023
1 parent 433f401 commit 5a61771
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion internal/fs_mkdir_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -112,6 +112,6 @@ func TestAccFsMkdirWhenFileExistsAtPath(t *testing.T) {
// assert run fails
_, _, err = RequireErrorRun(t, "fs", "mkdir", "dbfs:"+path.Join(tmpDir, "hello"))
// Different cloud providers return different errors.
regex := regexp.MustCompile(`^Path is a file: .*$|^Cannot create directory .* because .* is an existing file$|^mkdirs\(hadoopPath: .*, permission: rwxrwxrwx\): failed$`)
regex := regexp.MustCompile(`^Path is a file: .*$|^Cannot create directory .* because .* is an existing file\.$|^mkdirs\(hadoopPath: .*, permission: rwxrwxrwx\): failed$`)
assert.Regexp(t, regex, err.Error())
}

0 comments on commit 5a61771

Please sign in to comment.