From 6a4f3fb11e672ecae4eb7f30f1bb33542cafa656 Mon Sep 17 00:00:00 2001 From: Tim Ekl Date: Tue, 31 Dec 2019 19:08:29 -0600 Subject: [PATCH] Fix build errors in sync stream operations tests Prefix two throwing calls with `try`. Label the enclosing test functions with `throws` so the XCTest machinery helps us with error reporting. --- .../Feedly/FeedlySyncStreamContentsOperationTests.swift | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/Frameworks/Account/AccountTests/Feedly/FeedlySyncStreamContentsOperationTests.swift b/Frameworks/Account/AccountTests/Feedly/FeedlySyncStreamContentsOperationTests.swift index 52d03bef4..6a3f83d91 100644 --- a/Frameworks/Account/AccountTests/Feedly/FeedlySyncStreamContentsOperationTests.swift +++ b/Frameworks/Account/AccountTests/Feedly/FeedlySyncStreamContentsOperationTests.swift @@ -26,7 +26,7 @@ class FeedlySyncStreamContentsOperationTests: XCTestCase { super.tearDown() } - func testIngestsOnePageSuccess() { + func testIngestsOnePageSuccess() throws { let service = TestGetStreamContentsService() let resource = FeedlyCategoryResourceId(id: "user/1234/category/5678") let newerThan: Date? = Date(timeIntervalSinceReferenceDate: 0) @@ -56,7 +56,7 @@ class FeedlySyncStreamContentsOperationTests: XCTestCase { waitForExpectations(timeout: 2) let expectedArticleIds = Set(items.map { $0.id }) - let expectedArticles = account.fetchArticles(.articleIDs(expectedArticleIds)) + let expectedArticles = try account.fetchArticles(.articleIDs(expectedArticleIds)) XCTAssertEqual(expectedArticles.count, expectedArticleIds.count, "Did not fetch all the articles.") } @@ -90,7 +90,7 @@ class FeedlySyncStreamContentsOperationTests: XCTestCase { waitForExpectations(timeout: 2) } - func testIngestsManyPagesSuccess() { + func testIngestsManyPagesSuccess() throws { let service = TestGetPagedStreamContentsService() let resource = FeedlyCategoryResourceId(id: "user/1234/category/5678") let newerThan: Date? = Date(timeIntervalSinceReferenceDate: 0) @@ -132,7 +132,7 @@ class FeedlySyncStreamContentsOperationTests: XCTestCase { // Find articles inserted. let articleIds = Set(service.pages.values.map { $0.items }.flatMap { $0 }.map { $0.id }) - let articles = account.fetchArticles(.articleIDs(articleIds)) + let articles = try account.fetchArticles(.articleIDs(articleIds)) XCTAssertEqual(articleIds.count, articles.count) } }