TestReview
This page is an archive of the previous Group's wiki from http://www.w3.org/Graphics/SVG/Group/wiki.
How to use this page
When a test has been created and requires reviewing the Allocation Table must be updated. The following test details must be added to the Allocation Table: Test Name, Test Owner, Allocated Reviewer. If a reviewer for the test has not been found the test should still be added to the Allocation Table.
Once a test has been approved, then its entry in the Allocation Table must be marked as done.
Test Review allocation for SVG 1.1
Allocation Table
| Test | Owner | Reviewer(s) |
| masking-path-07-b.svg | ED | [Reviewer] |
Test Review allocation for SVG Tiny 1.2
NOTE: This table should not be modified, this page has moved to the new public wiki: http://www.w3.org/Graphics/SVG/WG/wiki/Tiny_12
Allocation Table
| Test | Owner | Reviewer(s) | Comments |
| udom-glob-202-t.svg | AE | [Reviewer] | |
| udom-glob-203-t.svg | AE | [Reviewer] | |
| udom-glob-204-t.svg | AE | [Reviewer] | |
| struct-use-205-t.svg | ED | [Reviewer] | |
| udom-svg-237-t.svg | AE | CM | Done. The string “Testing TraitAccess methods on an element removed from the DOM tree...” is displayed, but that really only applies to those tests on animm1, not animm2.Sub-tests b09 and b10 should be expecting NOT_SUPPORTED_ERR exceptions instead of NO_MODIFICATION_ALLOWED_ERR, according to the “Accessing rules for animation related elements” section of the uDOM appendix.The display is quite cluttered with messages shown for each sub-test; maybe this could be changed to highlight those messages from failed sub-tests in red, or perhaps just omit the messages from successful sub-tests? When running this test in Opera, text in the two columns overlap (and messages fall outside the test frame),but maybe that’s not too important when they’re just error messages. |
| udom-svg-236-t.svg | AE | [Reviewer] | |
| udom-svg-235-t.svg | AE | [Reviewer] | |
| udom-svg-234-t.svg | AE | [Reviewer] | |
| udom-svg-233-t.svg | AE | [Reviewer] | |
| udom-svg-232-t.svg | AE | [Reviewer] | |
| udom-svg-231-t.svg | AE | AG | |
| udom-svg-230-t.svg | AE | AG | |
| udom-svgpoint-201-t.svg | AE | ED | |
| udom-svgpoint-202-t.svg | AE | ED | |
| udom-svg-229-t.svg | AE | [Reviewer] | |
| udom-svg-228-t.svg | AE | [Reviewer] | |
| udom-svg-227-t.svg | AE | [Reviewer] | |
| udom-svg-226-t.svg | AE | [Reviewer] | |
| udom-svglocatable-201-t.svg | AE | ED | |
| udom-svglocatable-202-t.svg | AE | ED | |
| udom-svglocatable-203-t.svg | AE | ED | |
| udom-svgmatrix-201-t.svg | AE | ED | |
| udom-svgmatrix-202-t.svg | AE | ED | |
| udom-svgmatrix-203-t.svg | AE | ED | |
| udom-svgmatrix-204-t.svg | AE | ED | |
| udom-svgmatrix-205-t.svg | AE | ED | |
| udom-svgmatrix-206-t.svg | AE | ED | |
| udom-svgpath-201-t.svg | AE | AN | Done. I changed a few id to xml:id and changed setAttributeNS to setFloatTrait where applicable (for numbers) |
| udom-svgpath-202-t.svg | AE | AN | Done. I changed a few id to xml:id and changed setAttributeNS to setFloatTrait where applicable (for numbers) |
| udom-svgrect-201-t.svg | AE | AN | Done. Fixed invalid ev namespace declaration, I changed a few id to xml:id and changed setAttributeNS to setFloatTrait where applicable (for numbers) |
| udom-svgcolor-201-t.svg | AE | CL | |
| udom-svg-225-t.svg | ED | AE | Done. Even if subtest 1 fails, the test indicates 'passed' on the bottom. Test uses xe: prefix for events. Suggest using traditional ev: prefix to focus the test on one feature |
| text-edit-201-t.svg | ED | AN | Done. Changed text description for check 3 slightly: one additional instead of one more. I accidentally read one or more initially, and in that case the test wouldn't pass. |
| script-element-203-t.svg | ED | AE | Done. Issues with set/getAttributeNS. It uses the SVG namespace instead of null. Also, uses "xlink:href" as local name instead of only 'href'. |
| media-audio-215-t.svg | AE | AG | |
| animate-elem-204-t.svg | DOH | AS | Done |
| animate-elem-208-t.svg | DOH | AS | Done |
| animate-elem-219-t.svg | DOH | AE | Done |
| animate-elem-223-t.svg | DOH | AE | Done |
| animate-elem-224-t.svg | DOH | AE | |
| extend-namespace-02-t.svg | JF | OH | |
| linking-a-03-t.svg | LH | OH | |
| linking-frag-203-t.svg | DS | OH | |
| struct-use-08-b.svg | ED | DS | |
| struct-use-09-b.svg | AN, DS | AE | Done |
| styling-inherit-01-t.svg | CL | AN | Done, fixed d:OperatorScript namespace issue |
| styling-pres-01-t.svg | CL | AN | |
| types-basicDOM-02-t.svg | AN,AE | OH | |
| types-data-201-t.svg | DOH | AN | |
| types-data-202-t.svg | DOH | AN | |
| types-data-203-t.svg | DOH | AN | |
| types-data-204-t.svg | DOH | AN | |
| types-data-205-t.svg | DOH | AN | |
| udom-event-209-t.svg | CM | ED | Done |
| udom-svg-216-t.svg | CM | DOH, ED | Done. Test could be more extensive though. |
| udom-svg-217-t.svg | CM | DOH, ED | |
| udom-svg-218-t.svg | CM | DOH, ED | |
| udom-svg-224-t.svg | CM | AE | Done. The applicableElementNames setup does not work. Should be replaced with simple array, with i used in mian loop and name set to applicableElementNames[i]. svg.createSVGRGBColor needs paramaters. setAttributeNS() for xlink:href should only have href as the local name. Nice to have expected/returned exception codes in results string |
| udom-svgtimedelement-202-t.svg | ASl | DOH | Should not be added, duplicate test. Should be removed from CVS |
| udom-svgtimedelement-203-t.svg | ASl | DOH | Should not be added, duplicate test. Should be removed from CVS |
| udom-svgtimedelement-204-t.svg | ASl | DOH | Should not be added, duplicate test. Should be removed from CVS |
| udom-svgtimedelement-205-t.svg | ASl | OH | Should not be added, duplicate test. Should be removed from CVS |
| udom-textcontent-201-t.svg | ASl | OH |
