Skip to content

Commit 088e3fe

Browse files
committed
fix tests for datafile and events format compat
1 parent 1c83e39 commit 088e3fe

File tree

2 files changed

+7
-10
lines changed

2 files changed

+7
-10
lines changed

Tests/OptimizelyTests-Common/DatafileHandlerTests.swift

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -525,18 +525,19 @@ class DatafileHandlerTests: XCTestCase {
525525

526526
#if os(tvOS)
527527
var url = FileManager.default.urls(for: .cachesDirectory, in: .userDomainMask).first!
528-
url = url.appendingPathComponent(testSDKKey, isDirectory: false)
529-
try! datafileData.write(to: url, options: .atomic)
530528
#else
531529
var url = FileManager.default.urls(for: .documentDirectory, in: .userDomainMask).first!
530+
#endif
532531
url = url.appendingPathComponent(testSDKKey, isDirectory: false)
533532
try! datafileData.write(to: url, options: .atomic)
534-
#endif
535533

536534
// verify that a new datafileHandler can read an existing datafile cache
537535

538536
let datafileFromCache = DefaultDatafileHandler().loadSavedDatafile(sdkKey: testSDKKey)
539537
XCTAssert(datafileFromCache == datafileData, "failed to support old datafile cached data format")
538+
539+
let projectConfig = try! ProjectConfig(datafile: datafileFromCache!)
540+
XCTAssert(projectConfig.project.revision == "241")
540541
}
541542

542543
}

Tests/OptimizelyTests-Common/EventDispatcherTests.swift

Lines changed: 3 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -299,21 +299,17 @@ class EventDispatcherTests: XCTestCase {
299299
let saveFormat = try! JSONEncoder().encode(events)
300300

301301
#if os(tvOS)
302-
let dispatcher = MockEventDispatcher(backingStore: .memory)
303-
let memoryStore: DataStoreMemory<[Data]> = dispatcher.dataStore.dataStore as! DataStoreMemory
304302
var url = FileManager.default.urls(for: .cachesDirectory, in: .userDomainMask).first!
305-
url = url.appendingPathComponent(queueName, isDirectory: false)
306-
try! saveFormat.write(to: url, options: .atomic)
307-
memoryStore.load(forKey: queueName)
308303
#else
309-
let dispatcher = MockEventDispatcher()
310304
var url = FileManager.default.urls(for: .documentDirectory, in: .userDomainMask).first!
305+
#endif
311306
url = url.appendingPathComponent(queueName, isDirectory: false)
312307
try! saveFormat.write(to: url, options: .atomic)
313-
#endif
314308

315309
// verify that a new dataStore can read an existing queue items
316310

311+
let dispatcher = MockEventDispatcher()
312+
317313
XCTAssert(dispatcher.dataStore.count == 2)
318314
dispatcher.flushEvents()
319315
dispatcher.dispatcher.sync {}

0 commit comments

Comments
 (0)