-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add unit tests #6
Labels
Comments
JDGrimes
added a commit
that referenced
this issue
Jul 26, 2016
JDGrimes
added a commit
that referenced
this issue
Jul 26, 2016
JDGrimes
added a commit
that referenced
this issue
Jul 26, 2016
JDGrimes
added a commit
that referenced
this issue
Jul 26, 2016
It doesn’t make sense to run it against them against the development version, because they will inevitably fail as there will be no updates for WordPoints. Possibly we’ll change this in the future by adding some sort of workaround. See #6
JDGrimes
added a commit
that referenced
this issue
Jul 26, 2016
After clicking the install button, we need to switch back out of the iframe to the parent page, or else the web driver will be looking for the ‘Updated!’ message inside of the iframe, and so the tests will fail. See #6
JDGrimes
added a commit
that referenced
this issue
Jul 26, 2016
JDGrimes
added a commit
that referenced
this issue
Jul 26, 2016
Normally we disable these during the acceptance tests, but in this case we need them to happen. See #6
I just can't get the acceptance tests to work properly. I guess we'll have to flush them out further later. |
JDGrimes
added a commit
that referenced
this issue
Jul 28, 2016
JDGrimes
added a commit
that referenced
this issue
Jul 28, 2016
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
We really need unit tests here.
The text was updated successfully, but these errors were encountered: