Skip to content

Commit f7a7de2

Browse files
committed
fix: add SkipQuarantine to field coverage test exclusion list
SkipQuarantine is a runtime-only config field (Spec 032) that does not need BBolt persistence. Add it to the SaveServerSync field coverage test's exclusion list alongside Shared.
1 parent 7fc5566 commit f7a7de2

1 file changed

Lines changed: 20 additions & 15 deletions

File tree

internal/storage/async_ops_test.go

Lines changed: 20 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -228,21 +228,22 @@ func TestSaveServerSyncPreservesNilFields(t *testing.T) {
228228
func TestSaveServerSyncFieldCoverage(t *testing.T) {
229229
// List of ServerConfig fields that ARE expected to be copied
230230
expectedFields := map[string]bool{
231-
"Name": true,
232-
"URL": true,
233-
"Protocol": true,
234-
"Command": true,
235-
"Args": true,
236-
"WorkingDir": true,
237-
"Env": true,
238-
"Headers": true,
239-
"OAuth": true,
240-
"Enabled": true,
241-
"Quarantined": true,
242-
"Created": true,
243-
"Updated": true, // Updated is set by saveServerSync, not copied
244-
"Isolation": true,
245-
"Shared": true, // Teams-only: persisted in JSON config, not in BBolt
231+
"Name": true,
232+
"URL": true,
233+
"Protocol": true,
234+
"Command": true,
235+
"Args": true,
236+
"WorkingDir": true,
237+
"Env": true,
238+
"Headers": true,
239+
"OAuth": true,
240+
"Enabled": true,
241+
"Quarantined": true,
242+
"Created": true,
243+
"Updated": true, // Updated is set by saveServerSync, not copied
244+
"Isolation": true,
245+
"Shared": true, // Teams-only: persisted in JSON config, not in BBolt
246+
"SkipQuarantine": true, // Spec 032: runtime-only field, not persisted to BBolt
246247
}
247248

248249
// Get all fields from ServerConfig
@@ -274,6 +275,10 @@ func TestSaveServerSyncFieldCoverage(t *testing.T) {
274275
// Teams-only field, persisted in JSON config not BBolt
275276
continue
276277
}
278+
if fieldName == "SkipQuarantine" {
279+
// Spec 032: runtime-only field, not persisted to BBolt
280+
continue
281+
}
277282
if !upstreamFields[fieldName] {
278283
t.Errorf("Expected field %q in UpstreamRecord but not found", fieldName)
279284
}

0 commit comments

Comments
 (0)