Refactor: Initialize parser in setUp() in TestNonNegativeAction

Change-Id: I12846acc4450d31d19897bbdfc6846bde8c8f2ce

Author: tang-chen

openstackclient/tests/common/test_parseractions.py

@@ -62,53 +62,37 @@ def test_error_values(self):
 
 
 class TestNonNegativeAction(utils.TestCase):
-    def test_negative_values(self):
-        parser = argparse.ArgumentParser()
+
+    def setUp(self):
+        super(TestNonNegativeAction, self).setUp()
+
+        self.parser = argparse.ArgumentParser()
 
         # Set up our typical usage
-        parser.add_argument(
+        self.parser.add_argument(
             '--foo',
             metavar='<foo>',
             type=int,
             action=parseractions.NonNegativeAction,
         )
 
+    def test_negative_values(self):
         self.assertRaises(
             argparse.ArgumentTypeError,
-            parser.parse_args,
+            self.parser.parse_args,
             "--foo -1".split()
         )
 
     def test_zero_values(self):
-        parser = argparse.ArgumentParser()
-
-        # Set up our typical usage
-        parser.add_argument(
-            '--foo',
-            metavar='<foo>',
-            type=int,
-            action=parseractions.NonNegativeAction,
-        )
-
-        results = parser.parse_args(
+        results = self.parser.parse_args(
             '--foo 0'.split()
         )
 
         actual = getattr(results, 'foo', None)
         self.assertEqual(actual, 0)
 
     def test_positive_values(self):
-        parser = argparse.ArgumentParser()
-
-        # Set up our typical usage
-        parser.add_argument(
-            '--foo',
-            metavar='<foo>',
-            type=int,
-            action=parseractions.NonNegativeAction,
-        )
-
-        results = parser.parse_args(
+        results = self.parser.parse_args(
             '--foo 1'.split()
         )