Skip to content

moved checkError method to base.screen, renamed it to checkErrorModal…#1180

Merged
tomholub merged 3 commits intomasterfrom
tests/issue-1162
Dec 7, 2021
Merged

moved checkError method to base.screen, renamed it to checkErrorModal…#1180
tomholub merged 3 commits intomasterfrom
tests/issue-1162

Conversation

@fcvakintos
Copy link
Contributor

@fcvakintos fcvakintos commented Dec 6, 2021

…, added checking error to mock/setup/SetupFailsWithInconsistentClientConfiguration spec

This PR contains changes for checkError method, it is moved to base screen and renamed as checkErrorModal, added this method to mock/setup/SetupFailsWithInconsistentClientConfiguration spec
close #1162

issue #1179


Tests

  • Tests added or updated

To be filled by reviewers

I have reviewed that this PR... (tick whichever items you personally focused on during this review):

  • addresses the issue it closes (if any)
  • code is readable and understandable
  • is accompanied with tests, or tests are not needed
  • is free of vulnerabilities
  • is documented clearly and usefully, or doesn't need documentation

…, added checking error to mock/setup/SetupFailsWithInconsistentClientConfiguration spec
@fcvakintos fcvakintos requested a review from tomholub December 6, 2021 13:05
@fcvakintos fcvakintos requested a review from tomholub December 7, 2021 12:44
@tomholub tomholub enabled auto-merge (squash) December 7, 2021 13:16
@tomholub tomholub merged commit a5f83fb into master Dec 7, 2021
@tomholub tomholub deleted the tests/issue-1162 branch December 7, 2021 13:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

use accessibility identifier for error modal

3 participants