search

aerogear / aerogear-android-sdk

AeroGear Services Android SDK
Apache License 2.0
8 stars 29 forks source link

Split of unit and integration tests execution #270

Closed trepel closed 6 years ago

trepel commented 6 years ago

Motivation

Issue https://github.com/aerogear/aerogear-android-sdk/issues/170 (see also discussion there)

Test suite classes have been removed. All the unit test now are in unit package. CONTRIBUTING.md has been updated to describe how to run both the unit and integration tests.

Tests are now annotated and based on the annotation it is decided whether the test is unit or integration test.

Note that the modifications have only been done for core module. Other modules (auth, push, etc) only contain unit tests so this PR is not necessary for them. If you agree with the modifications I am happy to update other modules as well though.

Aditional notes

I believe the https://github.com/aerogear/aerogear-android-sdk/commit/c67c0503a5d75ba754ef1475f84c42c69e2ad49b can be reverted now since test task now only run unit tests. I can send another PR if there is an agreement on that.

How to test

Just follow the CONTRIBUTING.md

trepel commented 6 years ago

Thanks for review @danielpassos , I'll merge it some time today and I will also prepare another PR for reverting https://github.com/aerogear/aerogear-android-sdk/commit/c67c0503a5d75ba754ef1475f84c42c69e2ad49b