-
Notifications
You must be signed in to change notification settings - Fork 5
From all passing to 7 failed, 73 skipped #310
Comments
I haven't been following the mass of changes lately from the Java team, and it's definitely possible they made breaking changes in the TCK. I'll have to defer to @dogeared |
Let's setup a time after the conference I'm at to do a screenshare. We did make a lot of changes (and additions for stuff that wasn't represented). The intention was to ensure that we wouldn't disturb other existing tests. One thing that was introduced prior to our work that you may want to look at is that you must set a |
Hey @dogeared! I've got that env already because the Rails integration needs it. I'm going on vacation today, we can hear from each other when I'm back two weeks from now! |
This is also failing for @bretterer |
Just by way of follow up for @edjiang @DamirSvrtan @omgitstom - I setup a rails app following the instructions here: https://github.com/stormpath/stormpath-rails And I ran the TCK against it and the results are 112 tests run, 4 failures, none skipped. This was running the TCK from IntelliJ. Now, when I first ran it from the command line, I was getting similar results that you indicated above:
The failures were because I had the wrong Once I changed the command line to this:
Then, I got the same results as in IntelliJ - 4 failures. The I tested locally by making sure I had a So, I think the root cause of the failures you saw is that the TCK is not finding the API key id and secret. Please shoot me the command line you have been using when you get back from your vacation. |
Hey everyone,
I'm doing the Rails integration and until a month ago I was 99% compatible with the TCK (except the social login that we decided to skip for this release).
In the last month lots of stuff in the TCK stopped working.
I believe this commit caused those failures: 736b1ea
Since then, I've started getting:
I've reverted to a commit before it and the only spec that's failing is
testNoRedirectionStickinessHtml
.@edjiang @dogeared @omgitstom can you check this out?
I'm also on the stormpath slack, we can do more debugging over there.
The text was updated successfully, but these errors were encountered: