You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cordova.apache.org by filmaj <gi...@git.apache.org> on 2017/06/27 22:38:08 UTC

[GitHub] cordova-android pull request #392: WIP: Run native android unit tests in CI

GitHub user filmaj opened a pull request:

    https://github.com/apache/cordova-android/pull/392

    WIP: Run native android unit tests in CI

    Still a work in progress, let's see how appveyor and travis react to this.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/filmaj/cordova-android ci-native-tests

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/cordova-android/pull/392.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #392
    
----
commit 90053eb9dfc03c6cdab31aab665324556c3358be
Author: filmaj <ma...@gmail.com>
Date:   2017-06-27T17:02:38Z

    CB-12950: lots of tweaks for end-to-end test runs, especially on CI:
    - rename npm tasks to reflect what they do (npm run unit-tests, npm run e2e-tests). main `npm test` runs linter, unit tests and e2e tests now.
    - locked jasmine down to ~2.6.0.
    - consolidate gitignores.
    - updated travis to run `npm test`. add android sdk installation to appveyor ci run.align android dpendencies across travis and appveyor. have appveyor install gradle. force gradle to version 3.4.1 in appveyor, as that seems to be the only version choco has. explicitly invoke sdkmanager to move license accepting process along.

commit 8ec72bee4af733ca049d59d18913ddb58913c770
Author: filmaj <ma...@gmail.com>
Date:   2017-06-27T22:37:07Z

    try to run native unit tests in both travis and appveyor.

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@cordova.apache.org
For additional commands, e-mail: dev-help@cordova.apache.org


[GitHub] cordova-android issue #392: WIP: Run native android unit tests in CI

Posted by codecov-io <gi...@git.apache.org>.
Github user codecov-io commented on the issue:

    https://github.com/apache/cordova-android/pull/392
  
    # [Codecov](https://codecov.io/gh/apache/cordova-android/pull/392?src=pr&el=h1) Report
    > Merging [#392](https://codecov.io/gh/apache/cordova-android/pull/392?src=pr&el=desc) into [master](https://codecov.io/gh/apache/cordova-android/commit/540929c6a0ced3d186ce9b1efea03c32403b89d1?src=pr&el=desc) will **not change** coverage.
    > The diff coverage is `n/a`.
    
    [![Impacted file tree graph](https://codecov.io/gh/apache/cordova-android/pull/392/graphs/tree.svg?token=q14nMf6C5a&src=pr&height=150&width=650)](https://codecov.io/gh/apache/cordova-android/pull/392?src=pr&el=tree)
    
    ```diff
    @@           Coverage Diff           @@
    ##           master     #392   +/-   ##
    =======================================
      Coverage   41.54%   41.54%           
    =======================================
      Files          16       16           
      Lines        1497     1497           
      Branches      279      279           
    =======================================
      Hits          622      622           
      Misses        875      875
    ```
    
    
    
    ------
    
    [Continue to review full report at Codecov](https://codecov.io/gh/apache/cordova-android/pull/392?src=pr&el=continue).
    > **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
    > `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
    > Powered by [Codecov](https://codecov.io/gh/apache/cordova-android/pull/392?src=pr&el=footer). Last update [540929c...576dc72](https://codecov.io/gh/apache/cordova-android/pull/392?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@cordova.apache.org
For additional commands, e-mail: dev-help@cordova.apache.org