-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Detox: Browser Tests #1147
Detox: Browser Tests #1147
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.
🔥 🔥 🔥 🔥 🔥 🔥 🔥 🔥 v
@@ -264,7 +264,7 @@ class AccountApproval extends PureComponent { | |||
<Text style={styles.headerTitle} testID={'dapp-name-title'} numberOfLines={1}> | |||
{title} | |||
</Text> | |||
<Text style={styles.headerUrl} numberOfLines={1}> | |||
<Text style={styles.headerUrl} testID={'dapp-name-url'} numberOfLines={1}> |
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.
Break down props in different lines
@@ -128,7 +128,7 @@ export default class BrowserBottomBar extends PureComponent { | |||
<TouchableOpacity onPress={showTabs} style={styles.iconButton} testID={'show-tabs-button'}> | |||
<TabCountIcon style={styles.tabIcon} /> | |||
</TouchableOpacity> | |||
<TouchableOpacity onPress={goHome} style={styles.iconButton}> | |||
<TouchableOpacity onPress={goHome} style={styles.iconButton} testID={'home-button'}> |
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.
Break down props in different lines
e2e/browser-tests.spec.js
Outdated
'use strict'; | ||
import TestHelpers from './helpers'; | ||
|
||
const EthereumEnable = 'https://brunobar79.github.io/dapp-issues-repro/'; |
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.
Make this all caps
* added tests for browser and dapps * rename files to be picked up by e2e tests * updates
This PR tests the following:
Seen here = http://recordit.co/z9mlQdH7ug