Fix TestharnessTest to have a correct meta_key

tools/manifest/item.py

@@ -129,7 +129,10 @@ def script_metadata(self):
             return self._source_file.script_metadata
 
     def meta_key(self):
-        return (self.timeout, self.testdriver)
+        script_metadata = self.script_metadata
+        if script_metadata is not None:
+            script_metadata = tuple(tuple(x) for x in script_metadata)
+        return (self.timeout, self.testdriver, self.jsshell, script_metadata)
 
     def to_json(self):
         rv = URLManifestItem.to_json(self)

tools/manifest/tests/test_item.py

@@ -1,6 +1,6 @@
 import pytest
 
-from ..item import URLManifestItem
+from ..item import URLManifestItem, TestharnessTest
 
 
 @pytest.mark.parametrize("path", [
@@ -39,3 +39,28 @@ def test_url_not_https(path):
     m = URLManifestItem("/foobar", "/" + path, "/", "/foo.bar/" + path)
 
     assert m.https is False
+
+
+def test_testharness_meta_key_includes_jsshell():
+    a = TestharnessTest("/foobar", "/foo", "/foo.bar", "/foo.bar/foo",
+                        jsshell=False, script_metadata=[])
+    b = TestharnessTest("/foobar", "/foo", "/foo.bar", "/foo.bar/foo",
+                        jsshell=True, script_metadata=[])
+
+    assert a.meta_key() != b.meta_key()
+
+
+@pytest.mark.parametrize("script_metadata", [
+    None,
+    [],
+    [('script', '/resources/WebIDLParser.js'), ('script', '/resources/idlharness.js')],
+    [[u'script', u'/resources/WebIDLParser.js'], [u'script', u'/resources/idlharness.js']],
+])
+def test_testharness_hashable_script_metadata(script_metadata):
+    a = TestharnessTest("/",
+                        "BackgroundSync/interfaces.https.any.js",
+                        "/",
+                        "/BackgroundSync/interfaces.https.any.js",
+                        script_metadata=script_metadata)
+
+    assert hash(a) is not None