Saving tags in tests different

[zuphilip]

It looks that previously in the test cases the tags were simply saved as a list of strings, but now with the newest Zotero 5.0 compatible version the tags are saved as a list of objects of strings. This makes the test result looks different, e.g. tests from ACM translator

"tags": [
-     "calling context tree"
+     {
+       "tag": "calling context tree"
+     }
-     "performance-aware revision control"
+     {
+       "tag": "performance-aware revision control"
+     }
-     "profiling"
+     {
+       "tag": "profiling"
+     }
]

Do we want to continue the new way or switch back to the old way?

[dstillman]

I fixed the comparison for tests (including in Scaffold, I believe) in zotero/zotero#1254 on the xhr-processDocuments branch of the client, which we'll try to merge soon. But as I noted in zotero/translators#1356 (comment), it's incorrect as an object anyway if it's not including type: 1 (automatic), so if that's still happening we should fix it, and for the sake of readability and conciseness probably just have it return a string (since Zotero RDF is the only case where we'd actually want type: 0).