erisu opened a new pull request #204:
URL: https://github.com/apache/cordova-electron/pull/204


   ### Motivation and Context
   
   A mysterious computer glitch made me decide to remove as many references to 
other platforms.
   
   ### Description
   
   * `Api.js`
     * The `cordovaJs` location is not used anyways. Appears to be leftover 
code from other platforms but also not used in those platforms as well
     * The `cordovaJsSrc` location is not used anyways. Appears to be leftover 
code from other platforms but also not used in those platforms as well
   * Test fixture 
`tests/spec/fixtures/test-app-with-electron-plugin/plugins/cordova-plugin-device`
 was renamed to 
`tests/spec/fixtures/test-app-with-electron-plugin/plugins/cordova-plugin-sample`
     * This test fixure was also dummy down to a simple example.
     * Can later be used as guidance on making a plugin.
     * Tests were also updated to support the name change.
   * Any refence to `android` or `ios` platform in test code was removed. It 
was not tested or used.
   * Only 1 test case was left with `android` to test the case of an install 
not performed.
   
   ### Testing
   
   - `npm t`
   
   ### Checklist
   
   - [x] I've run the tests to see all new and existing tests pass
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to