WebApps testharness.js needs to handle 'EventException'

• From: Alex Kuang <Alex.Kuang@microsoft.com>
• Date: Fri, 14 Oct 2011 19:26:50 +0000
• To: James Graham <jgraham@opera.com>, "public-webapps-testsuite@w3.org" <public-webapps-testsuite@w3.org>
• Message-ID: <1F0F046621E01A41A108CEC800104D9D0D813567@TK5EX14MBXC254.redmond.corp.microsoft.>

While converting some event tests to use the harness, I noticed that function assert_throws(code, func, description) in testharness.js needs to handle 'EventException'. Here is a 'patch' for testharness.js around line ~752 -- James Graham do you agree and if so can you update testharness.js?

                        else if (code in EventException)
                        {
                                expected_type = "EventException";
                                required_props[code] = EventException[code];
                                required_props.code = EventException[code];
                                //As above
                                //required_props.name = code;
                       }

Here are the test cases testing EventException:
http://w3c-test.org/webapps/DOMEvents/tests/submissions/Microsoft/converted/dispatchEvent.DISPATCH_REQUEST_ERR.html 
http://w3c-test.org/webapps/DOMEvents/tests/submissions/Microsoft/converted/dispatchEvent.UNSPECIFIED_EVENT_TYPE_ERR.html 

Thanks,
Alex

Received on Friday, 14 October 2011 22:11:33 UTC