Skip to content
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

make "npm run e2e" work again #50

Closed
talkol opened this issue Nov 30, 2016 · 1 comment
Closed

make "npm run e2e" work again #50

talkol opened this issue Nov 30, 2016 · 1 comment

Comments

@talkol
Copy link

talkol commented Nov 30, 2016

cd detox/detox
npm install
npm run build
npm run test-clean
npm run test-install
# don't run this now: npm run test
cd test
npm run e2e

The last line is how end users (not people developing detox itself) will eventually run detox
this means any parameters you want to send need to be designed to make sense there, like:
npm run e2e -- --scheme=ios-simulator.release
npm run e2e -- --verbose

If you want to improve the scripts for people developing detox, this must come as step 2 after the API for end users is finished and working

@LeoNatan
Copy link
Contributor

LeoNatan commented Jan 10, 2017

It just works©. npm run e2e is how we run tests on Travis CI.

Other improvements such as verbose, target and scheme should all come when CLI tools are developed (#56).

@wix wix locked and limited conversation to collaborators Jul 23, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants