search

TWiStErRob / net.twisterrob.libraries

1 stars 0 forks source link

AndroidX upgrade consistently freezes the emulator #41

Closed TWiStErRob closed 1 year ago

TWiStErRob commented 1 year ago

image

This is already a reduced trial, testDialogIsDisplayedForAppCompatAlert is disabled, but it still reproes.

TWiStErRob commented 1 year ago

Removing everything except 0,1 and last of potentialDialogs passes, no repro. Removing the middle 8 of 16 test cases in potentialDialogs passes, no repro.

TWiStErRob commented 1 year ago

Removing the 4-8 of 16 test cases in potentialDialogs fails, reproduced. Removing the 8-12 of 16 test cases in potentialDialogs fails, reproduced. Removing the 12-16 of 16 test cases in potentialDialogs fails, reproduced.

image suspicion: some kind of leak

TWiStErRob commented 1 year ago

Further confirmed by reproducing on a single test by double-ing the test parameters in potentialDialogs image

TWiStErRob commented 1 year ago

Does not reproduce when only testDialogIsDisplayedForAndroidAlert is enabled with double or triple parameters.

Does not reproduce when only testDialogIsDisplayedForAppCompatAlert is enabled with double or triple parameters.

Does not reproduce when only testDialogIsDisplayedForAndroidAlert AND testDialogIsDisplayedForAppCompatAlert is enabled with double or triple parameters.

Suspect those "no" tests.

Does not reproduce with testNoDialogIsDisplayedWhenActivityVisible OR testNoDialogIsDisplayedWhenActivityVisible_fail is enabled together with doubled params testDialogIsDisplayedForAppCompatAlert, BUT their order is swapped.

Does not reproduce when only the @Tests listed in the above screenshot are enabled, this is baffling.

TWiStErRob commented 1 year ago

At this point I gave up trialing and produced a reproducer by reducing the code to just a few lines: https://github.com/TWiStErRob/repros/commit/16ae30f21617b83b5eb611b8bfb3a242023d5654