Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

Commit

Permalink
Merge pull request #8594 from brave/luixxiul-patch-1
Browse files Browse the repository at this point in the history
Update README.md
  • Loading branch information
cezaraugusto authored May 1, 2017
2 parents 2dce3c5 + 2da149b commit 4bc5b67
Showing 1 changed file with 10 additions and 6 deletions.
16 changes: 10 additions & 6 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -114,19 +114,23 @@ Now actually run Brave in another terminal

npm start

To run the webdriver tests:

npm run watch-test or npm run watch-all

npm test

Some errors related to [brave/electron](https://github.com/brave/electron) update can be fixed by doing a clean install:

rm -rf node_modules/
npm install

If this does not work, please clear out your ~/.electron first and try again.

### Running webdriver tests

To run the webdriver tests

npm run watch-test or npm run watch-all

Now run tests in another terminal

npm test

### Port

Brave uses port 8080 to communicate between its client and server sides by default. If you are using port 8080 for something else (e.g. a web proxy) then you can set the node config to make it use a different one.
Expand Down

0 comments on commit 4bc5b67

Please sign in to comment.