It was discovered that we didn't have tests for the following statement in the specification:
The dependencies argument is optional. If omitted, it should default to ["require", "exports", "module"]. However, if the factory function's arity (length property) is less than 3, then the loader may choose to only call the factory with the number of arguments corresponding to the function's arity or length.
We created the tests as part of the "basic" spec. However, this test did not exist, so some libraries are not compatible with this part of the specification. https://github.com/amdjs/amdjs-tests/issues/14 contains some very good discussion around the issue. To go live with this, we have temporarily disabled "basic" compliance for the RequireJS and lsjs libraries. These libraries are still compliant with the original set of "basic" tests, but we do not want people to see the test suite as failing on account of a missed test on our part. RequireJS and lsjs will address these items and update their implemented object when resolved.
The second change resolves https://github.com/amdjs/amdjs-tests/issues/13 which was relying on universal config() for adjusting paths instead of just testing relative module IDs. The use of path creates a new test that says "if using config() to adjust the module's resolving URL, relative paths must remain relative to the module ID and not its URL". This new test config_paths_relative was turned on for all frameworks who are already passing. There was no frameworks who were not in compliance with this.
Before committing these, however, we wanted to ensure there was no further discussion on these items.
The proposed pull request is at
https://github.com/amdjs/amdjs-tests/pull/17
Hopefully we can close discussion by Sunday or sooner and move forward.
--Jakob