Skip to content

Commit

Permalink
fix
Browse files Browse the repository at this point in the history
  • Loading branch information
nkvuong committed Jul 26, 2024
1 parent caa6167 commit 8fb5f9d
Showing 1 changed file with 9 additions and 8 deletions.
17 changes: 9 additions & 8 deletions catalog/resource_sql_table_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -415,7 +415,7 @@ func TestResourceSqlTableUpdateTable(t *testing.T) {
func TestResourceSqlTableUpdateTableAndOwner(t *testing.T) {
d, err := qa.ResourceFixture{
CommandMock: func(commandStr string) common.CommandResults {
assert.Equal(t, "ALTER TABLE `main`.`foo`.`bar` ALTER COLUMN `two` DROP NOT NULL", commandStr)

return common.CommandResults{
ResultType: "",
Data: nil,
Expand Down Expand Up @@ -492,6 +492,7 @@ func TestResourceSqlTableUpdateTableAndOwner(t *testing.T) {
Type: "string",
},
},
Owner: "old group",
},
},
{
Expand All @@ -502,6 +503,13 @@ func TestResourceSqlTableUpdateTableAndOwner(t *testing.T) {
},
Status: 404,
},
{
Method: "PATCH",
Resource: "/api/2.1/unity-catalog/tables/main.foo.bar",
ExpectedRequest: catalog.UpdateTableRequest{
Owner: "new groups",
},
},
}, createClusterForSql...),
Resource: ResourceSqlTable(),
ID: "main.foo.bar",
Expand Down Expand Up @@ -591,13 +599,6 @@ func TestResourceSqlTableUpdateTableClusterKeys(t *testing.T) {
},
Status: 404,
},
{
Method: "PATCH",
Resource: "/api/2.1/unity-catalog/tables/main.foo.bar",
ExpectedRequest: catalog.UpdateTableRequest{
Owner: "new groups",
},
},
}, createClusterForSql...),
Resource: ResourceSqlTable(),
ID: "main.foo.bar",
Expand Down

0 comments on commit 8fb5f9d

Please sign in to comment.