Fix ServiceTracker race

[jeffdiclemente]

The race occurs when a customizer is still running and the ServiceTracker object is in the process of being destroyed.

The fix is to have ServiceTracker::Close unregister its service listener and then wait for any existing service events to finish processing before cleaning up and calling untrack.

Signed-off-by: The MathWorks, Inc. jdicleme@mathworks.com

[codecov-commenter]

Codecov Report

Merging #558 (6338684) into development (0a3d4f7) will increase coverage by 0.40%.
The diff coverage is 85.52%.

@@               Coverage Diff               @@
##           development     #558      +/-   ##
===============================================
+ Coverage        89.79%   90.20%   +0.40%     
===============================================
  Files              234      234              
  Lines             9480     9606     +126     
===============================================
+ Hits              8513     8665     +152     
+ Misses             967      941      -26     

Impacted Files	Coverage Δ
...eclarativeServices/src/manager/ConcurrencyUtil.hpp	90.00% <ø> (-6.30%)	⬇️
...ativeServices/src/manager/states/CCActiveState.hpp	66.66% <ø> (ø)
...de/cppmicroservices/detail/BundleAbstractTracked.h	100.00% <ø> (ø)
...ework/include/cppmicroservices/detail/ScopeGuard.h	100.00% <ø> (ø)
.../cppmicroservices/detail/BundleAbstractTracked.tpp	82.19% <64.28%> (+8.21%)	⬆️
...ork/include/cppmicroservices/detail/CounterLatch.h	86.95% <86.95%> (ø)
...include/cppmicroservices/detail/TrackedService.tpp	75.80% <86.95%> (+9.84%)	⬆️
...ompendium/DeclarativeServices/src/SCRActivator.cpp	84.09% <100.00%> (+2.03%)	⬆️
...ativeServices/src/manager/states/CCActiveState.cpp	68.57% <100.00%> (-2.49%)	⬇️
...include/cppmicroservices/detail/ServiceTracker.tpp	80.59% <100.00%> (+8.62%)	⬆️
... and 28 more