[ 
https://issues.apache.org/jira/browse/CB-267?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13211954#comment-13211954
 ] 

Patrick Mueller commented on CB-267:
------------------------------------

There are test cases for CommonJS bits that we should probably support, here: 

bq. https://github.com/commonjs/commonjs/tree/master/tests

The license for the test cases is here:

bq. https://github.com/commonjs/commonjs/blob/master/docs/license.html.markdown

Proposal is to pull those into cordova-js in a new directory '{{cjs-tests}}' 
under the '{{thirdparty}}' directory.

We'll need to update the LICENSE file to indicate the licensing of that (and 
add the jasmine bits as a "drive by" fix).

To run the test cases, we'll need a separate loader/runner, as the tests have a 
distinct set of runtime requirements.  I've already built and used a 
loader/runner for this with modjewel, so I'll pull those in, I guess under 
'{{test}}' as '{{cjs-tests}}'.  Not sure if it's appropriate or desired to have 
these tests folded into some uber-test suite or not, given the different 
runtime set up, and the fact that these tests shouldn't need to be run all that 
often anyway - they are generally orthogonal to the existing cordova test cases.
In order to run the
                
> add test cases for require() and friends in codova-js
> -----------------------------------------------------
>
>                 Key: CB-267
>                 URL: https://issues.apache.org/jira/browse/CB-267
>             Project: Apache Callback
>          Issue Type: Improvement
>          Components: CordovaJS
>            Reporter: Patrick Mueller
>
> We should add some test cases to cordova-js to test the require() and friends 
> bits.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to