diff --git a/tests/qunit/ext.uls.tests.js b/tests/qunit/ext.uls.tests.js index 47259667..731a9e6f 100644 --- a/tests/qunit/ext.uls.tests.js +++ b/tests/qunit/ext.uls.tests.js @@ -23,14 +23,10 @@ QUnit.module( 'ext.uls', QUnit.newMwEnvironment() ); QUnit.test( 'Initial check', function ( assert ) { - QUnit.expect( 1 ); - assert.ok( $.fn.uls, '$.fn.uls is defined' ); } ); QUnit.test( 'Custom langdb', function ( assert ) { - QUnit.expect( 1 ); - // This is a custom non-standard language code used in MW. // If it's not defined, then, for example, // its direction cannot be acquired using the langdb utils. @@ -38,9 +34,7 @@ } ); QUnit.test( 'User preferences', function ( assert ) { - var prefName, prefs, prefsToSave, readPrefs; - - QUnit.expect( 2 ); + var prefName, prefs, prefsToSave, readPrefs, done; // 'gofanim' means "fonts" in Hebrew. // Here it's used as a meaningless word, to test @@ -64,25 +58,18 @@ 'Correct value for the font name' ); - QUnit.stop(); + done = assert.async(); prefs.save( function ( successSave ) { - QUnit.start(); assert.ok( successSave, 'Options saving API did not produce an error.' ); - // Delete old options prefs.set( prefName, undefined ); - QUnit.stop(); - prefs.save( function () { - QUnit.start(); - } ); + prefs.save( done ); } ); } ); QUnit.test( 'Common languages', function ( assert ) { var i, foundTagalog, languagesInPH; - QUnit.expect( 1 ); - // Bug 49847 foundTagalog = false; languagesInPH = mw.uls.getFrequentLanguageList( 'PH' );