17999 - Gather XUL regression tests to viewer test suite

Status: RESOLVED WORKSFORME

(Core :: XUL, defect, P2)

Description

[Chris Waterson]

From a private email exchange with karnaze/hyatt:

> > The general problem is that there are no  (there is only one xul test
> > and it may have a tree) table regression tests that have trees. To
> > prevent this from happening in the future, some tests need to get
> > added to the table regression tests. I think Hyatt mentioned some
> > tests before, but I can't remember.

> In layout\html\tests\table there are subdirectories core, bugs,
> marvin, viewer_tests, other. Each of these has a file called file_
> list.txt. To add a new test, select the appropriate subdirectory,
> put an entry in file_list.txt and put the test page in the
> subdirectory. To run the tests, "rtest baseline" will create a
> set of regression data files and "rtest verify" will compare against
> them. There is also an rtest.bat in the parent directory which
> cycles through the subdirectories.
>
> That is for windows. For Unix,  Kipp added corresponding shell
> scripts, but I have never used them.

Let's get this done! I have a couple of tree test cases, evaughan has a pile of
box test cases, i'm sure pink has toolbar stuff.

Comment 1

[Peter Trudelle]

Sounds good, let's target getting this done in M12. cc'ing folks mentioned.

Comment 2

[karnaze (gone)]

When doing this, please create new subdirectories tree, box, etc.

Comment 3

[leger]

This is a very good idea, but not mandatory for dogfood.  But impt to do.
Putting on PDT- radar.

Comment 4

[Peter Trudelle]

We could get to dogfood without it, sure, but I think we'll get there sooner,
and keep the quality up there better by doing this.  I don't know why this was
even marked dogfood, it is not a feature or a defect, just good engineering.
Leaving target M12, P2

Comment 5

[Peter Trudelle]

assigning to sdagley

Comment 6

[Paul MacQuiddy]

spam: changing qa contact from ckritzer -> paulmac for xul bugs

Comment 7

[Peter Trudelle]

moving to m13

Comment 8

[Peter Trudelle]

moving to m14

Comment 9

[leger]

BULK MOVE: Changing component from XUL to XP Toolkit/Widgets: XUL.  XUL 
component will be deleted.

Comment 10

[Steve Dagley]

Doesn't meet the BETA1 criteria, moving out of M14

Comment 11

[leger]

Putting dogfood in the keyword field.

Comment 12

[Peter Trudelle]

changing QA to jrgm. John, I doubt we're going to worry much about this in the 
current release, but could you own this issue in genreral, going forward?

Comment 13

[Steve Dagley]

Clearing the target milestone and assigning the bug to QA as I don't know who 
owns the test suite but I do know it isn't me.

Comment 14

[leger]

Putting on [dogfood-] radar.

Comment 15

[Mike Shaver (:shaver -- probably not reading bugmail closely)]

we have XUL tests now, automated even.