Skip to content
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

fix ignored ShiroTest #192

Merged
merged 1 commit into from
Jan 30, 2017
Merged

fix ignored ShiroTest #192

merged 1 commit into from
Jan 30, 2017

Conversation

kumm
Copy link
Contributor

@kumm kumm commented Jan 13, 2017

The test worked fine on firefox, and chrome, but failed on phantomjs.
Upgrading phantomjsdriver to 1.3.0 does not help.
Workaround was to change LoginPane, and enclosing view to full-Sized.
Related issue #186


This change is Reviewable

@hesara
Copy link
Contributor

hesara commented Jan 26, 2017

:lgtm: but might need a rebase after 183 is merged


Reviewed 3 of 3 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.


Comments from Reviewable

@hesara
Copy link
Contributor

hesara commented Jan 26, 2017

Rebase needed to accommodate for another change in ShiroTest from #183.

@kumm kumm force-pushed the to-upstream/fix-186 branch from 5c3766d to ae7dafc Compare January 27, 2017 21:12
@kumm
Copy link
Contributor Author

kumm commented Jan 27, 2017

rebased onto master

@hesara
Copy link
Contributor

hesara commented Jan 30, 2017

Reviewed 1 of 1 files at r2.
Review status: all files reviewed at latest revision, all discussions resolved.


Comments from Reviewable

@hesara hesara merged commit 8cce1e7 into vaadin:master Jan 30, 2017
@kumm kumm deleted the to-upstream/fix-186 branch April 27, 2017 20:00
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.

2 participants