Skip to content
This repository was archived by the owner on Aug 27, 2020. It is now read-only.

Update nightwatch.conf.js (fixes e2e test) #1

Merged
merged 1 commit into from
Mar 31, 2017
Merged

Conversation

dvic
Copy link

@dvic dvic commented Mar 31, 2017

On my machine the e2e test fails, the original template also uses 'localhost' instead of '127.0.0.1', do you remember why you changed it from 'localhost' to '127.0.0.1'?

@Toilal
Copy link
Owner

Toilal commented Mar 31, 2017

It was a test to make it pass on travisCI, and I forgot to revert it back. Thanks.

@Toilal Toilal merged commit 57dd845 into Toilal:master Mar 31, 2017
@Toilal
Copy link
Owner

Toilal commented Mar 31, 2017

I've also enabled issues on the project. Hope you like it :). Do you use TypeScript with it ?

@dvic
Copy link
Author

dvic commented Mar 31, 2017

Yes :) I was looking for a Typescript template that has testing (unit and e2e) set up along with code coverage 👍

I've also submitted the first issue: #2 😉

@Toilal
Copy link
Owner

Toilal commented Mar 31, 2017

About code coverage with TypeScript, i'm not sure it's accurate. i think we need to compute the source coverage using sourcemaps, using https://github.com/sshev/karma-remap-coverage. If you can make some tests about this, it would be great

@dvic dvic deleted the patch-1 branch March 31, 2017 16:53
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants