Bug 1526402 [wpt PR 14866] - Improve the test names of tests for toJSON behavior., a=testonly

Automatic update from web-platform-tests
Improve the test names of tests for toJSON behavior. (#14866)

In particular, this avoids duplicate test names when testing with multiple objects (ref #14645).
--

wpt-commits: 7fb15bfac1e7380aa74176886120edab9afb5d02
wpt-pr: 14866

testing/web-platform/tests/resources/idlharness.js

@@ -2309,7 +2309,7 @@ IdlInterface.prototype.do_member_operation_asserts = function(memberHolderObject
     }
 }
 
-IdlInterface.prototype.test_to_json_operation = function(memberHolderObject, member) {
+IdlInterface.prototype.test_to_json_operation = function(desc, memberHolderObject, member) {
     var instanceName = memberHolderObject && memberHolderObject.constructor.name
         || member.name + " object";
     if (member.has_extended_attribute("Default")) {
@@ -2325,12 +2325,12 @@ IdlInterface.prototype.test_to_json_operation = function(memberHolderObject, mem
                 this.array.assert_type_is(json[k], type);
                 delete json[k];
             }, this);
-        }.bind(this), "Test default toJSON operation of " + instanceName);
+        }.bind(this), this.name + " interface: default toJSON operation on " + desc);
     } else {
         subsetTestByKey(this.name, test, function() {
             assert_true(this.array.is_json_type(member.idlType), JSON.stringify(member.idlType) + " is not an appropriate return value for the toJSON operation of " + instanceName);
             this.array.assert_type_is(memberHolderObject.toJSON(), member.idlType);
-        }.bind(this), "Test toJSON operation of " + instanceName);
+        }.bind(this), this.name + " interface: toJSON operation on " + desc);
     }
 };
 
@@ -2722,7 +2722,7 @@ IdlInterface.prototype.test_interface_of = function(desc, obj, exception, expect
         }
 
         if (member.is_to_json_regular_operation()) {
-            this.test_to_json_operation(obj, member);
+            this.test_to_json_operation(desc, obj, member);
         }
     }
 };

testing/web-platform/tests/resources/test/tests/functional/idlharness/IdlInterface/test_to_json_operation.html

@@ -29,19 +29,19 @@
 
     var i, obj;
     i = interfaceFrom("interface A { [Default] object toJSON(); attribute long foo; };");
-    i.test_to_json_operation(wrap(i, { foo: 123 }), i.members[0]);
+    i.test_to_json_operation("object", wrap(i, { foo: 123 }), i.members[0]);
 
     // should fail (wrong type)
     i = interfaceFrom("interface B { [Default] object toJSON(); attribute long foo; };");
-    i.test_to_json_operation(wrap(i, { foo: "a value" }), i.members[0]);
+    i.test_to_json_operation("object", wrap(i, { foo: "a value" }), i.members[0]);
 
     // should handle extraneous attributes (e.g. from an extension specification)
     i = interfaceFrom("interface C { [Default] object toJSON(); attribute long foo; };");
-    i.test_to_json_operation(wrap(i, { foo: 123, bar: 456 }), i.members[0]);
+    i.test_to_json_operation("object", wrap(i, { foo: 123, bar: 456 }), i.members[0]);
 
     // should fail (missing property)
     i = interfaceFrom("interface D { [Default] object toJSON(); attribute long foo; };");
-    i.test_to_json_operation(wrap(i, { }), i.members[0]);
+    i.test_to_json_operation("object", wrap(i, { }), i.members[0]);
 
     // should fail (should be writable)
     obj = Object.defineProperties({}, { foo: {
@@ -51,7 +51,7 @@
         value: 123
     }});
     i = interfaceFrom("interface F { [Default] object toJSON(); attribute long foo; };");
-    i.test_to_json_operation(wrap(i, obj), i.members[0]);
+    i.test_to_json_operation("object", wrap(i, obj), i.members[0]);
 
     // should fail (should be enumerable)
     obj = Object.defineProperties({}, { foo: {
@@ -61,7 +61,7 @@
         value: 123
     }});
     i = interfaceFrom("interface G { [Default] object toJSON(); attribute long foo; };");
-    i.test_to_json_operation(wrap(i, obj), i.members[0]);
+    i.test_to_json_operation("object", wrap(i, obj), i.members[0]);
 
     // should fail (should be configurable)
     obj = Object.defineProperties({}, { foo: {
@@ -71,27 +71,27 @@
         value: 123
     }});
     i = interfaceFrom("interface H { [Default] object toJSON(); attribute long foo; };");
