r109677 MediaWiki - Code Review archive

[WebFonts.tests] Fixes

* Variables isFontFaceLoaded and fontFamilyList are lacking a "var" statement, were leaking to global scope. Adding local closure and turning them into (local) function expressions and moved to the top of the scope for clarity.

* Removing redundant tests:
** ok( $inputElement.remove() )
** ok( $( 'body' ).append( .. ) )
-- jQuery collections always return a jQuery object, they are chainable prototypes without a status-related return value. These checks don't do anything, they will always return a jQuery object and all objects case to true in JavaScript

* Removing useless tests:
** assertTrue( $selectElement !== [] )
** In JavaScript all objects are compared by reference, this comparison can never succeed because 1) jQuery objects are not arrays, 2) even the, "[] !== []" because they are both separate instances of Array. Arrays and objects must be iterated in order to compare them by value, this is done by deepEqual.

* Fixing element selection.
-- Instead of:
$('body').append( '<input class="foo"/>' );
var $foo = $('input.foo');
-- Use:
var $foo = $( '<input class="foo"/>' );
$('body').append( $foo );
-- No need to query the DOM for this.

* Using equal() instead of ok() as much as possible. ok() is not useful in most unit testing because it makes you have to do the comparison inline. Which means the value given to ok() is only the end result, not the expected/result values themselves. Which means in case of a failure, ok() can only say "Passed" or "Failed", whereas equal() (having both values), can output useful debug information. This is especially important when working with test reports generated externally (eg. through TestSwarm) in which are the test report is all you have.

// Not very useful, the comparison is done inline and ok()
// can only know about the result of the comparison
var x = 'foo';
ok( x === 'bar' ); // Failed.
ok( x === 'foo' ); // Passed.


// Useful! The unit test is give both values so it can also mention
// both in the output:
var x = 'foo';
equal( x, 'bar' ); // Failed. Expected: "bar". Result: "foo"
equal( x, 'foo' ); // Passed. Expected: "foo".