Bug 536564 - Test failures on Mac OS X 10.13 from I20180628-0230
Summary: Test failures on Mac OS X 10.13 from I20180628-0230
Status: CLOSED WONTFIX
Alias: None
Product: Platform
Classification: Eclipse Project
Component: SWT (show other bugs)
Version: 4.9   Edit
Hardware: PC Mac OS X
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Platform-SWT-Inbox CLA
QA Contact:
URL:
Whiteboard: stalebug
Keywords:
Depends on:
Blocks: 537658
  Show dependency tree
 
Reported: 2018-07-02 01:41 EDT by Sravan Kumar Lakkimsetti CLA
Modified: 2022-02-06 18:20 EST (History)
3 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Sravan Kumar Lakkimsetti CLA 2018-07-02 01:41:35 EDT
There are two failures in SWT related to active shell in Mac OS X.

The test machine is changed from Mac OSX 10.11 to 10.13 on 27th June
Comment 1 Eclipse Genie CLA 2018-07-02 07:06:55 EDT
New Gerrit change created: https://git.eclipse.org/r/125329
Comment 3 Lakshmi P Shanmugam CLA 2018-07-05 03:14:10 EDT
The tests pass locally now but they are still failing on the test machine. (http://download.eclipse.org/eclipse/downloads/drops4/I20180704-0805/testresults/html/org.eclipse.swt.tests_ep49I-unit-mac64_macosx.cocoa.x86_64_8.0.html)

I'll increase the timeouts and try again.
Comment 4 Eclipse Genie CLA 2018-07-05 06:02:45 EDT
New Gerrit change created: https://git.eclipse.org/r/125574
Comment 6 Lakshmi P Shanmugam CLA 2018-07-10 02:31:15 EDT
Increasing the timeouts didn't fix the problem, the tests are still failing on the Test machine.
Comment 7 Andrey Loskutov CLA 2018-07-10 02:49:15 EDT
Lakshmi, can it be that we started to have test failures on Mac *and* Windows at same time?
Comment 8 Lakshmi P Shanmugam CLA 2018-07-10 05:48:32 EDT
(In reply to Andrey Loskutov from comment #7)
> Lakshmi, can it be that we started to have test failures on Mac *and*
> Windows at same time?

Andrey, I don't see any Windows failures for SWT. Which failures are you referring to?
Comment 9 Andrey Loskutov CLA 2018-07-16 02:42:39 EDT
(In reply to Lakshmi Shanmugam from comment #8)
> (In reply to Andrey Loskutov from comment #7)
> > Lakshmi, can it be that we started to have test failures on Mac *and*
> > Windows at same time?
> 
> Andrey, I don't see any Windows failures for SWT. Which failures are you
> referring to?

Failures on Windows, but I've found they are unrelated, see bug 453954 comment 6 and comment 7.

Looking on the Mac specific tests failures, I believe they are all due the not active shell: "Application does not have an active window". We had this in the past if some "Mac update" window appears during the test. Can we check that?
Comment 10 Lakshmi P Shanmugam CLA 2018-08-03 08:41:51 EDT
(In reply to Andrey Loskutov from comment #9)
> (In reply to Lakshmi Shanmugam from comment #8)
> > (In reply to Andrey Loskutov from comment #7)
> > > Lakshmi, can it be that we started to have test failures on Mac *and*
> > > Windows at same time?
> > 
> > Andrey, I don't see any Windows failures for SWT. Which failures are you
> > referring to?
> 
> Failures on Windows, but I've found they are unrelated, see bug 453954
> comment 6 and comment 7.
> 
> Looking on the Mac specific tests failures, I believe they are all due the
> not active shell: "Application does not have an active window". We had this
> in the past if some "Mac update" window appears during the test. Can we
> check that?

I verified that there is no dialog open on the machine.
Comment 11 Eclipse Genie CLA 2018-08-16 03:00:32 EDT
New Gerrit change created: https://git.eclipse.org/r/127475
Comment 13 Lakshmi P Shanmugam CLA 2018-08-17 07:27:50 EDT
I added code to sleep and capture screenshots in the failing tests. The screenshots [1] & [2] show that the shells are not active, but there is no other active shell. So, I'm not sure why.
The screenshots captured locally show the shells be tested are active.

[1] http://download.eclipse.org/eclipse/downloads/drops4/I20180816-2000/testresults/ep49I-unit-mac64_macosx.cocoa.x86_64_8.0/org.eclipse.swt.tests.junit.Test_org_eclipse_swt_widgets_Display.test_setActive2.png
[2] http://download.eclipse.org/eclipse/downloads/drops4/I20180816-2000/testresults/ep49I-unit-mac64_macosx.cocoa.x86_64_8.0/org.eclipse.swt.tests.junit.Test_org_eclipse_swt_widgets_Shell.test_setActive1.png

The screenshots are captured by a AWT application. Will remove the code and try again to ensure that it's not interfering with the shell activation.
Comment 14 Eclipse Genie CLA 2018-08-17 08:30:31 EDT
New Gerrit change created: https://git.eclipse.org/r/127576
Comment 16 Eclipse Genie CLA 2018-11-09 00:25:07 EST
New Gerrit change created: https://git.eclipse.org/r/132162
Comment 18 Lakshmi P Shanmugam CLA 2018-11-13 03:37:44 EST
(In reply to Eclipse Genie from comment #17)
> Gerrit change https://git.eclipse.org/r/132162 was merged to [master].
> Commit:
> http://git.eclipse.org/c/platform/eclipse.platform.swt.git/commit/
> ?id=c7c5b3cf1f6cd1c15bb48f636ae3cf05c6ed047c

Reordering fixes the local test failure in Shell.setActive() failure on 10.14.
Comment 19 Eclipse Genie CLA 2019-12-04 05:49:19 EST
New Gerrit change created: https://git.eclipse.org/r/153782
Comment 21 Lakshmi P Shanmugam CLA 2019-12-04 06:13:17 EST
(In reply to Eclipse Genie from comment #20)
> Gerrit change https://git.eclipse.org/r/153782 was merged to [master].
> Commit:
> http://git.eclipse.org/c/platform/eclipse.platform.swt.git/commit/
> ?id=e6caf796ec280c52a246b71ace1c685ea0b7c45e

The commit disables the 2 failing tests on Mac.
Comment 22 Eclipse Genie CLA 2022-02-06 18:20:57 EST
This bug hasn't had any activity in quite some time. Maybe the problem got resolved, was a duplicate of something else, or became less pressing for some reason - or maybe it's still relevant but just hasn't been looked at yet. As such, we're closing this bug.

If you have further information on the current state of the bug, please add it and reopen this bug. The information can be, for example, that the problem still occurs, that you still want the feature, that more information is needed, or that the bug is (for whatever reason) no longer relevant.

--
The automated Eclipse Genie.