block until deprecated MapboxNavigation#resetTripSession() finishes

[LukasPaczos]

Description

Refs 562e8e7#r92415581.

[codecov]

Codecov Report

Merging #6705 (0c9e479) into main (79e3cb2) will increase coverage by 0.00%.
The diff coverage is 83.33%.

@@            Coverage Diff            @@
##               main    #6705   +/-   ##
=========================================
  Coverage     72.46%   72.46%           
+ Complexity     5415     5414    -1     
=========================================
  Files           764      764           
  Lines         29427    29432    +5     
  Branches       3491     3491           
=========================================
+ Hits          21323    21327    +4     
- Misses         6712     6713    +1     
  Partials       1392     1392           

Impacted Files	Coverage Δ
...ava/com/mapbox/navigation/core/MapboxNavigation.kt	70.63% <83.33%> (+0.07%)	⬆️

[VysotskiVadim]

@LukasPaczos , from the first glance it seems like it should always cause a deadlock:

1. User calls MapboxNavigation#resetTripSession() from a main thread
2. runBlocking stops and holds main thread until it receives callback from NN
3. NN does some heavy actions and posts callback to main thread via events queue
4. Event's queue aren't processes because main thread has stopped during step 2

I was surprised that it works and it turned out that NN calls the callback from their worker thread without rescheduling to the main thread, which is not typical for NN's API. Do you think it's safe to rely on this implementation detail of NN? I worry that if they change it, deprecated version of resetTripSession will always cause a deadlock.

[LukasPaczos]

Thanks for raising this. In the end no, we shouldn't really rely on this. This is a strong enough reason to try and find a different approach or work with Nav Native to leave the old function be, even if deprecated.

Tracking in NAVAND-984.

[LukasPaczos]

Adding a test as a safeguard that would block us from adopting a NN version that would break this logic - #6710.