Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 398954

Summary:

IE7 has a red js test in VisibilityAnimationMixinTest

Product:

[RT] RAP

Reporter:

Tim Buschtoens <tbuschto>

Component:

RWT

Assignee:

Project Inbox <rap-inbox>

Status:

RESOLVED FIXED

QA Contact:

Severity:

normal

Priority:

CC:

ivan, tbuschto

Version:

2.0

Target Milestone:

2.0 RC2

Hardware:

All

OS:

All

Whiteboard:

Attachments:

Description	Flags
Fix for the test	tbuschto: review?

Description Tim Buschtoens

2013-01-24 04:59:38 EST

Here is the output:
testShellFlyInTopFinish failed:
'ObjectManager.getEntry(...).handler' ist Null oder kein Objekt
0 asserts succeeded.
Tests aborted!

Patch coming soon.

Comment 1 Tim Buschtoens

2013-01-24 05:05:23 EST

Created attachment 226025 [details]
Fix for the test

This is simply the result of a different test order in IE7 vs. IE9. The test testShellFlyOutDelayActive doesn't dispose a shell it created, which causes a red test if it's executed first.

Comment 2 Tim Buschtoens

2013-01-24 05:09:05 EST

NOTE: The test is only red in a "real" IE7, not in an IE9 in IE7 mode.

Comment 3 Ralf Sternberg

2013-01-24 11:47:15 EST

+1

Comment 4 Ivan Furnadjiev

2013-01-25 11:53:54 EST

It's OK with me too +1

Comment 5 Tim Buschtoens

2013-01-28 05:12:47 EST

Applied patch to master with commit 978eea51fa0130ad19de9c81ca2127288f7bd356