diff --git a/tests/qunit/CardModel.test.js b/tests/qunit/CardModel.test.js index fd9d8df1..d22401b9 100644 --- a/tests/qunit/CardModel.test.js +++ b/tests/qunit/CardModel.test.js @@ -19,7 +19,7 @@ model = new CardModel( {} ); model.on( 'change', function () { - assert.ok( false, 'It doesn\'t emit an event when silenced.' ); + assert.true( false, 'It doesn\'t emit an event when silenced.' ); } ); model.set( 'foo', 'bar', true ); diff --git a/tests/qunit/RelatedPagesGateway.test.js b/tests/qunit/RelatedPagesGateway.test.js index d806e608..7cb53762 100644 --- a/tests/qunit/RelatedPagesGateway.test.js +++ b/tests/qunit/RelatedPagesGateway.test.js @@ -32,7 +32,7 @@ this.sandbox.stub( this.api, 'get' ).returns( $.Deferred().resolve( relatedPages ) ); return gateway.getForCurrentPage( 1 ).then( function ( results ) { - assert.ok( Array.isArray( results ), 'Results must be an array' ); + assert.true( Array.isArray( results ), 'Results must be an array' ); assert.strictEqual( results[ 0 ].title, 'Oh noes' ); } ); } ); @@ -42,7 +42,7 @@ this.sandbox.stub( this.api, 'get' ).returns( $.Deferred().resolve( emptyRelatedPages ) ); return gateway.getForCurrentPage( 1 ).then( function ( results ) { - assert.ok( Array.isArray( results ), 'Results must be an array' ); + assert.true( Array.isArray( results ), 'Results must be an array' ); assert.strictEqual( results.length, 0 ); } ); } ); @@ -52,8 +52,8 @@ spy = this.sandbox.stub( this.api, 'get' ).returns( $.Deferred().resolve( relatedPages ) ); return gateway.getForCurrentPage( 1 ).then( function ( results ) { - assert.ok( Array.isArray( results ), 'Results must be an array' ); - assert.notOk( spy.called, 'API is not invoked' ); + assert.true( Array.isArray( results ), 'Results must be an array' ); + assert.false( spy.called, 'API is not invoked' ); assert.strictEqual( results.length, 0 ); } ); } ); @@ -97,7 +97,7 @@ .returns( $.Deferred().resolve( relatedPages ) ); return gateway.getForCurrentPage( 1 ).then( function ( results ) { - assert.ok( Array.isArray( results ), 'Results must be an array' ); + assert.true( Array.isArray( results ), 'Results must be an array' ); assert.strictEqual( results.length, 1, 'API is invoked to source articles.' ); } ); } );