-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
Jasmine upgrade #412
Jasmine upgrade #412
Conversation
Looks good. All of our changes still work. Looking forward to adding categories. |
It looks like both the "debug" links and the link to "select a test to run" were both lost as part of this upgrade. Was that intentional? Apologies for the late comments. |
The debug links are still there on each spec, but not a suite. I'm not sure what the "select a test to run" link was, but we can still click on a spec or suite to run just that. |
There both there, but only at the individual test level and not the module On Thu, Jan 3, 2013 at 2:32 PM, Scott Hunter notifications@github.comwrote:
|
I checked more closely and it turns out that they are present on passing specs, but missing on failed specs, which is actually the only case you want that link. I'll work on fixing it at some point. |
Gotcha, submit an issue. This is going to become a problem quickly. |
Upgrade Jasmine from version 1.1.0 to 1.3.0, incorporating our previous changes.