Switch to updated version of keystone-nightwatch-e2e #3866

Merged
merged 5 commits into from Jan 10, 2017

Projects

None yet

1 participant

@jstockwin
Member

Description of changes

This is good to go provided tests pass.

Related issues (if any)

Testing

  • Please confirm npm run test-all ran successfully.
@jstockwin
Member
jstockwin commented Jan 9, 2017 edited

@webteckie Tests are seemingly randomly failing.

I've got to go right now, but will try and have a look later. Any ideas about what could be happening?

In some of them, the tunnel identifier is not being set on the travis end, even though it's in nightwatch.json, and in others the selenium session seemingly ends far too early.

jstockwin added some commits Jan 9, 2017
@jstockwin jstockwin Switch to updated version of keystone-nightwatch-e2e
2668aa1
@jstockwin jstockwin Additional KNE version bump
1a19b8f
@jstockwin jstockwin Debug KNE bump
5ee3079
@jstockwin
Member
jstockwin commented Jan 9, 2017 edited

Somewhat perplexingly, the tunnel identifier not being set appears to occur only on the group006 jobs....

@jstockwin jstockwin One more
53b2545
@jstockwin
Member

The group006 jobs were trying to run an excluded test. I've updated the settings path in nightwatch.conf.js, will see if that solves the issue.

@jstockwin
Member

Yeah, fixed it. Will remove all the debug comments then merge if it finishes all the tests properly (which it looks like it's going to)

@jstockwin jstockwin Final version bump (removes debugging)
e61b6a3
@jstockwin jstockwin merged commit 633a55e into master Jan 10, 2017

2 of 3 checks passed

continuous-integration/travis-ci/pr The Travis CI build could not complete due to an error
Details
codeclimate/coverage 57.25%
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
@jstockwin jstockwin deleted the kne-v0.2.0 branch Jan 10, 2017
@bassjacob bassjacob added a commit that referenced this pull request Jan 16, 2017
@jstockwin @bassjacob jstockwin + bassjacob Switch to updated version of keystone-nightwatch-e2e (#3866)
* Switch to updated version of keystone-nightwatch-e2e

* Additional KNE version bump

* Debug KNE bump

* One more

* Final version bump (removes debugging)
eae3699
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment