Skip to content

Commit

Permalink
Merge pull request #234 from nodlesh/fix-bcw-test-id-is-this-app-for-you
Browse files Browse the repository at this point in the history
BCW Updated TestID on is this app for you text
  • Loading branch information
nodlesh authored Feb 29, 2024
2 parents 49a6e62 + 287dd86 commit 58fbbb6
Showing 1 changed file with 1 addition and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ class OnboardingIsThisAppForYouPage(BasePage):
# we could create a locator module that has all the locators. Given a specific app we could load the locators for that app.
# not sure this would be a use case that would be common. Leaving locators with the page objects for now.
on_this_page_text_locator = "Is this app for you?"
on_this_page_locator = (AppiumBy.NAME, "Is this app for you?") # works on iOS
on_this_page_locator = (AppiumBy.ID, "com.ariesbifold:id/DeveloperCounter")
confirm_locator = (AppiumBy.ID, "com.ariesbifold:id/IAgree")
continue_locator = (AppiumBy.ID, "com.ariesbifold:id/Continue")

Expand All @@ -22,8 +22,6 @@ def on_this_page(self):
timeout = 10
if "Local" in os.environ['DEVICE_CLOUD']:
timeout = 100
if self.current_platform == "Android":
return super().on_this_page(self.on_this_page_text_locator, timeout)
return super().on_this_page(self.on_this_page_locator, timeout)

def get_onboarding_text(self):
Expand Down

0 comments on commit 58fbbb6

Please sign in to comment.