Skip to content

Adding db reset helper to e2e tests.#167

Merged
sean-sq-chen merged 2 commits into
masterfrom
reset-db-e2e-tests
Apr 1, 2016
Merged

Adding db reset helper to e2e tests.#167
sean-sq-chen merged 2 commits into
masterfrom
reset-db-e2e-tests

Conversation

@randyochoa
Copy link
Copy Markdown
Contributor

@mwhagedorn @ongk

  - Unsure if this is best way to fix. Opening PR to get feedback.
  - This PR is dependent on changes to stratos-node-server:
    https://github.com/hpcloud/stratos-node-server/pull/31
@mwhagedorn
Copy link
Copy Markdown
Contributor

that looks reasonable

@randyochoa randyochoa changed the title WIP: Adding db reset helper to e2e tests. Adding db reset helper to e2e tests. Apr 1, 2016
Comment thread e2e/po/helpers.po.js Outdated
url: loginUrl,
body: JSON.stringify({username: "dev", password: "dev"})
}, function (error, response) {
if (!error && response.statusCode == 200) {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

===

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

@sean-sq-chen
Copy link
Copy Markdown
Contributor

LGTM, some nits.

@ongk
Copy link
Copy Markdown
Contributor

ongk commented Apr 1, 2016

LGTM

@sean-sq-chen sean-sq-chen merged commit e4fd739 into master Apr 1, 2016
@sean-sq-chen sean-sq-chen deleted the reset-db-e2e-tests branch April 1, 2016 19:14
irfanhabib added a commit that referenced this pull request Jun 1, 2017
Standardize logging and bring in the configuration reader
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants