Skip to content

Commit

Permalink
test(bigtable): Adding isolation to app profile (#8982)
Browse files Browse the repository at this point in the history
  • Loading branch information
bhshkh authored Nov 9, 2023
1 parent 5d26f0d commit a0b64f8
Showing 1 changed file with 17 additions and 5 deletions.
22 changes: 17 additions & 5 deletions bigtable/integration_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -2821,6 +2821,13 @@ func TestIntegration_InstanceAdminClient_AppProfile(t *testing.T) {
t.Fatalf("Get app profile: %v", err)
}

defer func() {
err = iAdminClient.DeleteAppProfile(ctx, adminClient.instance, profileID)
if err != nil {
t.Fatalf("Delete app profile: %v", err)
}
}()

if !proto.Equal(createdProfile, gotProfile) {
t.Fatalf("created profile: %s, got profile: %s", createdProfile.Name, gotProfile.Name)
}
Expand Down Expand Up @@ -2881,6 +2888,11 @@ func TestIntegration_InstanceAdminClient_AppProfile(t *testing.T) {
Description: "",
RoutingPolicy: gotProfile.RoutingPolicy,
Etag: gotProfile.Etag,
Isolation: &btapb.AppProfile_StandardIsolation_{
StandardIsolation: &btapb.AppProfile_StandardIsolation{
Priority: btapb.AppProfile_PRIORITY_HIGH,
},
},
},
},
{
Expand All @@ -2898,6 +2910,11 @@ func TestIntegration_InstanceAdminClient_AppProfile(t *testing.T) {
ClusterId: testEnv.Config().Cluster,
},
},
Isolation: &btapb.AppProfile_StandardIsolation_{
StandardIsolation: &btapb.AppProfile_StandardIsolation{
Priority: btapb.AppProfile_PRIORITY_HIGH,
},
},
},
},
} {
Expand All @@ -2923,11 +2940,6 @@ func TestIntegration_InstanceAdminClient_AppProfile(t *testing.T) {
})

}

err = iAdminClient.DeleteAppProfile(ctx, adminClient.instance, profileID)
if err != nil {
t.Fatalf("Delete app profile: %v", err)
}
}

func TestIntegration_InstanceUpdate(t *testing.T) {
Expand Down

0 comments on commit a0b64f8

Please sign in to comment.