From 71110f09acc45fd72054d171d98366732f7fe294 Mon Sep 17 00:00:00 2001 From: Pete Schwamb Date: Mon, 12 Oct 2026 00:16:33 -0500 Subject: [PATCH] SettingsManagerTests: only wait for the settings manager's notification testChangingMaxBasalUpdatesLoopData fulfilled its expectation on any LoopDataUpdated notification. The test host is a running Loop app, whose own LoopDataManager posts that notification too, so the expectation was fulfilled twice and XCTest aborted the host, failing the CI run. --- LoopTests/Managers/SettingsManagerTests.swift | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/LoopTests/Managers/SettingsManagerTests.swift b/LoopTests/Managers/SettingsManagerTests.swift index a4768bcd28..45611191a8 100644 --- a/LoopTests/Managers/SettingsManagerTests.swift +++ b/LoopTests/Managers/SettingsManagerTests.swift @@ -21,7 +21,7 @@ final class SettingsManagerTests: XCTestCase { let settingsManager = SettingsManager(cacheStore: persistenceController, expireAfter: .days(1), alertMuter: AlertMuter()) let exp = expectation(description: #function) - let observer = NotificationCenter.default.addObserver(forName: .LoopDataUpdated, object: nil, queue: nil) { _ in + let observer = NotificationCenter.default.addObserver(forName: .LoopDataUpdated, object: settingsManager, queue: nil) { _ in exp.fulfill() }