Skip to content
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

Test test_layout_load fails on i3wm (tiling window manager) #569

Open
MartinSavc opened this issue Jun 13, 2023 · 1 comment
Open

Test test_layout_load fails on i3wm (tiling window manager) #569

MartinSavc opened this issue Jun 13, 2023 · 1 comment

Comments

@MartinSavc
Copy link

Running the tests gives the following failure under i3wm:
FAILED envisage/ui/tasks/tests/test_tasks_application.py::TestTasksApplication::test_layout_load - AssertionError: Tuples differ: (956, 1029) != (492, 743)

The issue is that any tiling window manager will resize any non-floating window to a shape fitting the current layout. This will cause an error in the above test, where it expects a specific window size.

This is nothing but a minor nuisance, very specific to a particular desktop setup and I think incredibly hard to properly fix. But I though I'd let you know anyways, maybe someone will know how to fix this issue, or maybe it's relevant to another problem you might have down the road.

@mdickinson
Copy link
Member

Thank you for the report; it's appreciated. ETS development isn't a focus right now, so please be aware that it may be some time before we're able to get to this.

Possibly we should replace the test with something that simply verifies that the right dimensions are being supplied when creating the relevant panes/windows, rather than measuring the size of the resulting window.

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

No branches or pull requests

2 participants