Merge "Trivial: Remove unuseful comments for assertRaise() checking"

Author: Jenkins

openstackclient/tests/compute/v2/test_server.py

@@ -146,7 +146,6 @@ def test_server_create_no_options(self):
             ('server_name', self.new_server.name),
         ]
 
-        # Missing required args should bail here
         self.assertRaises(utils.ParserException, self.check_parser,
                           self.cmd, arglist, verifylist)
 

openstackclient/tests/network/v2/test_floating_ip.py

@@ -88,7 +88,6 @@ def test_create_no_options(self):
         arglist = []
         verifylist = []
 
-        # Missing required args should bail here
         self.assertRaises(tests_utils.ParserException, self.check_parser,
                           self.cmd, arglist, verifylist)
 
@@ -314,7 +313,6 @@ def test_create_no_options(self):
         arglist = []
         verifylist = []
 
-        # Missing required args should bail here
         self.assertRaises(tests_utils.ParserException, self.check_parser,
                           self.cmd, arglist, verifylist)
 

openstackclient/tests/network/v2/test_network.py

@@ -112,7 +112,6 @@ def test_create_no_options(self):
         arglist = []
         verifylist = []
 
-        # Missing required args should bail here
         self.assertRaises(tests_utils.ParserException, self.check_parser,
                           self.cmd, arglist, verifylist)
 
@@ -599,7 +598,6 @@ def test_show_no_options(self):
         arglist = []
         verifylist = []
 
-        # Missing required args should bail here
         self.assertRaises(tests_utils.ParserException, self.check_parser,
                           self.cmd, arglist, verifylist)
 
@@ -916,7 +914,6 @@ def test_show_no_options(self):
         arglist = []
         verifylist = []
 
-        # Missing required args should bail here
         self.assertRaises(tests_utils.ParserException, self.check_parser,
                           self.cmd, arglist, verifylist)
 

openstackclient/tests/network/v2/test_router.py

@@ -47,7 +47,6 @@ def test_add_port_no_option(self):
         arglist = []
         verifylist = []
 
-        # Missing required args should bail here
         self.assertRaises(tests_utils.ParserException, self.check_parser,
                           self.cmd, arglist, verifylist)
 
@@ -88,7 +87,6 @@ def test_add_subnet_no_option(self):
         arglist = []
         verifylist = []
 
-        # Missing required args should bail here
         self.assertRaises(tests_utils.ParserException, self.check_parser,
                           self.cmd, arglist, verifylist)
 
@@ -154,7 +152,6 @@ def test_create_no_options(self):
         arglist = []
         verifylist = []
 
-        # Missing required args should bail here
         self.assertRaises(tests_utils.ParserException, self.check_parser,
                           self.cmd, arglist, verifylist)
 
@@ -335,7 +332,6 @@ def test_remove_port_no_option(self):
         arglist = []
         verifylist = []
 
-        # Missing required args should bail here
         self.assertRaises(tests_utils.ParserException, self.check_parser,
                           self.cmd, arglist, verifylist)
 
@@ -375,7 +371,6 @@ def test_remove_subnet_no_option(self):
         arglist = []
         verifylist = []
 
-        # Missing required args should bail here
         self.assertRaises(tests_utils.ParserException, self.check_parser,
                           self.cmd, arglist, verifylist)
 
@@ -475,7 +470,6 @@ def test_set_distributed_centralized(self):
             ('distributed', False),
         ]
 
-        # Missing required args should bail here
         self.assertRaises(tests_utils.ParserException, self.check_parser,
                           self.cmd, arglist, verifylist)
 
@@ -534,7 +528,6 @@ def test_set_route_clear_routes(self):
             ('clear_routes', True),
         ]
 
-        # Argument parse failing should bail here
         self.assertRaises(tests_utils.ParserException, self.check_parser,
                           self.cmd, arglist, verifylist)
 
@@ -591,7 +584,6 @@ def test_show_no_options(self):
         arglist = []
         verifylist = []
 
-        # Missing required args should bail here
         self.assertRaises(tests_utils.ParserException, self.check_parser,
                           self.cmd, arglist, verifylist)
 

openstackclient/tests/network/v2/test_subnet_pool.py

@@ -109,7 +109,6 @@ def test_create_no_options(self):
         arglist = []
         verifylist = []
 
-        # Missing required args should bail here
         self.assertRaises(tests_utils.ParserException, self.check_parser,
                           self.cmd, arglist, verifylist)
 
@@ -624,7 +623,6 @@ def test_show_no_options(self):
         arglist = []
         verifylist = []
 
-        # Missing required args should bail here
         self.assertRaises(tests_utils.ParserException, self.check_parser,
                           self.cmd, arglist, verifylist)