danwrong / loadrunner

Simple, flexible and sane JavaScript loader and build tool for browsers.
MIT License
277 stars 40 forks source link

AMD Module qunit test is failing #10

Closed kennethkufluk closed 13 years ago

kennethkufluk commented 13 years ago

"AMD support: should load an AMD module with relative dependencies." fails.

mzsanford commented 13 years ago

I'm also seeing this.

loaded mod with relative dependencies
Expected:   
    "AMD success"
Result: 
    "success"
Source: 
    at file://localhost/Users/matt/src/external/loadrunner/test/test.html:342:15

It seems to be loading modb instead of amd/modb. Since this test is for relative includes in AMD it seems to point to a real bug.

kennethkufluk commented 13 years ago

I fixed it. This was a problem since I changed the context of the module to be window instead of the module itself.

The AMD spec doesn't specify what the context should be, so this might conflict with other loaders.