From a676771e8d7d50e377c4dc5e81ff8651efea7f07 Mon Sep 17 00:00:00 2001 From: riccardom Date: Fri, 25 Sep 2026 16:13:38 +0200 Subject: [PATCH] [management] Count the new setting in the Settings round-trip guard Test_AccountSettings_SaveAndRetrieve asserts how many exported fields types.Settings has, so that adding one forces whoever added it to confirm it survives a save and reload. DebugBundleUploadURL made it 28 and the assertion still said 27. With the count corrected the test populates the new field too and passes on sqlite, postgres and mysql, which is the confirmation the guard exists to get. --- management/server/store/sql_store_test.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/management/server/store/sql_store_test.go b/management/server/store/sql_store_test.go index 4b7bcf068..44c28a2f4 100644 --- a/management/server/store/sql_store_test.go +++ b/management/server/store/sql_store_test.go @@ -331,7 +331,7 @@ func Test_AccountSettings_SaveAndRetrieve(t *testing.T) { settings := types.Settings{} numOfExportedFields, err := populateFields.PopulateAll(reflect.ValueOf(&settings).Elem()) assert.NoError(t, err) - assert.Equal(t, 27, numOfExportedFields) + assert.Equal(t, 28, numOfExportedFields) account.Settings = &settings err = store.SaveAccount(context.Background(), account)