added test demonstrating bug

cujojs / curl

curl.js is small, fast, extensible module loader that handles AMD, CommonJS Modules/1.1, CSS, HTML/text, and legacy scripts.

https://github.com/cujojs/curl/wiki

Other

1.89k stars 216 forks source link

added test demonstrating bug #233

Closed mmacaula closed 10 years ago

mmacaula commented 10 years ago

Just the tests to demonstrate a problem documented here: https://groups.google.com/forum/#!msg/cujojs/8mmcCY1z6UY/Zn-Rq3z7BdMJ

Let me know if this is not quite the correct way to do this, I don't pretend to know the source at all, but hopefully this accurately demonstrates the problem and will allow a fix to be verified.

unscriptable commented 10 years ago

Hey @mmacaula,

Sorry, I should have a CONTRIBUTING.md file that explains what to do. I'll manually pull this into my local repo, fix the problem, merge it into dev, and then push it back to github.

Thanks for the tests!

-- John

mmacaula commented 10 years ago

OK Great!

unscriptable commented 10 years ago

I manually merged this and pushed a fix in https://github.com/cujojs/curl/commit/fb64597c4ee1d2e537e2a1121d6ce38a84ebec97

Thanks @mmacaula!