8000 test: fix open notification test by anujkumar21 · Pull Request #1749 · appium/java-client · GitHub
[go: up one dir, main page]

Skip to content

test: fix open notification test#1749

Merged
mykola-mokhnach merged 1 commit intoappium:masterfrom
anujkumar21:test-execution
Aug 26, 2022
Merged

test: fix open notification test#1749
mykola-mokhnach merged 1 commit intoappium:masterfrom
anujkumar21:test-execution

Conversation

@anujkumar21
Copy link
Contributor

Change list

Test openNotification was failing because on the current version 'settings_button' is not visible, so added check for different element. Also add the test to CI.

Types of changes

  • No changes in production code.
  • Bugfix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

@linux-foundation-easycla
Copy link
linux-foundation-easycla bot commented Aug 26, 2022

CLA Signed

The committers listed above are authorized under a signed CLA.

  • ✅ login: anujkumar21 / name: Anuj Kumar (0f5f4e1)

@mykola-mokhnach mykola-mokhnach merged commit 80be558 into appium:master Aug 26, 2022
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.

3 participants

0