From 6aba3d0f731539eac1843eab2324f0c9510b7a70 Mon Sep 17 00:00:00 2001 From: mlsmaycon Date: Wed, 12 Aug 2026 14:24:53 +0000 Subject: [PATCH] [management] Bump peer-metadata field-count guard for RemoteJobsAllowed TestSqlStore_SavePeer reflects over every field of PeerSystemMeta via PopulateAll and asserts the count, so that a newly added metadata field forces the author to confirm it round-trips through the store. This PR added Flags.RemoteJobsAllowed (a value bool inside the JSON-serialized Flags), taking the recursive leaf count from 32 to 33. The field does round-trip via the existing Flags JSON serializer, so update the expected count. Fixes the deterministic Management/Unit store failure on all backends. --- 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 ed3419dd7..4b7bcf068 100644 --- a/management/server/store/sql_store_test.go +++ b/management/server/store/sql_store_test.go @@ -573,7 +573,7 @@ func TestSqlStore_SavePeer(t *testing.T) { numOfFields, err := populateFields.PopulateAll(reflectedMetadata) assert.NoError(t, err) - assert.Equal(t, 32, numOfFields) + assert.Equal(t, 33, numOfFields) // save status of non-existing peer peer := &nbpeer.Peer{