testdouble / grunt-jasmine-bundle

A "spec" grunt task for Jasmine that includes a standard pack of helpers (jasmine-given, jasmine-stealth, jasmine-only). Uses minijasminenode.
6 stars 4 forks source link

Missing Test Description Output #8

Open dtothefp opened 10 years ago

dtothefp commented 10 years ago

I learned about your plugin on David Mosher's Frontend Masters online course and have been testing it out. Everything was going great and now all of a sudden my terminal output for the test titles is printed to the screen. I have gotten it working intermittently but nothing consistent. I'm wondering if I'm making some sort of syntactical error?

https://github.com/dtothefp/grunt-resemble-cli/blob/master/spec/jasmine_bundle_spec.js

davemo commented 10 years ago

Hey @dtothefp, I took a quick peek at your spec and nothing looks out of place to me; I've got a grunt plugin I tested with this approach since recording the frontend masters course, perhaps it might be helpful to compare? https://github.com/davemo/grunt-datauri-variables/blob/master/spec/grunt_datauri_variables_spec.coffee

(I realize it's in coffeescript, but it's pretty much the same setup as what you have barring the usage of an older version of grunt-jasmine-bundle).

searls commented 10 years ago

I suspect it'd be worthwhile to test a fork that upgrades minijasminenode as well.

On Tue, Sep 9, 2014 at 9:54 AM, David Mosher notifications@github.com wrote:

Hey @dtothefp, I took a quick peek at your spec and nothing looks out of place to me; I've got a grunt plugin I tested with this approach since recording the frontend masters course, perhaps it might be helpful to compare? https://github.com/davemo/grunt-datauri-variables/blob/master/spec/grunt_datauri_variables_spec.coffee

(I realize it's in coffeescript, but it's pretty much the same setup as what you have barring the usage of an older version of grunt-jasmine-bundle).

Reply to this email directly or view it on GitHub: https://github.com/testdouble/grunt-jasmine-bundle/issues/8#issuecomment-54972010

dtothefp commented 10 years ago

@davemo I cloned your repo and I guess it has the same behavior. I swear when I was using Jasmine Bundle earlier I was getting the test description and assertion output in my terminal but maybe I just imagined it. Anyway, here is a screenshot for your datauri output:

screen shot 2014-09-09 at 10 36 41 am

and here is one out of my linked repo above running grunt-jasmine-node, showing the terminal output of the test descriptions/assertions

screen shot 2014-09-09 at 10 38 01 am

Maybe this is not desired/expected behavior for the minijasminenode plugin. @searls I'll try pulling down a fork with the upgrade if you think it would potentially have different behavior.

Also, @davemo I want to say thanks so much for the Frontend Masters course. I've been working with Grunt for a while but there is a lack of good E2E documentation, and your solution for testing was just what I've been looking for.

davemo commented 10 years ago

One thing occurs looking at the screenshots, the jasmine-given syntax supports an optional title string for its aliases, ie:

Given('something exists', function() {});
When('something happens', function() {});
Then('this should happen', function() {});

Could you try adding some title strings to your test and seeing if the console output comes back?

Glad to hear the FEM course and the grunt e2e approach I came up with was helpful :)