jQuery Forum

Hi,

as I couldn't find a QUnit-specific bugtracker, I am posting this here:

When debugging some failing tests, I noticed that the source of an exception lays inside the QUnit.diff method.

The code there is something like (line 1348):

1. for (var i in ns) {
2.    if (ns[i].rows.length == 1 && typeof(os[i]) != "undefined" && os[i].rows.length == 1) {
3.       //...
4.    }
5. }

However, when "Object.prototype" is extended, this piece of code will fail as the name of e.g. the method is enumerated, too.

Example:

1. Object.prototype.foo = function() { };
2. QUnit.diff("\"TEXT\"", "\"OTHER\"");

 causes: "TypeError: Cannot read property 'length' of undefined" when extending Object.prototype.

I was able to fix this bug by adding:

1. if (!ns.hasOwnProperty(i))
2.     continue;

as the first statement in the loop (into line 1349).

I hope you can add this fix to the official source code.

Many thanks for your great library!

Cheers
winSharp93