linkedin / dustjs

Asynchronous Javascript templating for the browser and server
http://dustjs.com
MIT License
2.91k stars 479 forks source link

dustjs-helpers server test loads wrong dust #119

Closed rragan closed 12 years ago

rragan commented 12 years ago

Currently, dustjs/dustjs-helpers/test/jasmine-test/server/specRunner.js references

dust = require(dustjs-linkedin),

to get dust for testing. If you have been making changes and want to test them, you need dust from the latest build with your changes. Right now I see failures in my new tests since my new code is not present. If I change this line to

dust = require('../../../../lib/dust),

then my new tests pass fine. This seems like a correct change to me and is how the specRunner.js at the dust level is coded. Also the way the client test in dustjs-helpers is coded.

If it is OK, I will include it as part of my next pull request for the $idx/$len work.

jairodemorais commented 12 years ago

@rragan

The reference is fine, because we a re going to remove the herlpers folder in the next dust version, so it will be imposible to use dust with relative path.

if you have changed the dust code and you need to see it in the helpers just do this:

1) In the helper folder execute, npm uninstall dustjs-linkedin
2) npm install ../  (this command will install dustjs with your local changes)