[opensuse]Do not launch firefox from xterm in firefox_audio
With https://github.com/os-autoinst/os-autoinst-distri-opensuse/pull/3327 firefox_audio now launch firefox from xterm, it is not normal behavior from customer,
it should launch firefox from desktop launcher like what firefox_audio test did in the past.
#3 Updated by vsvecova almost 5 years ago
- Status changed from New to In Progress
- % Done changed from 0 to 90
I have opened a pull request that changes the firefox_audio behavior back to launching firefox directly from launcher.
SLE12 SP2: http://dreamyhamster.suse.cz/tests/800#step/firefox_audio/9
SLE12 SP3: http://dreamyhamster.suse.cz/tests/799#step/firefox_audio/9
#4 Updated by coolo almost 5 years ago
I would close this and the PR with Rejected - the generic function is very handy to verify firefox is really stopped before we start the next firefox test. This might be a bit unlike what users do - but then again, users don't start firefoxes with fresh configs 10 times in a row either.
#5 Updated by vsvecova almost 5 years ago
So if I understand this correctly, this is mostly a conflict between the authenticity of the test behavior on one side, and ease of test debugging and maintenance on the other side.
Personally, I can understand the reasoning behind both arguments, although I would tend to stand by coolo's suggestion.
Do we have any generic guidelines as to which approach should take the preference?