### Pro
- Runtime: 217s -> 2s (on my machine)
- No dependency on the network
- Easier tests

### Con
- Code coverage caused by this test is considerably smaller
    - but it has no big overall impact; see below

Test names and comments will still have to be edited a bit if we want to merge 
this, but I first wanted to know what you think about this change.

|          |  % Stmts | % Branch |  % Funcs |  % Lines |
|----------|----------|----------|----------|----------|
| Before   |    80.29 |    75.93 |    72.66 |     80.8 |
| After    |    80.23 |     75.6 |    72.66 |    80.73 |


[ Full content available at: https://github.com/apache/cordova-lib/pull/665 ]
This message was relayed via gitbox.apache.org for [email protected]

Reply via email to