Fix: certificates not displayed after being earned

Core/Core/Configuration/Connectivity.swift

@@ -2,7 +2,7 @@
 //  Connectivity.swift
 //  OpenEdX
 //
-//  Created by  Stepanok Ivan on 15.12.2022.
+//  Created by  Stepanok Ivan on 15.12.2022.
 //
 
 import Alamofire
@@ -14,55 +14,99 @@ public enum InternetState: Sendable {
     case notReachable
 }
 
-//sourcery: AutoMockable
+// sourcery: AutoMockable
 @MainActor
 public protocol ConnectivityProtocol: Sendable {
     var isInternetAvaliable: Bool { get }
     var isMobileData: Bool { get }
     var internetReachableSubject: CurrentValueSubject<InternetState?, Never> { get }
 }
 
-@MainActor
 public class Connectivity: ConnectivityProtocol {
-    let networkManager = NetworkReachabilityManager()
-
-    public var isInternetAvaliable: Bool {
-        //        false
-        networkManager?.isReachable ?? false
+
+    private let networkManager = NetworkReachabilityManager()
+    private let verificationURL: URL
+    private let verificationTimeout: TimeInterval
+    private let cacheValidity: TimeInterval = 30
+
+    private var lastVerificationDate: TimeInterval?
+    private var lastVerificationResult: Bool = true
+
+    public let internetReachableSubject = CurrentValueSubject<InternetState?, Never>(nil)
+
+    private(set) var _isInternetAvailable: Bool = true {
+        didSet {
+            Task { @MainActor in
+                internetReachableSubject.send(_isInternetAvailable ? .reachable : .notReachable)
+            }
+        }
     }
-
-    public var isMobileData: Bool {
-        if let networkManager {
-           return networkManager.isReachableOnCellular
-        } else {
-            return false
+
+    public var isInternetAvaliable: Bool {
+        if let last = lastVerificationDate,
+           Date().timeIntervalSince1970 - last < cacheValidity {
+            return lastVerificationResult
+        }
+
+        Task {
+            await performVerification()
         }
+
+        return lastVerificationResult
     }
-
-    public let internetReachableSubject = CurrentValueSubject<InternetState?, Never>(nil)
-
-    public init() {
-        checkInternet()
+
+    public var isMobileData: Bool {
+        networkManager?.isReachableOnCellular == true
     }
-
-    func checkInternet() {
-        if let networkManager {
-            networkManager.startListening { status in
-                DispatchQueue.main.async {
-                    switch status {
-                    case .unknown:
-                        self.internetReachableSubject.send(InternetState.notReachable)
-                    case .notReachable:
-                        self.internetReachableSubject.send(InternetState.notReachable)
-                    case .reachable:
-                        self.internetReachableSubject.send(InternetState.reachable)
-                    }
+
+    public init(
+        config: ConfigProtocol,
+        timeout: TimeInterval = 15
+    ) {
+        self.verificationURL = config.baseURL
+        self.verificationTimeout = timeout
+
+        networkManager?.startListening(onQueue: .global()) { [weak self] status in
+            guard let self = self else { return }
+            Task { @MainActor in
+                switch status {
+                case .reachable:
+                    await self.performVerification()
+                case .notReachable, .unknown:
+                    self.updateAvailability(false, at: 0)
                 }
             }
-        } else {
-            DispatchQueue.main.async {
-                self.internetReachableSubject.send(InternetState.notReachable)
+        }
+    }
+
+    deinit {
+        networkManager?.stopListening()
+    }
+
+    private func performVerification() async {
+        let now = Date().timeIntervalSince1970
+        let live = await verifyInternet()
+        updateAvailability(live, at: now)
+    }
+
+    private func updateAvailability(_ available: Bool, at timestamp: TimeInterval) {
+        _isInternetAvailable = available
+        lastVerificationDate = timestamp
+        lastVerificationResult = available
+    }
+
+    private func verifyInternet() async -> Bool {
+        var request = URLRequest(url: verificationURL)
+        request.httpMethod = "HEAD"
+        request.timeoutInterval = verificationTimeout
+        do {
+            let (_, response) = try await URLSession.shared.data(for: request)
+            if let http = response as? HTTPURLResponse, (200..<400).contains(http.statusCode) {
+                return true
             }
+        } catch {
+            return false
         }
+        return false
     }
 }

Core/Core/View/Base/OfflineSnackBarView.swift

@@ -79,6 +79,7 @@ public struct OfflineSnackBarView: View {
 
 struct OfflineSnackBarView_Previews: PreviewProvider {
     static var previews: some View {
-        OfflineSnackBarView(connectivity: Connectivity(), reloadAction: {})
+        let configMock = ConfigMock()
+        OfflineSnackBarView(connectivity: Connectivity(config: configMock), reloadAction: {})
     }
 }

Core/Core/View/Base/WebBrowser.swift

@@ -81,6 +81,7 @@ public struct WebBrowser: View {
 
 struct WebBrowser_Previews: PreviewProvider {
     static var previews: some View {
-        WebBrowser(url: "", pageTitle: "", connectivity: Connectivity())
+        let configMock = ConfigMock()
+        WebBrowser(url: "", pageTitle: "", connectivity: Connectivity(config: configMock))
     }
 }

Course/Course/Data/Model/Data_CourseProgressResponse.swift

@@ -46,6 +46,32 @@ public extension DataLayer {
             case verificationData = "verification_data"
             case disableProgressGraph = "disable_progress_graph"
         }
+
+        public init(from decoder: Decoder) throws {
+            let values = try decoder.container(keyedBy: CodingKeys.self)
+            verifiedMode = try values.decodeIfPresent(String.self, forKey: .verifiedMode)
+            accessExpiration = try values.decodeIfPresent(String.self, forKey: .accessExpiration)
+            certificateData = try values.decodeIfPresent(CertificateData.self, forKey: .certificateData)
+                ?? CertificateData(
+                    certStatus: nil,
+                    certWebViewUrl: nil,
+                    downloadUrl: nil,
+                    certificateAvailableDate: nil
+                )
+            completionSummary = try values.decode(CompletionSummary.self, forKey: .completionSummary)
+            courseGrade = try values.decode(CourseGrade.self, forKey: .courseGrade)
+            creditCourseRequirements = try values.decodeIfPresent(String.self, forKey: .creditCourseRequirements)
+            end = try values.decodeIfPresent(String.self, forKey: .end)
+            enrollmentMode = try values.decode(String.self, forKey: .enrollmentMode)
+            gradingPolicy = try values.decode(GradingPolicy.self, forKey: .gradingPolicy)
+            hasScheduledContent = try values.decodeIfPresent(Bool.self, forKey: .hasScheduledContent)
+            sectionScores = try values.decode([SectionScore].self, forKey: .sectionScores)
+            studioUrl = try values.decode(String.self, forKey: .studioUrl)
+            username = try values.decode(String.self, forKey: .username)
+            userHasPassingGrade = try values.decode(Bool.self, forKey: .userHasPassingGrade)
+            verificationData = try values.decode(VerificationData.self, forKey: .verificationData)
+            disableProgressGraph = try values.decode(Bool.self, forKey: .disableProgressGraph)
+        }
 
         public init(
             verifiedMode: String?,

Course/Course/Presentation/Container/CourseContainerView.swift

@@ -387,7 +387,7 @@ public struct CourseContainerView: View {
             router: CourseRouterMock(),
             analytics: CourseAnalyticsMock(),
             config: ConfigMock(),
-            connectivity: Connectivity(),
+            connectivity: Connectivity(config: ConfigMock()),
             manager: DownloadManagerMock(),
             storage: CourseStorageMock(),
             isActive: true,
@@ -403,7 +403,7 @@ public struct CourseContainerView: View {
             interactor: CourseInteractor.mock,
             router: CourseRouterMock(),
             cssInjector: CSSInjectorMock(),
-            connectivity: Connectivity(),
+            connectivity: Connectivity(config: ConfigMock()),
             config: ConfigMock(),
             courseID: "1",
             courseName: "a",
@@ -414,7 +414,7 @@ public struct CourseContainerView: View {
             interactor: CourseInteractor.mock,
             router: CourseRouterMock(),
             analytics: CourseAnalyticsMock(),
-            connectivity: Connectivity(),
+            connectivity: Connectivity(config: ConfigMock()),
         ),
         courseID: "",
         title: "Title of Course",

Course/Course/Presentation/Container/CourseContainerViewModel.swift

@@ -122,6 +122,7 @@ public final class CourseContainerViewModel: BaseCourseViewModel {
 
     private let interactor: CourseInteractorProtocol
     private let authInteractor: AuthInteractorProtocol
+
     let analytics: CourseAnalytics
     let coreAnalytics: CoreAnalytics
     private(set) var storage: CourseStorage
@@ -214,7 +215,7 @@ public final class CourseContainerViewModel: BaseCourseViewModel {
             courseVideoStructure: nil
         )
     }
-    
+
     @MainActor
     func getCourseStructure(courseID: String) async throws -> CourseStructure? {
         if isInternetAvaliable {
@@ -1766,7 +1767,7 @@ extension CourseContainerViewModel {
             router: CourseRouterMock(),
             analytics: CourseAnalyticsMock(),
             config: ConfigMock(),
-            connectivity: Connectivity(),
+            connectivity: Connectivity(config: ConfigMock()),
             manager: DownloadManagerMock(),
             storage: CourseStorageMock(),
             isActive: true,

Course/Course/Presentation/Content/CourseContentView.swift

@@ -17,6 +17,7 @@ public struct CourseContentView: View {
     @StateObject private var viewModel: CourseContainerViewModel
     private let title: String
     private let courseID: String
+    @State private var openCertificateView: Bool = false
 
     private var videoContentData: VideoContentData {
         VideoContentData(
@@ -97,6 +98,8 @@ public struct CourseContentView: View {
                                 )
                                     .padding(.horizontal, 24)
                                     .padding(.top, 16)
+
+                                certificateView
 
                                 // MARK: - Content based on selected tab
                                 contentForSelectedTab(proxy: proxy)
@@ -176,6 +179,34 @@ public struct CourseContentView: View {
                 .ignoresSafeArea()
         )
     }
+
+    @ViewBuilder
+    private var certificateView: some View {
+        if let certificate = viewModel.courseStructure?.certificate,
+           let url = certificate.url,
+           url.count > 0 {
+            MessageSectionView(
+                title: CourseLocalization.Outline.passedTheCourse(title),
+                actionTitle: CourseLocalization.Outline.viewCertificate,
+                action: {
+                    openCertificateView = true
+                    viewModel.trackViewCertificateClicked(courseID: courseID)
+                }
+            )
+            .padding(.horizontal, 24)
+            .padding(.top, 16)
+            .fullScreenCover(
+                isPresented: $openCertificateView,
+                content: {
+                    WebBrowser(
+                        url: url,
+                        pageTitle: CourseLocalization.Outline.certificate,
+                        connectivity: viewModel.connectivity
+                    )
+                }
+            )
+        }
+    }
 
     @ViewBuilder
     private func contentForSelectedTab(proxy: GeometryProxy) -> some View {
@@ -266,7 +297,7 @@ public struct CourseContentView: View {
         router: CourseRouterMock(),
         analytics: CourseAnalyticsMock(),
         config: ConfigMock(),
-        connectivity: Connectivity(),
+        connectivity: Connectivity(config: ConfigMock()),
         manager: DownloadManagerMock(),
         storage: CourseStorageMock(),
         isActive: true,

Course/Course/Presentation/Content/Subviews/AllContentView.swift

@@ -142,7 +142,7 @@ struct AllContentView: View {
         router: CourseRouterMock(),
         analytics: CourseAnalyticsMock(),
         config: ConfigMock(),
-        connectivity: Connectivity(),
+        connectivity: Connectivity(config: ConfigMock()),
         manager: DownloadManagerMock(),
         storage: CourseStorageMock(),
         isActive: true,

Course/Course/Presentation/Dates/CourseDatesView.swift

@@ -196,7 +196,7 @@ struct CourseDatesView_Previews: PreviewProvider {
             interactor: CourseInteractor(repository: CourseRepositoryMock()),
             router: CourseRouterMock(),
             cssInjector: CSSInjectorMock(),
-            connectivity: Connectivity(),
+            connectivity: Connectivity(config: ConfigMock()),
             config: ConfigMock(),
             courseID: "",
             courseName: "",

Course/Course/Presentation/Handouts/HandoutsView.swift

@@ -123,7 +123,7 @@ struct HandoutsView_Previews: PreviewProvider {
         let viewModel = HandoutsViewModel(interactor: CourseInteractor.mock,
                                           router: CourseRouterMock(),
                                           cssInjector: CSSInjectorMock(),
-                                          connectivity: Connectivity(),
+                                          connectivity: Connectivity(config: ConfigMock()),
                                           courseID: "",
                                           analytics: CourseAnalyticsMock())
         HandoutsView(