test_resize_move_windows.xul fails on an ubuntu 12.04 VM, causes 45 other failures

RESOLVED FIXED

Status

()

RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: jmaher, Assigned: jmaher)

Tracking

(Blocks: 1 bug, {qawanted})

Trunk
All
Linux
qawanted
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

6 years ago
we are going to be running unittests on EC2 VMs in the near future and test_resize_move_windows.xul (from mochitest-chrome) fails consistently:
https://tbpl.mozilla.org/php/getParsedLog.php?id=19102371

This test times out and then about 45 other failures occur as a result of this failure.  I have verified this by removing this test and running the mochitest-chrome suite and only get a few failures.

If this test requires real hardware, please let us know, otherwise, we need to get this fixed or disabled.
I do not think it requires real hardware. Only real window manager.

Actually, I can reproduce this on my new Fedora. When I wrote the test, I wasn't using the same window manager. I will try to have a look and fix that next week.

This said, we run all our tests in old version of Fedora and I always thought that most of them wouldn't work well in another environment. Is this the only issue you have with running our test suite in a quite recent Ubuntu?
Assignee: nobody → mounir
Keywords: qawanted
Hardware: x86_64 → All
(Assignee)

Updated

6 years ago
Blocks: 834725
(Assignee)

Comment 2

6 years ago
I am tracking the failures in bug 834725, but there are very few issues as it stands.  I only have 3 other bugs to file.  

thanks for looking at this next week!
Blocks: 837020
(Assignee)

Comment 3

6 years ago
:mounir, any luck with this?
I had a look but I wasn't able to identify the problem. For the moment, it seems that a request to move the window is simply ignored by GTK.

BTW, this test is currently disabled on Linux.
(Assignee)

Comment 5

6 years ago
thanks for taking a look at this last week.  The mochitest-plain test is disabled for linux, but the mochitest-chrome test is still active:
http://dxr.mozilla.org/mozilla-central/dom/tests/mochitest/chrome/Makefile.in.html#l35

Should we disable the mochitest-chrome one for linux as we do for plain?
http://dxr.mozilla.org/mozilla-central/dom/tests/mochitest/bugs/Makefile.in.html#l145
(In reply to Mounir Lamouri (:mounir) from comment #4)
> For the moment, it
> seems that a request to move the window is simply ignored by GTK.

I wouldn't be surprised if the window manager ignored a request to move the window off the screen.

http://hg.mozilla.org/mozilla-central/annotate/de946331d886/dom/tests/mochitest/chrome/test_resize_move_windows.xul#l50
(Assignee)

Comment 7

6 years ago
so what I see happening here is that we resize the window to some really small amount and there is no focus on the window.  If I click the window it moves further along in the test, but it doesn't pass.  I am really thinking we need to disable this for gtk2.
(Assignee)

Comment 8

6 years ago
Created attachment 710268 [details] [diff] [review]
disable the chrome test_resize_windows.xul test for linux (1.0)
Assignee: mounir → jmaher
Status: NEW → ASSIGNED
Attachment #710268 - Flags: review?(mounir)
Attachment #710268 - Flags: review?(mounir) → review+
(Assignee)

Comment 9

6 years ago
Thanks for the review!

https://hg.mozilla.org/mozilla-central/rev/469333ea459c
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Comment on attachment 710268 [details] [diff] [review]
disable the chrome test_resize_windows.xul test for linux (1.0)

>+ifneq (Linux,$(OS_ARCH))
>+MOCHITEST_FILES += \
>+		test_resize_move_windows.xul \
>+		$(NULL)

This should be MOCHITEST_CHROME_FILES (getting timeout on m-c in this test):
https://tbpl.mozilla.org/php/getParsedLog.php?id=19575697&tree=Firefox
Pushed followup:
https://hg.mozilla.org/mozilla-central/rev/fe1600b22c77

(I've very almost made exactly the same mistake - I have a file to copy/paste the makefile disable-on-one-platform boilerplate, and is easy to forget to substitute the test type :-))
Depends on: 854117
No longer depends on: 854117
You need to log in before you can comment on or make changes to this bug.