Remove unused method from DeliveryService.

[lucaslabari]

Goal

• Removed sendEvent() method from DeliveryService because it was not being called.
• Renamed sendLogs() to sendLog()

Testing

Updated unit tests.

[codecov]

Codecov Report

Merging #90 (37ea394) into master (7b2e011) will increase coverage by 0.00%.
The diff coverage is 50.00%.

Additional details and impacted files

@@           Coverage Diff           @@
##           master      #90   +/-   ##
=======================================
  Coverage   76.02%   76.02%           
=======================================
  Files         314      314           
  Lines       10076    10075    -1     
  Branches     1457     1457           
=======================================
  Hits         7660     7660           
+ Misses       1827     1826    -1     
  Partials      589      589           

Files	Coverage Δ
...ce/android/embracesdk/event/EmbraceRemoteLogger.kt	93.50% <100.00%> (ø)
...mbracesdk/comms/delivery/EmbraceDeliveryService.kt	62.09% <0.00%> (+0.49%)	⬆️

[bidetofevil]

LGTM. Digging deeper into the code, I wonder if we can remove some of these methods in DeliveryService whose implementation is just calling ApiService - and just have the the caller consume the ApiService directly?

[lucaslabari]

LGTM. Digging deeper into the code, I wonder if we can remove some of these methods in DeliveryService whose implementation is just calling ApiService - and just have the the caller consume the ApiService directly?

Yeap, that would be a good think to do so we remove the deliveryService from being the middleman.