Closed juliemr closed 10 years ago
Trying out Jasmine 2.0 in the jasmine2 branch. Everything works except for specifying a timeout specifically for a test, which there is now no way to do without modifying the Jasmine code itself.
Can you elaborate on the timeout issue?
Hey there - we've now made our repo public, but are not yet ready to publish the NPM.
We are interested in using Protractor with jasmine 2.0, which relies on this issue being resolved. We tried your jasmine2 branch and everything appears to be working regarding jasmine timeout intervals. Is there any further hold up? We are part of Pivotal and would like to help.
Also, we noticed that you are not dependent on jasmine-core. We've submitted pull request #14 to address this. Thanks!
Hi @cyrret - I'd be happy to work to Jasmine 2.0! On my last view, things were still settling down with Jasmine, so I was waiting for the dust to settle on 2.0 and its relationship with node. If you think it's ready now, I'd be happy to take another look. Ideally, I don't think minijasminenode should exist at all!
Hi Julie - we've been working on a simple Jasmine runner for node here: https://github.com/pivotal/jasmine-npm
Compared to minijasminenode it's missing the following:
If you can live without these in Protractor, then we encourage you to migrate to jasmine-npm and deprecate minijasminenode.
If you need these features, then our current approach is that folks build something like minijasminenode, but do it on top of jasmine-core vs simply embedding a copy of jasmine within your runner. I have a pull request out on minijasminenode that makes the pertinent changes for this.
We're open to feedback/discussing whether this overall approach is reasonable. Thoughts?
Terry Chen tchen120@gmail.com
On Fri, Apr 4, 2014 at 4:18 PM, Julie Ralph notifications@github.comwrote:
Hi @cyrret https://github.com/cyrret - I'd be happy to work to Jasmine 2.0! On my last view, things were still settling down with Jasmine, so I was waiting for the dust to settle on 2.0 and its relationship with node. If you think it's ready now, I'd be happy to take another look. Ideally, I don't think minijasminenode should exist at all!
Reply to this email directly or view it on GitHubhttps://github.com/juliemr/minijasminenode/issues/9#issuecomment-39620683 .
@cyrret those features are all pretty key. Are there plans to include any of them in jasmine-npm in the future? Jasmine 1.3 is working as is, so there's not a real rush to update minijasminenode for my use cases - I'd be happy to wait for these features if they're in the pipeline.
Otherwise, using jasmine-core instead of embedding a copy sounds entirely reasonable. Just curious.
EDIT - to clarify, (1) and (3) are the really important features. Setting timeouts is also important, as described in pivotal/jasmine#483
Done in 889a53c76a4211f7a68fd4b89b25fd0979a35f3e
Comments here hint that they're doing their own npm package:
https://github.com/pivotal/jasmine/pull/482
Which would be great, but I'm not sure if there's any sort of ETA at all.