commit/galaxy-central: carlfeberhard: QA, browser tests: fix for history update with non-attribute key (fails silently now); fix for hda create when no from_ld_id is given (now 400)
1 new commit in galaxy-central: https://bitbucket.org/galaxy/galaxy-central/commits/b9505d30f85f/ Changeset: b9505d30f85f User: carlfeberhard Date: 2013-09-12 20:27:05 Summary: QA, browser tests: fix for history update with non-attribute key (fails silently now); fix for hda create when no from_ld_id is given (now 400) Affected #: 2 files diff -r 85e8246c95e4ea02d666d13de2264b00453c2bd6 -r b9505d30f85f6e5e49640c6428185ab427899ec4 test/casperjs/api-hda-tests.js --- a/test/casperjs/api-hda-tests.js +++ b/test/casperjs/api-hda-tests.js @@ -371,7 +371,7 @@ // ------------------------------------------------------------------------------------------- ERRORS - this.test.comment( 'create should error with "not implemented" when the param "from_ld_id" is not used' ); + this.test.comment( 'create should error with "Please define the source" when the param "from_ld_id" is not used' ); var errored = false; try { // sending an empty object won't work @@ -379,15 +379,14 @@ } catch( err ){ errored = true; - this.test.assert( err.message.indexOf( 'Not implemented' ) !== -1, + this.test.assert( err.message.indexOf( 'Please define the source' ) !== -1, 'Error has the proper message: ' + err.message ); - this.test.assert( err.status === 501, 'Error has the proper status code: ' + err.status ); + this.test.assert( err.status === 400, 'Error has the proper status code: ' + err.status ); } if( !errored ){ this.test.fail( 'create without "from_ld_id" did not cause error' ); } - //var returned = this.api.hdas.update( lastHistory.id, hdaIndex[0].id, { deleted: true, blerp: 'blerp' }); //var returned = this.api.hdas.update( lastHistory.id, { deleted: true, blerp: 'blerp' }); //this.debug( 'returned:' + this.jsonStr( returned ) ); diff -r 85e8246c95e4ea02d666d13de2264b00453c2bd6 -r b9505d30f85f6e5e49640c6428185ab427899ec4 test/casperjs/api-history-tests.js --- a/test/casperjs/api-history-tests.js +++ b/test/casperjs/api-history-tests.js @@ -172,8 +172,8 @@ }); this.test.assert( countKeys( returned ) === 0, "No changed returned: " + this.jsonStr( returned ) ); - this.test.comment( 'updating using a nonsense key should fail with an error' ); - var err = {}; + this.test.comment( 'updating using a nonsense key should fail silently' ); + var err = null; try { returned = this.api.histories.update( newFirstHistory.id, { konamiCode : 'uuddlrlrba' @@ -182,8 +182,7 @@ err = error; //this.debug( this.jsonStr( err ) ); } - this.test.assert( !!err.message, "Error occurred: " + err.message ); - this.test.assert( err.status === 400, "Error status is 400: " + err.status ); + this.test.assert( err === null, "No error occurred: " + this.jsonStr( err ) ); this.test.comment( 'updating by attempting to change type should cause an error' ); err = {}; Repository URL: https://bitbucket.org/galaxy/galaxy-central/ -- This is a commit notification from bitbucket.org. You are receiving this because you have the service enabled, addressing the recipient of this email.
participants (1)
-
commits-noreply@bitbucket.org