Skip to content

Fedora CI: enable GUI tests #23403

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.

Sign up for GitHub

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

Jump to bottom
Draft
swt2c wants to merge 4 commits into
base: master
Choose a base branch
from
Draft

Fedora CI: enable GUI tests #23403

swt2c wants to merge 4 commits into from

Conversation

@swt2c
Copy link
Contributor

@swt2c swt2c commented Mar 30, 2023

No description provided.

@swt2c swt2c force-pushed the fedora_ci_gui_tests branch 5 times, most recently from 2246cfe to 19c5a13 Compare April 4, 2023 23:19
swt2c added 3 commits April 4, 2023 19:20
@swt2c
Fedora CI: enable GUI tests
7d0da8d
@swt2c
WebView WebKit2: Fix extension loading fallback
5f347f5 
Previously the extension loading fallback would only try the first
directory and then give up.
@swt2c
WebView tests: Fix Selection test with WebKit 2.40+
f09711d 
For some reason, calling HasSelection() immediately after SelectAll()
causes the web extension to get stuck under WebKit 2.40.  Instead of
sleeping, call wxYield() a few times to let the event loop run a bit.
@swt2c swt2c force-pushed the fedora_ci_gui_tests branch from 19c5a13 to d352f58 Compare April 11, 2023 02:09
@swt2c
Fix ListCtrl Visible test on Fedora
d6264de 
It seems the window size isn't sorted out quite yet when the test
starts, so GetCountPerPage() was returning one too few.
@swt2c swt2c force-pushed the fedora_ci_gui_tests branch from d352f58 to d6264de Compare April 11, 2023 23:37
@vadz
Copy link
Contributor

vadz commented May 10, 2025

@swt2c Do you plan to return to this?

@swt2c
Copy link
Contributor Author

swt2c commented May 11, 2025

Maybe eventually. I think the issue I ran into was that when I went to debug this locally, I ran into a bunch more test failures locally. I'm not really sure why. Perhaps HiDPI related or Wayland related?

In any event, feel free to close this PR if it's been open too long without resolution. 😁

@vadz
Copy link
Contributor

vadz commented May 11, 2025

We can keep it open, it doesn't really cost us anything, I just wasn't sure if this wasn't completely abandoned.

FWIW the tests using wxUIActionSimulator only run under X11, not Wayland, so you either need to run them with GDK_BACKEND=x11 (to run the tests) or WX_UI_TESTS=0 (to skip them). You also need to run the tests from the tests subdirectory (which is annoying and I keep meaning to fix this and postponing it...). But otherwise they are supposed to work, and do work for me.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@swt2c @vadz