Sprint 13 #5
Sprint 13 #5
Conversation
| func code(from url: URL) -> String? | ||
| } | ||
|
|
||
| class AuthHelper: AuthHelperProtocol { |
| if segue.identifier == segueID { | ||
| guard | ||
| let webViewViewController = segue.destination as? WebViewViewController | ||
| else { fatalError("Failed to prepare for \(segueID)") } |
There was a problem hiding this comment.
По своему опыту хотел бы отметить, что в продакшене не используют fatalError, советую заменить на assertionFailure.
| } | ||
|
|
||
| func logout(completion: @escaping () -> Void) { | ||
| authToken?.removeAll() |
There was a problem hiding this comment.
Данный вызов не удаляет токен, он присваивает authToken пустую строку, которая записывается в значениие токена, затем в SplashScreene у тебя идет проверка на nil, а пустая строка != nil.
Здесь стоит присваивать nil, в результате чего удалиться значение токена
| // func configure(_ presenter: WebViewPresenterProtocol) { | ||
| // self.presenter = presenter | ||
| // self.presenter?.view = self | ||
| // } |
There was a problem hiding this comment.
Рекомендую удалить в проекте лишний закомментированный код
| webView.navigationDelegate = self | ||
| webView.accessibilityActivate() | ||
| // configure(WebViewPresenter(authHelper: AuthHelper())) | ||
| presenter?.viewDidLoad() |
There was a problem hiding this comment.
Для того, чтобы заработал ProgressBar здесь нужно присвоить значение estimatedProgressObservation:
estimatedProgressObservation = webView.observe(
.estimatedProgress,
options: [],
changeHandler: { [weak self] _ , _ in
guard let self else { return }
self.presenter?.didUpdateProgressValue(self.webView.estimatedProgress)
})
| } | ||
|
|
||
| override func viewWillAppear(_ animated: Bool) { | ||
| // rescaleAndCenterImageInScrollView(image: imageView.image!) |
| UIBlockingProgressHUD.show() | ||
| } else { | ||
| let storyboard = UIStoryboard(name: "Main", bundle: nil) | ||
| let authViewController = storyboard.instantiateViewController(withIdentifier: "AuthViewController") as! AuthViewController |
There was a problem hiding this comment.
Рекомендую сделать проверку через guard let
| } | ||
|
|
||
| private func switchToTabBarController() { | ||
| guard let window = UIApplication.shared.windows.first else { fatalError("Invalid Configuration") } |
There was a problem hiding this comment.
Рекомендую заменить на assertionFailure
| let loginTextField = webView.descendants(matching: .textField).element | ||
| XCTAssertTrue(loginTextField.waitForExistence(timeout: 5)) | ||
| loginTextField.tap() | ||
| loginTextField.typeText("course.ios@artsofte.ru") |
There was a problem hiding this comment.
Рекомендую вынести в константы логины, пароли и оставить пустым
| } | ||
|
|
||
| func testProfile() throws { | ||
| app.tabBars.buttons.element(boundBy: 1).tap() |
There was a problem hiding this comment.
Рекомендую делать поиск по идентификатору
2 participants
No description provided.