-
Notifications
You must be signed in to change notification settings - Fork 6.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(tests): Adds avatar test. #15382
base: master
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
✅ This pull request was succesfully scanned and it was determined that it does not contain any changes that present immediate security concerns. If you would still like for it to be reviewed by an expert from our reviewer community, you can submit it manually via the HackerOne PullRequest dashboard
const avatar | ||
= this.driver.$('//span[@id="localVideoContainer"]//img[contains(@class,"userAvatar")]'); | ||
|
||
return await avatar.isExisting() ? await avatar.getAttribute('src') : null; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
return await avatar.isExisting() ? await avatar.getAttribute('src') : null; | |
return await (avatar.isExisting() ? avatar.getAttribute('src') : null); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this would work.
* Returns resource part of the JID of the user who is currently displayed in the large video area. | ||
*/ | ||
async getLargeVideoResource() { | ||
return this.driver.execute(() => APP.UI.getLargeVideoID()); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
no await?
@@ -182,6 +183,7 @@ | |||
"jsonwebtoken": "9.0.2", | |||
"metro-react-native-babel-preset": "0.77.0", | |||
"patch-package": "6.4.7", | |||
"pretty": "2.0.0", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this really necessary? "2.0.0 • Public • Published 8 years ago" seems unmaintained and it only makes HTML more readable?
No description provided.