jasmine-tool and npm

15 views
Skip to first unread message

Steve Conover

unread,
Feb 28, 2011, 2:47:24 PM2/28/11
to jasmine...@googlegroups.com
It seems that "jasmine" has been claimed in the npm repo by James Carr:

https://github.com/jamescarr/jasmine-tool

I'm thinking of asking him to rename this to jasmine-tool in npm. Are
people here ok with me doing that?

Rajan Agaskar

unread,
Feb 28, 2011, 3:24:19 PM2/28/11
to jasmine...@googlegroups.com, Steve Conover
Would this cause confusion for current jasmine-tool users? 

What would we expect to be installed here? The jasmine-gem project? 

I don't see any reason not to ask, however.

Thanks!

Rajan


--
You received this message because you are subscribed to the Google Groups "Jasmine Dev" group.
To post to this group, send email to jasmine...@googlegroups.com.
To unsubscribe from this group, send email to jasmine-js-de...@googlegroups.com.
For more options, visit this group at http://groups.google.com/group/jasmine-js-dev?hl=en.


Steve Conover

unread,
Feb 28, 2011, 3:47:22 PM2/28/11
to jasmine...@googlegroups.com, Rajan Agaskar
I would expect

npm jasmine

to install jasmine...

https://github.com/pivotal/jasmine/pull/67

Davis Frank

unread,
Feb 28, 2011, 11:22:20 PM2/28/11
to jasmine...@googlegroups.com, Steve Conover, Rajan Agaskar
I'd like to talk more about Jasmine & Node in person - say next time you're in the office for breakfast.

I don't understand enough about Node yet to appreciate how it all works. My current assumption is that there should be some wrapper of Jasmine for Node. This is the point of Misko's jasmine-node project.

I'm more inclined to think about making changes that make Jasmine core work more easily in Node are great, but are lower priority in the immediate term. If standalone usage has to change in order to setup & exec the environment then this is great for 2.0.

--dwf

Steve Conover

unread,
Feb 28, 2011, 11:49:59 PM2/28/11
to Davis Frank, jasmine...@googlegroups.com, Rajan Agaskar
On Mon, Feb 28, 2011 at 8:22 PM, Davis Frank <dwf...@pivotallabs.com> wrote:
> I'd like to talk more about Jasmine & Node in person - say next time you're
> in the office for breakfast.
> I don't understand enough about Node yet to appreciate how it all works. My
> current assumption is that there should be some wrapper of Jasmine for Node.

Node, at least at it relates to jasmine, is mostly just a way of
saying "CommonJS compatibility".

> This is the point of Misko's jasmine-node project.

Hrm, Misko just made a jasmine runner for node. That's different from
making this work:

npm install jasmine
...
var jasmine = require("jasmine")

Misko couldn't make that happen unless he forked jasmine and submitted
pull requests back to pivotal/jasmine.

> I'm more inclined to think about making changes that make Jasmine core work
> more easily in Node are great, but are lower priority in the immediate term.
> If standalone usage has to change in order to setup & exec the environment
> then this is great for 2.0.

Davis I don't want to go upsetting some consensus here. Or maybe I
do. I find the talk of "2.0" to be odd. Why can't we just have some
broad ongoing shepherding of pull requests - merging pull requests,
asking for changes then merging, rejecting pull requests - and have
jasmine evolve the way any other github project does?

What I'm driving towards is getting jasmine in shape such that one can
properly declare a dependency into it from a node project. As of
today jasmine must be copied around everywhere:

https://github.com/mhevery/jasmine-node/blob/master/lib/jasmine/jasmine-1.0.1.js

or included via sibling directories:

https://github.com/sconover/knit-js/blob/master/test/helper.js

vs the ideal, which is a simple Gemfile-like dependency declaration:

https://github.com/assaf/zombie/blob/master/package.json

I fully intend to replace all browser testing I do with node/jsdom,
and I'd like to use jasmine as my testing framework, and am attempting
to do what's typically done in git-land to get it there. It won't
take all that much to get jasmine in compliance with CommonJS style
(an initial take is sitting in a pull request).

But if this is moving too fast, or if the aims of pivotal/jasmine are
to stay browser-centric, I have no problem letting this live on my
fork. I could talk to the jasmine-tool guy and make sconover/jasmine
the npm installed version.

Davis Frank

unread,
Mar 1, 2011, 12:42:32 AM3/1/11
to Steve Conover, jasmine...@googlegroups.com, Rajan Agaskar
It's a speed issue. I'm totally on board with your last paragraph.

Once you've got npm settled can you help us with a small Node project that allows us to self-test Jasmine in Node? That will help lots for smoke test of fixes and make sure that Node support in 2.0 isn't an afterthought.

--dwf

Steve Conover

unread,
Mar 1, 2011, 12:51:11 AM3/1/11
to Davis Frank, jasmine...@googlegroups.com, Rajan Agaskar

Steve Conover

unread,
Mar 1, 2011, 12:53:53 AM3/1/11
to Davis Frank, jasmine...@googlegroups.com, Rajan Agaskar
While we're talking about pull requests could someone
please-pretty-please merge this?:

https://github.com/pivotal/jasmine/pull/46

Steve Conover

unread,
Mar 1, 2011, 1:06:38 AM3/1/11
to Davis Frank, jasmine...@googlegroups.com, Rajan Agaskar
Btw when you look at that you're going to think "oh Steve copied in
some code from jasmine-node. Oh right because you can't just depend
on jasmine-node because of its own jasmine dependency."

Then after a while you might think "well maybe jasmine core should be
able to run its own tests in node out of the box". "You need
something like TrivialReporter, except for the console."

...

https://github.com/sconover/jasmine/blob/trivial_console_reporter/src/console/TrivialConsoleReporter.js

The logical conclusion is that jasmine grows its own jasmine-node
capability. I don't think we should think of that as a bad thing (or,
even if you think it's bad, it's at least a forced move). Imagine
jasmine as it stands with the browser-based suite sitting in another
project...it's weird to even consider (or, it is to me).

Davis Frank

unread,
Mar 1, 2011, 1:12:01 AM3/1/11
to Steve Conover, jasmine...@googlegroups.com, Rajan Agaskar
Once we can get a reliable CI build, then yes, we'll be looking at improving stack traces, etc.

--dwf

Steve Conover

unread,
Mar 1, 2011, 1:38:20 AM3/1/11
to Davis Frank, jasmine...@googlegroups.com, Rajan Agaskar
The tests run green for me, what does jasmine-core @ HEAD need besides that?

Steve Conover

unread,
Mar 1, 2011, 1:48:16 AM3/1/11
to Davis Frank, jasmine...@googlegroups.com, Rajan Agaskar
I realize this is email and I may sound like and asshole but here's a
question I have to everyone running CI for a project like this -
*especially* when you consider it a blocker for progress. Why are you
doing it? If you can simply run your tests in process - let's say

- browser run
- node run
- jshint check

For a project like jasmine, at HEAD, why is anyone worrying about a CI
green off in the world somewhere? I'm looking through the code and I
see nothing that requires a server process or anything complex...

Steve Conover

unread,
Mar 1, 2011, 2:45:47 AM3/1/11
to Davis Frank, jasmine...@googlegroups.com, Rajan Agaskar
> Then after a while you might think "well maybe jasmine core should be
> able to run its own tests in node out of the box".  "You need
> something like TrivialReporter, except for the console."
>
> ...
>
> https://github.com/sconover/jasmine/blob/trivial_console_reporter/src/console/TrivialConsoleReporter.js

...pull request:

https://github.com/pivotal/jasmine/pull/68

Reply all
Reply to author
Forward
0 new messages