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

Re-write integration tests for registering. #567

Open
okdistribute opened this issue Jun 1, 2017 · 1 comment
Open

Re-write integration tests for registering. #567

okdistribute opened this issue Jun 1, 2017 · 1 comment
Assignees
Labels

Comments

@okdistribute
Copy link
Collaborator

okdistribute commented Jun 1, 2017

The integration tests for registering and logging into accounts has been broken undeterministically.

They used to look like this:

https://github.com/datproject/datproject.org/blob/e8b1a801bf8250566c26f999906df042a33b882f/tests/e2e/user.js#L16-L26

but would half of the time fail like this:

https://travis-ci.org/datproject/datproject.org/builds/238098255#L2178

@clkao said it is likely a bug in nightwatch.js

I removed the tests for now because we need to move forward with release, but we should try to fix this.

@okdistribute okdistribute changed the title Re-write and integration test for registering. Re-write integration tests for registering. Jun 1, 2017
@clkao
Copy link
Contributor

clkao commented Jun 2, 2017

Should we just todo this test rather than completely removing it?

@clkao clkao self-assigned this Jun 8, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

2 participants