cwa-app-ios icon indicating copy to clipboard operation
cwa-app-ios copied to clipboard

Community build with MockENManager cannot disable Exposure detection

Open ndegendogo opened this issue 2 years ago • 1 comments

Avoid duplicates

  • [X] Bug is not mentioned in the FAQ
  • [X] Bug is specific for iOS only, for general issues / questions that apply to iOS and Android please raise them in the documentation repository
  • [ ] Bug is not already reported in another issue

Technical details

  • Device name: simulator iPhone SE
  • iOS version: iOS 14.3
  • App version: development version (branch: release/2.10.x as of commit 28aedac81cf81a66bec30ac7814c31f240d0edcf, 02 Sept 2021)

Describe the bug

Disable "Exposure Detection" is broken. The switch is triggered, but the observation of the status change is never triggered, and the status change is never distributed into the application.

Steps to reproduce the issue

  1. From Home tab press button "Exposure Logging Active"
  2. See the exposure logging status screen for enabled exposure logging. Both the artwork and the switch are consistent and show that exposure logging is enabled.
  3. press the switch.
  4. Now the switch shows "exposure logging disabled" but the artwork still shows enabled.
  5. Return to Home screen. Also here the switch still shows enabled exposure logging.
  6. Press again the button "Exposure Logging Active"
  7. Exposure logging status screen shows consistently enabled exposure logging, but is out of synch with the state of the MockExposureManager.

Expected behaviour

Artwork of exposure logging status screen should change when I disable the Exposure logging.

Possible Fix

Additional context

Simulator Screen Shot - iPhone SE (2nd generation) - 2021-09-02 at 10 43 23


Internal Tracking-ID: EXPOSUREAPP-9769

ndegendogo avatar Sep 02 '21 09:09 ndegendogo

It seems that the observer for the change of the state is never triggered. Not sure about the root cause. Can it be a race condition? Is there a bug in the implementation of the MockENManager?

ndegendogo avatar Sep 02 '21 09:09 ndegendogo

fixed with PR #5146

30mar avatar Apr 26 '23 12:04 30mar