492482 - [Linux] mochitest-a11y: content/a11y/accessible/test_aria_role_grid.html needs to report 'todo'

Status: VERIFIED FIXED

(Core :: Disability Access APIs, defect)

Description

[Serge Gautherie (:sgautherie)]

{
http://tinderbox.mozilla.org/showlog.cgi?log=Firefox/1242092605.1242099042.20956.gz&fulltext=1
Linux mozilla-central unit test on 2009/05/11 18:43:25

*** 141 INFO Running chrome://mochikit/content/a11y/accessible/test_aria_role_grid.html...
*** 142 INFO TEST-KNOWN-FAIL | chrome://mochikit/content/a11y/accessible/test_aria_role_grid.html | [SimpleTest.report()] No checks actually run.
}

Comment 1

[alexander :surkov (:asurkov)]

Attachment: patch

Comment 2

[Marco Zehe (:MarcoZ)]

Comment on attachment 376854 [details] [diff] [review]
patch

r=me

Comment 3

[Serge Gautherie (:sgautherie)]

Comment on attachment 376854 [details] [diff] [review]
patch

>+      if (LINUX)
>+        ok(true, "No tests on linux because of different hierarchies.");

Nit:
that's the idea;
yet, I would argue that, if you plan on doing Linux later, this should better be a todo(),
or, if you don't, you could do this earlier in the test.

Comment 4

[alexander :surkov (:asurkov)]

thanks, todo is great here

Comment 5

[Serge Gautherie (:sgautherie)]

http://tinderbox.mozilla.org/showlog.cgi?log=Firefox3.5/1242134886.1242140795.18418.gz&fulltext=1
Linux mozilla-1.9.1 unit test on 2009/05/12 06:28:06

Comment 6

[Serge Gautherie (:sgautherie)]

(In reply to comment #5)
> http://tinderbox.mozilla.org/showlog.cgi?log=Firefox3.5/1242134886.1242140795.18418.gz&fulltext=1
> Linux mozilla-1.9.1 unit test on 2009/05/12 06:28:06

Forget this comment: wrong bug.

Comment 7

[alexander :surkov (:asurkov)]

checked in, http://hg.mozilla.org/mozilla-central/rev/04d2e5a437dc

Comment 8

[Serge Gautherie (:sgautherie)]

{
http://tinderbox.mozilla.org/showlog.cgi?log=Firefox/1242190837.1242195491.4011.gz&fulltext=1
Linux mozilla-central unit test on 2009/05/12 22:00:37

*** 141 INFO Running chrome://mochikit/content/a11y/accessible/test_aria_role_grid.html...
*** 142 INFO TEST-KNOWN-FAIL | chrome://mochikit/content/a11y/accessible/test_aria_role_grid.html | No tests on linux because of different hierarchies.
}

V.Fixed