Merge pull request #17 from rcaos/testing-schedulers

Fix Testing schedulers

Author: rcaos

Projects/Features/Account/Project.swift

@@ -19,7 +19,8 @@ let project = Project.framework(
     .project(
       target: "TVShowsListInterface",
       path: .relativeToRoot("Projects/Features/TVShowsList")
-    )
+    ),
+    .package(product: "CombineSchedulers")
   ],
   testFolder: "Tests",
   testDependencies: [

Projects/Features/Account/Sources/Presentation/Account/View/AccountViewController.swift

@@ -30,14 +30,15 @@ class AccountViewController: NiblessViewController {
 
   override func viewDidLoad() {
     super.viewDidLoad()
+    viewModel.viewDidLoad()
     subscribe()
   }
 
   // MARK: - Setup UI
   private func subscribe() {
     viewModel
       .viewState
-      .receive(on: RunLoop.main)
+      .receive(on: DispatchQueue.main)
       .sink(receiveCompletion: { _ in }, receiveValue: { [weak self] viewState in
         self?.setupUI(with: viewState)
       })

Projects/Features/Account/Sources/Presentation/Account/ViewModel/AccountViewModel.swift

@@ -8,6 +8,7 @@
 
 import Foundation
 import Combine
+import CombineSchedulers
 import NetworkingInterface
 import Shared
 
@@ -17,6 +18,7 @@ enum AccountViewState: Equatable {
 }
 
 protocol AccountViewModelProtocol: AuthPermissionViewModelDelegate {
+  func viewDidLoad()
   var viewState: CurrentValueSubject<AccountViewState, Never> { get }
 }
 
@@ -28,6 +30,7 @@ final class AccountViewModel: AccountViewModelProtocol {
 
   weak var coordinator: AccountCoordinatorProtocol?
   private var disposeBag = Set<AnyCancellable>()
+  private let scheduler: AnySchedulerOf<DispatchQueue>
 
   // MARK: - Public Api
   let viewState: CurrentValueSubject<AccountViewState, Never> = .init(.login)
@@ -36,15 +39,21 @@ final class AccountViewModel: AccountViewModelProtocol {
   init(createNewSession: CreateSessionUseCase,
        fetchAccountDetails: FetchAccountDetailsUseCase,
        fetchLoggedUser: FetchLoggedUser,
-       deleteLoguedUser: DeleteLoguedUserUseCase) {
+       deleteLoguedUser: DeleteLoguedUserUseCase,
+       scheduler: AnySchedulerOf<DispatchQueue> = .main
+  ) {
     self.createNewSession = createNewSession
     self.fetchAccountDetails = fetchAccountDetails
     self.fetchLoggedUser = fetchLoggedUser
     self.deleteLoguedUser = deleteLoguedUser
+    self.scheduler = scheduler
+  }
+
+  func viewDidLoad() {
     checkIsLogued()
   }
 
-  fileprivate func checkIsLogued() {
+  private func checkIsLogued() {
     if fetchLoggedUser.execute() != nil {
       fetchUserDetails()
     } else {
@@ -54,7 +63,7 @@ final class AccountViewModel: AccountViewModelProtocol {
 
   private func fetchUserDetails() {
     fetchDetailsAccount()
-      .receive(on: RunLoop.main)
+      .receive(on: scheduler)
       .sink(receiveCompletion: { [weak self] completion in
         switch completion {
         case .failure:
@@ -77,7 +86,7 @@ final class AccountViewModel: AccountViewModelProtocol {
         }
         return strongSelf.fetchDetailsAccount()
       }
-      .receive(on: RunLoop.main)
+      .receive(on: scheduler)
       .sink(receiveCompletion: { [weak self] completion in
         switch completion {
         case .failure:

Projects/Features/Account/Sources/Presentation/Profile/View/ProfileRootView.swift

@@ -77,7 +77,7 @@ class ProfileRootView: NiblessView {
         }
         return snapShot
       }
-      .receive(on: RunLoop.main)
+      .receive(on: DispatchQueue.main)
       .sink(receiveCompletion: { _ in }, receiveValue: { [weak self] snapshot in
         self?.dataSource?.apply(snapshot)
       })

Projects/Features/Account/Sources/Presentation/Profile/View/ProfileViewController.swift

@@ -34,7 +34,7 @@ class ProfileViewController: NiblessViewController {
     viewModel
       .presentSignOutAlert
       .filter { $0 == true }
-      .receive(on: RunLoop.main)
+      .receive(on: DispatchQueue.main)
       .sink(receiveCompletion: { _ in }, receiveValue: { [weak self] _ in
         self?.showSignOutActionSheet()
       })

Projects/Features/Account/Sources/Presentation/SignIn/View/SignInViewController.swift

@@ -34,7 +34,7 @@ class SignInViewController: NiblessViewController {
   fileprivate func subscribe() {
     viewModel
       .viewState
-      .receive(on: RunLoop.main)
+      .receive(on: DispatchQueue.main)
       .sink(receiveCompletion: { _ in }, receiveValue: { [weak self] state in
         self?.setupView(with: state)
       })

Projects/Features/Account/Sources/Presentation/SignIn/ViewModel/SignInViewModel.swift

@@ -8,6 +8,7 @@
 
 import Foundation
 import Combine
+import CombineSchedulers
 import Shared
 import NetworkingInterface
 
@@ -17,11 +18,14 @@ class SignInViewModel: SignInViewModelProtocol {
 
   let viewState: CurrentValueSubject<SignInViewState, Never> = .init(.initial)
   weak var delegate: SignInViewModelDelegate?
+
+  private let scheduler: AnySchedulerOf<DispatchQueue>
   private var disposeBag = Set<AnyCancellable>()
 
   // MARK: - Initializers
-  init(createTokenUseCase: CreateTokenUseCase) {
+  init(createTokenUseCase: CreateTokenUseCase, scheduler: AnySchedulerOf<DispatchQueue> = .main) {
     self.createTokenUseCase = createTokenUseCase
+    self.scheduler = scheduler
     subscribe()
   }
 
@@ -42,7 +46,7 @@ class SignInViewModel: SignInViewModelProtocol {
         viewState.send(.loading)
         return createTokenUseCase.execute()
       }
-      .receive(on: RunLoop.main)
+      .receive(on: scheduler)
       .sink(receiveCompletion: { [weak self] completion in
         switch completion {
         case let .failure(error):

Projects/Features/Account/Tests/Account/Mocks/ViewModels/AccountViewModelMock.swift

@@ -10,6 +10,8 @@ import Combine
 
 final class AccountViewModelMock: AccountViewModelProtocol {
 
+  func viewDidLoad() { }
+
   let viewState: CurrentValueSubject<AccountViewState, Never>
 
   init(state: AccountViewState) {

Projects/Features/Account/Tests/Account/Presentation/AccountViewControllerFactoryMock.swift

@@ -19,3 +19,8 @@ class AccountViewControllerFactoryMock: AccountViewControllerFactory {
     return ProfileViewController(viewModel: viewModel)
   }
 }
+
+func configureWith(_ viewController: UIViewController, style: UIUserInterfaceStyle) {
+  viewController.overrideUserInterfaceStyle = style
+  _ = viewController.view
+}