From 4a56aeb7b0deb19f31657ffec2fd702d6ac2f5af Mon Sep 17 00:00:00 2001 From: John Chilton Date: Thu, 16 Apr 2015 14:28:56 -0400 Subject: [PATCH] test_tools.py enhancements... --- test/api/test_tools.py | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/test/api/test_tools.py b/test/api/test_tools.py index 88a9cd6f8ce..fe93e6cd436 100644 --- a/test/api/test_tools.py +++ b/test/api/test_tools.py @@ -29,7 +29,7 @@ class ToolsTestCase( api.ApiTestCase ): def test_show_repeat( self ): tool_info = self._show_valid_tool( "cat1" ) parameters = tool_info[ "inputs" ] - assert len( parameters ) == 2 + assert len( parameters ) == 2, "Expected two inputs - got [%s]" % parameters assert parameters[ 0 ][ "name" ] == "input1" assert parameters[ 1 ][ "name" ] == "queries" @@ -219,9 +219,7 @@ class ToolsTestCase( api.ApiTestCase ): 'col': "' ; echo 'moo", } response = self._run( "column_param", history_id, inputs ) - if response.status_code == 200: - message = "Known, high priority issue. Column parameters are sanitized but invalid values should prevent execution and doesn't." - raise AssertionError(message) + assert response.status_code != 200 @skip_without_tool( "collection_paired_test" ) def test_collection_parameter( self ):