Skip to content

Commit c786d39

Browse files
committed
test(storage): cover ServerConfig.AuthBroker in saveServerSync field canary
AuthBroker (spec 074) is server-edition per-upstream broker config carried in the JSON config (like Shared), not persisted to the BBolt UpstreamRecord. Mark it intentionally excluded in TestSaveServerSyncFieldCoverage so the field-coverage canary passes. Related #588
1 parent 6838354 commit c786d39

1 file changed

Lines changed: 7 additions & 0 deletions

File tree

internal/storage/async_ops_test.go

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -252,6 +252,9 @@ func TestSaveServerSyncFieldCoverage(t *testing.T) {
252252
// (and the custom-origin skip_quarantine guard) survive a restart.
253253
"SourceRegistryID": true,
254254
"SourceRegistryProvenance": true,
255+
// Spec 074: server-edition per-upstream broker config; lives in the JSON
256+
// config (like Shared), not persisted to the BBolt UpstreamRecord.
257+
"AuthBroker": true,
255258
}
256259

257260
// Get all fields from ServerConfig
@@ -287,6 +290,10 @@ func TestSaveServerSyncFieldCoverage(t *testing.T) {
287290
// Spec 032: runtime-only field, not persisted to BBolt
288291
continue
289292
}
293+
if fieldName == "AuthBroker" {
294+
// Spec 074: server-edition JSON-config field, not persisted to BBolt
295+
continue
296+
}
290297
if !upstreamFields[fieldName] {
291298
t.Errorf("Expected field %q in UpstreamRecord but not found", fieldName)
292299
}

0 commit comments

Comments
 (0)