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.

Sign up for GitHub

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

Jump to bottom

TS: FormAuthenticationWithSessionReplicationTest is still disabled #58

Open
mchoma opened this issue Jan 20, 2017 · 2 comments
Open

TS: FormAuthenticationWithSessionReplicationTest is still disabled #58

mchoma opened this issue Jan 20, 2017 · 2 comments
Milestone
1.0.0.Beta15

Comments

@mchoma
Copy link
Contributor

mchoma commented Jan 20, 2017 •
edited

FormAuthenticationWithSessionReplicationTest is currently ignored.
Once enabled I get test failure
FormAuthenticationWithSessionReplicationTest.testSessionInvalidation:115->AbstractHttpServerMechanismTest.assertSuccessfulResponse:82 expected:<1> but was:<0>

@pferraro ^^^
@mchoma mchoma changed the title FormAuthenticationWithSessionReplicationTest is still disabled TS: FormAuthenticationWithSessionReplicationTest is still disabled Jan 20, 2017
@pferraro
Copy link
Contributor

pferraro commented Jan 20, 2017 •
edited

@mchoma I would delete this test. I don't think it adds any value on top of the ClusteredSSO test. It also relies on an unverified distributed Undertow SessionManager test harness.

@darranl darranl modified the milestone: 1.0.0.Beta9 Jan 21, 2017
@mchoma
Copy link
Contributor Author

mchoma commented Jan 23, 2017

@pedroigor , as author of FormAuthenticationWithSessionReplicationTest, do you think it is redundant?

@darranl darranl modified the milestones: 1.0.0.Beta11, 1.0.0.Beta12 Feb 16, 2017
@darranl darranl modified the milestones: 1.0.0.Beta14, 1.0.0.Beta15 Mar 12, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
1.0.0.Beta15
Development

No branches or pull requests
3 participants
@pferraro @darranl @mchoma