-    i.test_to_json_operation(wrap(i, obj), i.members[0]);
+    i.test_to_json_operation("object", wrap(i, obj), i.members[0]);
 
     var idl = new IdlArray();
     idl.add_idls("interface I : J { [Default] object toJSON(); attribute long foo; };");
     idl.add_idls("interface J { [Default] object toJSON(); attribute DOMString foo;};");
     var i = idl.members.I;
-    i.test_to_json_operation(wrap(i, { foo: 123 }), i.members[0]);
+    i.test_to_json_operation("object", wrap(i, { foo: 123 }), i.members[0]);
 
     i = interfaceFrom("interface K { [Default] object toJSON(); };");
-    i.test_to_json_operation(wrap(i, {}), i.members[0]);
+    i.test_to_json_operation("object", wrap(i, {}), i.members[0]);
 
     i = interfaceFrom("interface L { DOMString toJSON(); };");
-    i.test_to_json_operation(wrap(i, "a string"), i.members[0]);
+    i.test_to_json_operation("object", wrap(i, "a string"), i.members[0]);
 
     // should fail (wrong output type)
     i = interfaceFrom("interface M { DOMString toJSON(); };");
-    i.test_to_json_operation(wrap(i, {}), i.members[0]);
+    i.test_to_json_operation("object", wrap(i, {}), i.members[0]);
 
     // should fail (not an IDL type)
     i = interfaceFrom("interface N { DOMException toJSON(); };");
-    i.test_to_json_operation(wrap(i, {}), i.members[0]);
+    i.test_to_json_operation("object", wrap(i, {}), i.members[0]);
 </script>
 <script type="text/json" id="expected">
     {
@@ -102,73 +102,73 @@
         "summarized_tests": [
             {
                 "message": null,
-                "name": "Test default toJSON operation of A",
+                "name": "A interface: default toJSON operation on object",
                 "properties": {},
                 "status_string": "PASS"
             },
             {
                 "message": "assert_equals: expected \"number\" but got \"string\"",
-                "name": "Test default toJSON operation of B",
+                "name": "B interface: default toJSON operation on object",
                 "properties": {},
                 "status_string": "FAIL"
             },
             {
                 "message": null,
-                "name": "Test default toJSON operation of C",
+                "name": "C interface: default toJSON operation on object",
                 "properties": {},
                 "status_string": "PASS"
             },
             {
                 "message": "assert_true: property \"foo\" should be present in the output of D.prototype.toJSON() expected true got false",
-                "name": "Test default toJSON operation of D",
+                "name": "D interface: default toJSON operation on object",
                 "properties": {},
                 "status_string": "FAIL"
             },
             {
                 "message": "assert_true: property foo should be writable expected true got false",
-                "name": "Test default toJSON operation of F",
+                "name": "F interface: default toJSON operation on object",
                 "properties": {},
                 "status_string": "FAIL"
             },
             {
                 "message": "assert_true: property foo should be enumerable expected true got false",
-                "name": "Test default toJSON operation of G",
+                "name": "G interface: default toJSON operation on object",
                 "properties": {},
                 "status_string": "FAIL"
             },
             {
                 "message": "assert_true: property foo should be configurable expected true got false",
-                "name": "Test default toJSON operation of H",
+                "name": "H interface: default toJSON operation on object",
                 "properties": {},
                 "status_string": "FAIL"
             },
             {
                 "message": null,
-                "name": "Test default toJSON operation of I",
+                "name": "I interface: default toJSON operation on object",
                 "properties": {},
                 "status_string": "PASS"
             },
             {
                 "message": null,
-                "name": "Test default toJSON operation of K",
+                "name": "K interface: default toJSON operation on object",
                 "properties": {},
                 "status_string": "PASS"
             },
             {
                 "message": null,
-                "name": "Test toJSON operation of L",
+                "name": "L interface: toJSON operation on object",
                 "properties": {},
                 "status_string": "PASS"
             },
             {
                 "message": "assert_equals: expected \"string\" but got \"object\"",
-                "name": "Test toJSON operation of M",
+                "name": "M interface: toJSON operation on object",
                 "properties": {},
                 "status_string": "FAIL"
             },
             {
                 "message": "assert_true: {\"type\":\"return-type\",\"generic\":null,\"nullable\":false,\"union\":false,\"idlType\":\"DOMException\",\"extAttrs\":[]} is not an appropriate return value for the toJSON operation of N expected true got false",
-                "name": "Test toJSON operation of N",
+                "name": "N interface: toJSON operation on object",
                 "properties": {},
                 "status_string": "FAIL"
             }