Add more tests to EmbraceApiService

[bidetofevil]

Goal

Add more tests for EmbraceApiService so that we can more safely refactor it and ApiClientImpl

Testing

Added happy path tests - additional tests will be added to handle the boundary cases.

[bidetofevil]

Current dependencies on/for this PR:

• master
  • PR Cleanup ApiClient and add more tests #59
    • PR Add more tests to EmbraceApiService #60 👈

This stack of pull requests is managed by Graphite.

[codecov]

Codecov Report

Merging #60 (c459bce) into hho/apiservice-cleanup (bfc8311) will increase coverage by 0.72%.
The diff coverage is 100.00%.

Additional details and impacted files

@@                    Coverage Diff                     @@
##           hho/apiservice-cleanup      #60      +/-   ##
==========================================================
+ Coverage                   75.00%   75.72%   +0.72%     
==========================================================
  Files                         307      307              
  Lines                       10053    10052       -1     
  Branches                     1455     1455              
==========================================================
+ Hits                         7540     7612      +72     
+ Misses                       1933     1846      -87     
- Partials                      580      594      +14     

Files	Coverage Δ
...brace/android/embracesdk/comms/api/CachedConfig.kt	100.00% <100.00%> (ø)
.../android/embracesdk/comms/api/EmbraceApiService.kt	76.69% <100.00%> (+49.82%)	⬆️
...esdk/comms/delivery/EmbraceDeliveryRetryManager.kt	73.97% <100.00%> (ø)
.../android/embracesdk/config/EmbraceConfigService.kt	85.44% <100.00%> (ø)

... and 3 files with indirect coverage changes

[lucaslabari]

LGTM