Skip to content

Commit bce3f08

Browse files
committed
Fix more test errors
1 parent 8910236 commit bce3f08

File tree

2 files changed

+15
-12
lines changed

2 files changed

+15
-12
lines changed

FirebaseCore/Internal/Tests/Integration/HeartbeatLoggingIntegrationTests.swift

+1-3
Original file line numberDiff line numberDiff line change
@@ -16,9 +16,6 @@
1616
import XCTest
1717

1818
class HeartbeatLoggingIntegrationTests: XCTestCase {
19-
// 2021-11-01 @ 00:00:00 (EST)
20-
let date = Date(timeIntervalSince1970: 1_635_739_200)
21-
2219
override func setUpWithError() throws {
2320
try HeartbeatLoggingTestUtils.removeUnderlyingHeartbeatStorageContainers()
2421
}
@@ -235,6 +232,7 @@ class HeartbeatLoggingIntegrationTests: XCTestCase {
235232

236233
func testLogRepeatedly_WithoutFlushing_LimitsOnWrite() throws {
237234
// Given
235+
let date = Date(timeIntervalSince1970: 1_635_739_200) // 2021-11-01 @ 00:00:00 (EST)
238236
let testDate = AdjustableDate(date: date)
239237
let heartbeatController = HeartbeatController(id: #function, dateProvider: { testDate.date })
240238

FirebaseCore/Internal/Tests/Unit/HeartbeatControllerTests.swift

+14-9
Original file line numberDiff line numberDiff line change
@@ -16,9 +16,6 @@
1616
import XCTest
1717

1818
class HeartbeatControllerTests: XCTestCase {
19-
// 2021-11-01 @ 00:00:00 (EST)
20-
let date = Date(timeIntervalSince1970: 1_635_739_200)
21-
2219
func testFlush_WhenEmpty_ReturnsEmptyPayload() throws {
2320
// Given
2421
let controller = HeartbeatController(storage: HeartbeatStorageFake())
@@ -28,9 +25,10 @@ class HeartbeatControllerTests: XCTestCase {
2825

2926
func testLogAndFlush() throws {
3027
// Given
28+
let date = Date(timeIntervalSince1970: 1_635_739_200) // 2021-11-01 @ 00:00:00 (EST)
3129
let controller = HeartbeatController(
3230
storage: HeartbeatStorageFake(),
33-
dateProvider: { self.date }
31+
dateProvider: { date }
3432
)
3533

3634
assertHeartbeatControllerFlushesEmptyPayload(controller)
@@ -58,11 +56,12 @@ class HeartbeatControllerTests: XCTestCase {
5856
assertHeartbeatControllerFlushesEmptyPayload(controller)
5957
}
6058

61-
func testLogAndFlushAsync() throws {
59+
@MainActor func testLogAndFlushAsync() throws {
6260
// Given
61+
let date = Date(timeIntervalSince1970: 1_635_739_200) // 2021-11-01 @ 00:00:00 (EST)
6362
let controller = HeartbeatController(
6463
storage: HeartbeatStorageFake(),
65-
dateProvider: { self.date }
64+
dateProvider: { date }
6665
)
6766
let expectation = expectation(description: #function)
6867

@@ -99,6 +98,7 @@ class HeartbeatControllerTests: XCTestCase {
9998

10099
func testLogAtEndOfTimePeriodAndAcceptAtStartOfNextOne() throws {
101100
// Given
101+
let date = Date(timeIntervalSince1970: 1_635_739_200) // 2021-11-01 @ 00:00:00 (EST)
102102
let testDate = AdjustableDate(date: date)
103103

104104
let controller = HeartbeatController(
@@ -148,9 +148,10 @@ class HeartbeatControllerTests: XCTestCase {
148148

149149
func testDoNotLogMoreThanOnceInACalendarDay() throws {
150150
// Given
151+
let date = Date(timeIntervalSince1970: 1_635_739_200) // 2021-11-01 @ 00:00:00 (EST)
151152
let controller = HeartbeatController(
152153
storage: HeartbeatStorageFake(),
153-
dateProvider: { self.date }
154+
dateProvider: { date }
154155
)
155156

156157
// When
@@ -178,9 +179,10 @@ class HeartbeatControllerTests: XCTestCase {
178179

179180
func testDoNotLogMoreThanOnceInACalendarDay_AfterFlushing() throws {
180181
// Given
182+
let date = Date(timeIntervalSince1970: 1_635_739_200) // 2021-11-01 @ 00:00:00 (EST)
181183
let controller = HeartbeatController(
182184
storage: HeartbeatStorageFake(),
183-
dateProvider: { self.date }
185+
dateProvider: { date }
184186
)
185187

186188
// When
@@ -308,6 +310,7 @@ class HeartbeatControllerTests: XCTestCase {
308310

309311
func testLoggingDependsOnDateNotUserAgent() throws {
310312
// Given
313+
let date = Date(timeIntervalSince1970: 1_635_739_200) // 2021-11-01 @ 00:00:00 (EST)
311314
let testDate = AdjustableDate(date: date)
312315
let heartbeatController = HeartbeatController(
313316
storage: HeartbeatStorageFake(),
@@ -355,6 +358,7 @@ class HeartbeatControllerTests: XCTestCase {
355358

356359
func testFlushHeartbeatFromToday_WhenTodayHasAHeartbeat_ReturnsPayloadWithOnlyTodaysHeartbeat() throws {
357360
// Given
361+
let date = Date(timeIntervalSince1970: 1_635_739_200) // 2021-11-01 @ 00:00:00 (EST)
358362
let yesterdaysDate = date.addingTimeInterval(-1 * 60 * 60 * 24)
359363
let todaysDate = date
360364
let tomorrowsDate = date.addingTimeInterval(60 * 60 * 24)
@@ -416,7 +420,8 @@ class HeartbeatControllerTests: XCTestCase {
416420

417421
func testFlushHeartbeatFromToday_WhenTodayDoesNotHaveAHeartbeat_ReturnsEmptyPayload() throws {
418422
// Given
419-
let heartbeatController = HeartbeatController(id: #function, dateProvider: { self.date })
423+
let date = Date(timeIntervalSince1970: 1_635_739_200) // 2021-11-01 @ 00:00:00 (EST)
424+
let heartbeatController = HeartbeatController(id: #function, dateProvider: { date })
420425
// When
421426
heartbeatController.flushHeartbeatFromToday()
422427
// Then

0 commit comments

Comments
 (0)