search

karma-runner / karma-commonjs

A Karma plugin. Test CommonJS modules.
MIT License
73 stars 30 forks source link

karma-commonjs doesn't seem to be able to follow symlinks #34

Open scamden opened 10 years ago

scamden commented 10 years ago

i have symlink from node_modules/@grid to a src folder that has util/index.js in it. works fine for node and browserify, but with karma-commonjs i get:

getting Error: Could not find module '@grid/util' from '/Users/scamden/projects/riq-grid/src/modules/cell-scroll-model/index.js' at /Users/scamden/projects/riq-grid/node_modules/karma-commonjs/client/commonjs_bridge.js:91

pkozlowski-opensource commented 10 years ago

@scamden I'm traveling right now so won't be able to look into this issue for another 3 weeks or so. If you want to speed up things it would be great if you could dig into the issue a bit - the CommonJS plugin is fairly small and should be easy to follow. Ping me if you've got more detailed questions.

scamden commented 10 years ago

i would love to normally but i was just testing out commonjs as an alternative to browserifying a bundle for tests. i'm sticking with browserify for now (well really watchify), but just wanted to flag this for you guys.

On 24 September 2014 04:27, Pawel Kozlowski notifications@github.com wrote:

@scamden https://github.com/scamden I'm traveling right now so won't be able to look into this issue for another 3 weeks or so. If you want to speed up things it would be great if you could dig into the issue a bit - the CommonJS plugin is fairly small and should be easy to follow. Ping me if you've got more detailed questions.

— Reply to this email directly or view it on GitHub https://github.com/karma-runner/karma-commonjs/issues/34#issuecomment-56656631 .

Sterling Camden Software Engineer 502 Emerson Street | Palo Alto, CA 94301 303.520.2968 | sterling@relateiq.com

See RelateIQ in the Wall Street Journal http://online.wsj.com/news/articles/SB10001424127887323949904578539983425941490, TechCrunch http://techcrunch.com/2013/06/12/relateiq-launches-with-29m-from-formation-8-dustin-moskovitz-and-more-to-be-your-next-gen-relationship-manager/, Forbes http://www.forbes.com/sites/michaelsimmons/2014/01/14/the-one-thing-you-should-do-after-meeting-anyone-new/, and More http://blog.relateiq.com/tag/relateiqinthenews/!

peter-mouland commented 9 years ago

I am having this issue too. A project i have come onto follows browserify sym-link method has this been looked into?

brunobasto commented 7 years ago

Any updates on this?

fernandosouza commented 7 years ago

Any updates on this?