test(bigtable): Fix TestIntegration_Aggregates (#10789)

* Update integration_test.go * Update integration_test.go * Update integration_test.go * Update inmem.go --------- Co-authored-by: Baha Aiman <[email protected]>
googleapis · Aug 30, 2024 · 633dc86 · 633dc86
1 parent 16317a8
commit 633dc86
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/bigtable/bttest/inmem.go b/bigtable/bttest/inmem.go
@@ -339,7 +339,7 @@ func (s *server) ModifyColumnFamilies(ctx context.Context, req *btapb.ModifyColu
 				return nil, fmt.Errorf("no such family %q", mod.Id)
 			}
 			if cf.valueType != newcf.valueType {
-				return nil, status.Errorf(codes.InvalidArgument, "Immutable fields 'value_type' cannot be updated")
+				return nil, status.Errorf(codes.InvalidArgument, "Immutable fields 'value_type.aggregate_type' cannot be updated")
 			}
 
 			// assume that we ALWAYS want to replace by the new setting

diff --git a/bigtable/integration_test.go b/bigtable/integration_test.go
@@ -347,11 +347,11 @@ func TestIntegration_Aggregates(t *testing.T) {
 		t.Fatalf("Read row mismatch.\n got %#v\nwant %#v", row, wantRow)
 	}
 
-	err = ac.UpdateFamily(ctx, tableName, family, Family{ValueType: Int64Type{}})
+	err = ac.UpdateFamily(ctx, tableName, family, Family{ValueType: StringType{}})
 	if err == nil {
 		t.Fatalf("Expected UpdateFamily to fail, but it didn't")
 	}
-	wantError := "Immutable fields 'value_type' cannot be updated"
+	wantError := "Immutable fields 'value_type.aggregate_type' cannot be updated"
 	if !strings.Contains(err.Error(), wantError) {
 		t.Errorf("Wrong error. Expected to containt %q but was %v", wantError, err)
 	}