Webmediaplayer error when running tests

medic / cht-conf-test-harness

A test harness for configurations of Community Health Toolkit applications

6 stars 3 forks source link

Webmediaplayer error when running tests #235

Open jonathanbataire opened 1 year ago

jonathanbataire commented 1 year ago

Sometimes run into the following error when running thess tests. Seem to be a memory leak bug in chromium

harness v2.4.0

Can not replicate in v3.0.10


ConsoleMessage {
_type: 'error',
_text: 'Blocked attempt to create a WebMediaPlayer as there are too many WebMediaPlayers already in existence. See crbug.com/1144736#c27',
_args: [],
_stackTraceLocations: [ [Object] ]
},

jonathanbataire commented 1 year ago

recently been working on this PR https://github.com/medic/config-ug-vht/pull/803 which introduces a geopoint widget in forms this has made this issue more persistent. Found some solutions

problem:

Rapidly referencing harness object (like in a loop), while filling in forms with widgets, without restarting harness.

solution:

some tests were badly written not optimal. rewrote these for optimization.
group tests in nested describes where i can stop and start harness in the before and after hooks between problematic tests

kennsippell commented 1 year ago

Hey @jonathanbataire . I think harness.clear() in between tests should be sufficient between tests. Hopefully you don't actually have to start/stop. Are you finding this to be the case?

jonathanbataire commented 1 year ago

Hey @jonathanbataire . I think harness.clear() in between tests should be sufficient between tests. Hopefully you don't actually have to start/stop. Are you finding this to be the case?

wasnt enough i think, its always been in the beforeEach but would still fail ended up splitting the problematic tests into separate test suites about 2 files