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

Integration tests - #181

Open
ghost wants to merge 9 commits into
masterfrom
integration_tests
Open

ghost wants to merge 9 commits into
masterfrom
integration_tests

Conversation

@ghost

@ghost ghost commented Jun 1, 2016

Copy link
Copy Markdown

Comment thread tests/integration/test_login_email.js Outdated
phantom.clearCookies();
casper.thenOpenAndEvaluate(host+"/login", function(test_username, test_password){
document.querySelector('#username').value = username;
document.querySelector('#password').value = password;

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.

How are username and password in scope here? Shouldn't these be assigned to test_username and test_password?

@ghost
ghost force-pushed the integration_tests branch from 2da9b74 to 97dfdad Compare June 1, 2016 11:09
@jace
jace requested a review from iambibhas October 1, 2018 17:23
@jace

jace commented Oct 1, 2018

Copy link
Copy Markdown
Member

@iambibhas please review if this PR can be rebased and merged.

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