205747 - Add link to serializer-tests document from TestOutSinks.pl

Status: RESOLVED FIXED

(Core :: DOM: Serializers, defect)

Description

[Akkana Peck]

When the serializer tests fail in Tinderbox, they should print the url for the
new serializer test howto that I'm about to check in, which will be
http://www.mozilla.org/editor/serializer-tests.html
It includes a section on "What do I do if I broke the tests and need to fix
them, fast?" which I hope will help people diagnosing bustage that shows up from
Tinderbox.

Comment 1

[Akkana Peck]

Attachment: Patch: print the url

Print the url in case of error; also change CONVERSION TESTS to SERIALIZER
TESTS.

Comment 2

[Akkana Peck]

Add some potential review victims.  (Don't worry, the patch is trivial.)
Setting TM of 1.4 final since it's not a code change to mozilla, only to a
built-time test.

Comment 3

[Chris McAfee]

Comment on attachment 123434 [details] [diff] [review]
Patch: print the url

good idea to print the URL out, r=mcafee.

Comment 4

[Alec Flett]

Comment on attachment 123434 [details] [diff] [review]
Patch: print the url

sr=alecf

Comment 5

[Akkana Peck]

Comment on attachment 123434 [details] [diff] [review]
Patch: print the url

Requesting 1.4 approval since I've seen some pending serializer changes which
might land soon (I'm not sure whether those changes are 1.4 approved or not)
and this might help sheriffs or other tree helpers.

Comment 6

[(not reading, please use [e-mail] instead)]

Comment on attachment 123434 [details] [diff] [review]
Patch: print the url

test stuff, no brainer.

Comment 7

[Akkana Peck]

Fixed.