Skip to content

Fix adjusting firefox config #239

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

okurz
Copy link
Member

@okurz okurz commented Apr 22, 2025

"prepare_firefox_autoconfig" needs a text terminal to type into. It
seems at some time in the past a regression was introduced that the
firefox config is not typed into a text terminal anymore after any
previous test module did not end up on the text terminal anymore.

This fixes the entering of configuration by explicitly switching to the
root console. This should fix the sporadic issue of firefox asking in a
first-time wizard of confirmation instead of showing the selected page
that the openQA test expects.

Related progress issue: https://progress.opensuse.org/issues/181187

"prepare_firefox_autoconfig" needs a text terminal to type into. It
seems at some time in the past a regression was introduced that the
firefox config is not typed into a text terminal anymore after any
previous test module did not end up on the text terminal anymore.

This fixes the entering of configuration by explicitly switching to the
root console. This should fix the sporadic issue of firefox asking in a
first-time wizard of confirmation instead of showing the selected page
that the openQA test expects.

Related progress issue: https://progress.opensuse.org/issues/181187
@okurz
Copy link
Member Author

okurz commented Apr 22, 2025

I guess I was wrong. The firefox config is already adjusted earlier. Probably needs to be extended for the new dialog.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants