douglasmeyer / test_it Goto Github PK

TestIt.inspect will return a '{}' for empty arrays in nodejs because of a bad javascript parctice:

var TestIt = require('test_it');
TestIt.inspect([])
'{}'

The problem is caused because of a bad Javascript practice in TestIt source (test_it.js line 347) - you are comparing the constructor to Array which is only good if you are in a browser and do not work across IFrames.

The correct way to compare an array is to either use Array.isArray (not supported everywhere) or roll your own:

function isArray(instance) {
    return Object.prototype.toString.call(instance) == '[object Array]';
}

What is to come has some seriously helpful proposals for Testit.

• Easier to add custom assertions
• Easier to write extensions
• More standard reporting
• Easier to extend
• Provide default styling

Not entirely sold on the names for TestIt.extend or TestIt.bind, but I think they make sense.