meteor-velocity / html-reporter

HTML reporter for Meteor velocity testing framework
MIT License
23 stars 20 forks source link

Cannot read property 'rootUrl' of undefined #82

Closed ghost closed 9 years ago

ghost commented 9 years ago

Originally posted here https://github.com/meteor-velocity/html-reporter/issues/72#issuecomment-126965089 by @sircharleswatson.

This may be similar but, when I updated to the latest, I started getting this error in the console:

Exception in template helper: TypeError: Cannot read property 'rootUrl' of undefined
    at Object.Template.velocityReports.helpers.mirrorUrl (http://localhost:3000/packages/velocity_html-reporter.js?9bd02910ea39c4b8833365509d30e6f81f4e39f0:637:59)
    at bindDataContext (http://localhost:3000/packages/blaze.js?a5c324925e5f6e800a4c618d71caf2848b53bf51:2880:16)
    at Blaze._wrapCatchingExceptions (http://localhost:3000/packages/blaze.js?a5c324925e5f6e800a4c618d71caf2848b53bf51:1651:16)
    at http://localhost:3000/packages/blaze.js?a5c324925e5f6e800a4c618d71caf2848b53bf51:2928:66
    at Function.Template._withTemplateInstanceFunc (http://localhost:3000/packages/blaze.js?a5c324925e5f6e800a4c618d71caf2848b53bf51:3476:12)
    at wrapHelper (http://localhost:3000/packages/blaze.js?a5c324925e5f6e800a4c618d71caf2848b53bf51:2927:27)
    at Spacebars.call (http://localhost:3000/packages/spacebars.js?7bafbe05ec09b6bbb6a3b276537e4995ab298a2f:172:18)
    at Spacebars.mustacheImpl (http://localhost:3000/packages/spacebars.js?7bafbe05ec09b6bbb6a3b276537e4995ab298a2f:109:25)
    at Object.Spacebars.mustache (http://localhost:3000/packages/spacebars.js?7bafbe05ec09b6bbb6a3b276537e4995ab298a2f:113:39)
    at Template.velocityReports.HTML.SECTION.HTML.DIV.HTML.SPAN.HTML.A.href (http://localhost:3000/packages/velocity_html-reporter.js?9bd02910ea39c4b8833365509d30e6f81f4e39f0:314:24)
debug.js:41 

Exception in template helper: TypeError: Cannot read property 'state' of undefined
    at Object.Template.velocityReports.helpers.mirrorStatus (http://localhost:3000/packages/velocity_html-reporter.js?9bd02910ea39c4b8833365509d30e6f81f4e39f0:634:59)
    at bindDataContext (http://localhost:3000/packages/blaze.js?a5c324925e5f6e800a4c618d71caf2848b53bf51:2880:16)
    at Blaze._wrapCatchingExceptions (http://localhost:3000/packages/blaze.js?a5c324925e5f6e800a4c618d71caf2848b53bf51:1651:16)
    at http://localhost:3000/packages/blaze.js?a5c324925e5f6e800a4c618d71caf2848b53bf51:2928:66
    at Function.Template._withTemplateInstanceFunc (http://localhost:3000/packages/blaze.js?a5c324925e5f6e800a4c618d71caf2848b53bf51:3476:12)
    at wrapHelper (http://localhost:3000/packages/blaze.js?a5c324925e5f6e800a4c618d71caf2848b53bf51:2927:27)
    at Spacebars.call (http://localhost:3000/packages/spacebars.js?7bafbe05ec09b6bbb6a3b276537e4995ab298a2f:172:18)
    at Spacebars.mustacheImpl (http://localhost:3000/packages/spacebars.js?7bafbe05ec09b6bbb6a3b276537e4995ab298a2f:109:25)
    at Object.Spacebars.mustache (http://localhost:3000/packages/spacebars.js?7bafbe05ec09b6bbb6a3b276537e4995ab298a2f:113:39)
    at null._render (http://localhost:3000/packages/velocity_html-reporter.js?9bd02910ea39c4b8833365509d30e6f81f4e39f0:320:22)
ghost commented 9 years ago

@sircharleswatson Can you tell me what your previous version of velocity:html-reporter was.

ghost commented 9 years ago

This should already be fixed by https://github.com/meteor-velocity/html-reporter/commit/c7cf317a640ff730d61c2deb151a15de803a64fc in 0.7.3.

sircharleswatson commented 9 years ago

@Sanjo Went from 6.2 to 7.2

That was from the automatic update when updating cucumber. I didn't know there was a 7.3

Should xolvio:cucumber be updated to use v7.3?

ghost commented 9 years ago

Yes. But you can force a new version by adding the version constraint to the package in .meteor/packages. velocity:html-reporter@0.7.3.

(In my opinion test frameworks should not declare the dependency to a reporter. Then you don't have this kind of problems.)

sircharleswatson commented 9 years ago

@Sanjo That did it! no more errors :)