Testing in Scaffold not working

[zuphilip]

It seems for me that running any tests in scaffold is not working. @adam3smith @aurimasv Can you still run tests in scaffold or can you reproduce the error?

My browser console shows the error:

TypeError: items[i].getElementsByTagName is not a function scaffold.js:928:4

[adam3smith]

same problem. I can create but not run.

[zuphilip]

Okay, I debbuged the problem further. It looks that first the loop variable i equals 0 which is okay and then the variable equals append (function name) and the error happens:

I guess we have to rewrite the for-loop here... Would you agree that it should be simply be

for (var i=0; i<items.length; i++) {

?

[aurimasv]

Yes, but where is the "append"coming from? Sounds like something is
modifying the array prototype, which is ok, I guess, but that shouldn't be
our code.
On Apr 3, 2016 13:24, "Philipp Zumstein" notifications@github.com wrote:

Okay, I debbuged the problem further. It looks that first the loop
variable i equals 0 which is okay and then the variable equals append
(function name) and the error happens:

[image: scaffold-error]
https://cloud.githubusercontent.com/assets/5199995/14233704/4c1bc4c2-f9d0-11e5-85d1-023955de849a.jpg

I guess we have to rewrite the for-loop here... Would you agree that it
should be simply be

for (var i=0; i<items.length; i++) {

?

—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
#26 (comment)

[zuphilip]

I just tested that loop separately: for (j in items) { Zotero.debug(j); } gives

(3)(+0006464): 0
(3)(+0000000): append
(3)(+0000000): remove
(3)(+0000000): item
(3)(+0000000): length

Thus, it seems that these are the functions/methods of (the object) items. No?

[zuphilip]

@aurimasv items is a ChromeNodeList and not an array. Does this make sense?

[zuphilip]

Hm.. the for loops seem to work now in zuphilip@e8052b1, but there is another error when trying to update a test:

TypeError: listbox.selectedItems.slice is not a function

My impression is that selectedItems changed its type to ChromeNodeList.

Update: Yes, indeed this happend in Firefox 45, see https://hg.mozilla.org/releases/mozilla-b2g44_v2_5/rev/fb998e8c0b94

[zuphilip]

Okay, I think I fixed it.

[dstillman]

Changed in https://bugzilla.mozilla.org/show_bug.cgi?id=120684, BTW

(Along with [...listbox.selectedItems], Array.from(listbox.selectedItems) also works for this sort of thing.)