From b5b677bafb1ea5961854bdd1ef75a7fe5a10c97b Mon Sep 17 00:00:00 2001 From: mazevedo Date: Fri, 11 Oct 2024 16:05:56 -0300 Subject: [PATCH] Use discardable return in func return to allow better tetsing --- .../Year In Review/WMFYearInReviewDataController.swift | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/WMFData/Sources/WMFData/Data Controllers/Year In Review/WMFYearInReviewDataController.swift b/WMFData/Sources/WMFData/Data Controllers/Year In Review/WMFYearInReviewDataController.swift index be2d2070ed..e0ca627832 100644 --- a/WMFData/Sources/WMFData/Data Controllers/Year In Review/WMFYearInReviewDataController.swift +++ b/WMFData/Sources/WMFData/Data Controllers/Year In Review/WMFYearInReviewDataController.swift @@ -43,10 +43,11 @@ public class WMFYearInReviewDataController { return true } - public func createOrRetrieveYearInReview(for year: Int, countryCode: String, primaryAppLanguageProject: WMFProject?) async { + @discardableResult + public func createOrRetrieveYearInReview(for year: Int, countryCode: String, primaryAppLanguageProject: WMFProject?) async -> WMFYearInReviewReport? { guard shouldCreateOrRetrieveYearInReview(countryCode: countryCode, primaryAppLanguageProject: primaryAppLanguageProject) else { - return + return nil } var report = try? await fetchYearInReviewReport(forYear: year) @@ -58,6 +59,7 @@ public class WMFYearInReviewDataController { report = try? await fetchYearInReviewReport(forYear: year) } + return report } func getSlides() -> [WMFYearInReviewSlide] {