Revive tests for Document.createAttributeNS() and Element.setAttributeNodeNS()

LayoutTests/fast/dom/Document/script-tests/createAttributeNS-namespace-err.js

-description("createDocument tests modeled after createElementNS tests from mozilla which were attached to webkit bug 16833");
+description("createAttirbuteNS tests adapted from createElementNS tests attached to webkit bug 16833");
 
-// document.implementation.createDocument() should throw the same set of errors
-// as document.createElementNS()
-// http://www.w3.org/TR/DOM-Level-3-Core/core.html#Level-2-Core-DOM-createDocument
-// Thus we copied these test cases from:
-// LayoutTests/fast/dom/Document/resources/createDocument-namespace-err.js
-
 function assert(c, m)
 {
     if (!c)
         testFailed(m);
     else
         testPassed(m);
 }
 
 function stringForExceptionCode(c)
 {
     var exceptionName;
     switch(c) {
         case DOMException.INVALID_CHARACTER_ERR:
             exceptionName = "INVALID_CHARACTER_ERR";
             break;
         case DOMException.NAMESPACE_ERR:
             exceptionName = "NAMESPACE_ERR";
     }
     if (exceptionName)
         return exceptionName; // + "(" + c + ")";
     return c;
 }
 
-function assertEquals(actual, expect, m)
+function assertExceptionCode(exception, expect, m)
 {
+    var actual = exception.code;
     if (actual !== expect) {
         m += "; expected " + stringForExceptionCode(expect) + ", threw " + stringForExceptionCode(actual);
         testFailed(m);
     } else {
-        m += "; threw " + stringForExceptionCode(actual);;
+        m += "; threw " + exception.toString();
         testPassed(m);
     }
 }
 
 var allNSTests = [
    { args: [undefined, undefined] },
    { args: [null, undefined] },
    { args: [undefined, null], code: 5 },
    { args: [null, null], code: 5 },
    { args: [null, ""], code: 5 },
@@ skipping 32 matching lines @@
               "NCName (Gecko chooses to throw NAMESPACE_ERR here, but either is valid " +
               "as this is both an invalid XML name and an invalid QName)" },
    { args: ["http://example.com/", "a:a\u0BC6"] },
    { args: ["http://example.com/", "a\u0BC6:a"] },
    { args: ["http://example.com/", "xml:test"], code: 14, message: "binding xml prefix wrong" },
    { args: ["http://example.com/", "xmlns:test"], code: 14, message: "binding xmlns prefix wrong" },
    { args: ["http://www.w3.org/2000/xmlns/", "x:test"], code: 14, message: "binding namespace namespace to wrong prefix" },
    { args: ["http://www.w3.org/2000/xmlns/", "xmlns:test"] },
    { args: ["http://www.w3.org/XML/1998/namespace", "xml:test"] },
    { args: ["http://www.w3.org/XML/1998/namespace", "x:test"] },
+   { args: ["http://www.w3.org/2000/xmlns/", "xmlns"] }, // See http://www.w3.org/2000/xmlns/
+   { args: ["http://example.com/", "xmlns"], code: 14 }, // from the createAttributeNS section
 ];
 
 function sourceify(v)
 {
     switch (typeof v) {
-        case "undefined":
-            return v;
-        case "string":
-            return '"' + v.replace('"', '\\"') + '"';
-        default:
-            return String(v);
+    case "undefined":
+        return v;
+
+    case "string":
+        return '"' + v.replace('"', '\\"') + '"';
+
+    default:
+        return String(v);
     }
 }
 
-function sourceifyArgs(args)
+function runNSTests()
 {
-    var copy = new Array(args.length);
-    for (var i = 0, sz = args.length; i < sz; i++)
-        copy[i] = sourceify(args[i]);
+    var createFunction = document.createAttributeNS;
+    var createFunctionName = "createAttributeNS";
+    var doc = document;
 
-    return copy.join(", ");
-}
-
-function runNSTests(tests, doc, createFunctionName)
-{
-    for (var i = 0, sz = tests.length; i < sz; i++) {
-        var test = tests[i];
-
-        // Gecko throws "undefined" if createDocument isn't
-        // called with 3 arguments.  Instead of modifying all
-        // of the values in the arrays above (which were taken from createElementNS tests)
-        // we will instead just hack the args list here.
-        var argsWithExtraLastNull = test.args.slice(); // copy the args arary
-        argsWithExtraLastNull.push(null);
+    for (var i = 0, sz = allNSTests.length; i < sz; i++) {
+        var test = allNSTests[i];
 
         var code = -1;
-        var argStr = sourceifyArgs(argsWithExtraLastNull);
+        var argStr = sourceify(test.args[0]) + ", " + sourceify(test.args[1]);
         var msg = createFunctionName + "(" + argStr + ")";
         if ("message" in test)
             msg += "; " + test.message;
         try {
-            doc[createFunctionName].apply(doc, argsWithExtraLastNull);
+            createFunction.apply(doc, test.args);
             assert(!("code" in test), msg);
         } catch (e) {
-            assertEquals(e.code, test.code || "expected no exception", msg);
+            assertExceptionCode(e, test.code || "expected no exception", msg);
         }
     }
 }
 
-shouldThrow("document.implementation.createDocument()", '"TypeError: Failed to execute \'createDocument\' on \'DOMImplementation\': 2 arguments required, but only 0 present."');
-shouldThrow("document.implementation.createDocument(\"http://www.example.com\")", '"TypeError: Failed to execute \'createDocument\' on \'DOMImplementation\': 2 arguments required, but only 1 present."');
+// Moz throws a "Not enough arguments" exception in these, we don't:
+shouldBeEqualToString("document.createAttributeNS().toString()", "[object Attr]");
+shouldBeEqualToString("document.createAttributeNS(\"http://www.example.com\").toString()", "[object Attr]");
 
-runNSTests(allNSTests, document.implementation, "createDocument");
+runNSTests();