Pass pylint test with networking scenario tests.

- remove parameters from setUp method
- take test name from id() test case method
- always use default template parameters
- create_stack method doesn't takes any parameter

Change-Id: I2d606c2329eae1d5b4c8eceb330feafb0d2af815

tobiko/tests/scenario/base.py

@@ -15,10 +15,10 @@
 from __future__ import absolute_import
 
 import os
+import sys
 
 from heatclient import exc
 
-
 from tobiko.tests import base
 from tobiko.common.managers import stack
 from tobiko.common.managers import network
@@ -28,20 +28,30 @@ from tobiko.common import constants
 class ScenarioTestsBase(base.TobikoTest):
     """All scenario tests inherit from this scenario base class."""
 
-    def setUp(self, file_path, params=None):
+    def setUp(self):
         super(ScenarioTestsBase, self).setUp()
         templates_dir = os.path.join(os.path.dirname(__file__), 'templates')
         self.stackManager = stack.StackManager(self.clientManager,
                                                templates_dir)
         self.networkManager = network.NetworkManager(self.clientManager)
-        self.params = params or self.default_params
+        self.params = self.default_params
-        file_name = os.path.basename(file_path)
+
-        self.stack_name = file_name.split(".py")[0]
+        test_name = self.id()
+        while test_name:
+            test_module = sys.modules.get(test_name)
+            if test_module:
+                break
+            name_parts = test_name.rsplit('.', 1)
+            if len(name_parts) == 1:
+                msg = "Invalid test name: {!r}".format(self.id())
+                raise RuntimeError(msg)
+            test_name = name_parts[0]
+        self.stack_name = test_name.rsplit('.', 1)[-1]
 
         try:
             self.stackManager.get_stack(self.stack_name)
         except exc.HTTPNotFound:
-            self.create_stack(self.stack_name)
+            self.create_stack()
 
     def create_stack(self):
         """Creates stack to be used by all scenario tests."""
@@ -57,7 +67,7 @@ class ScenarioTestsBase(base.TobikoTest):
         return st['stack']
 
     def _get_stack(self):
-        stack = self.stackManager.get_stack(self.stack_name)
+        _stack = self.stackManager.get_stack(self.stack_name)
-        if not stack:
+        if not _stack:
-            stack = self.create_stack()
+            _stack = self.create_stack()
-        return stack
+        return _stack

tobiko/tests/scenario/test_continuous_ping.py

@@ -27,7 +27,7 @@ class ContinuousPingTest(base.ScenarioTestsBase):
     MAX_PACKET_LOSS = 5
 
     def setUp(self):
-        super(ContinuousPingTest, self).setUp(__file__)
+        super(ContinuousPingTest, self).setUp()
         self.stack = self._get_stack()
         self.fip = self.stackManager.get_output(self.stack, "fip")
 

tobiko/tests/scenario/test_floatingip.py

@@ -22,7 +22,7 @@ class FloatingIPTest(base.ScenarioTestsBase):
     """Tests server connectivity"""
 
     def setUp(self):
-        super(FloatingIPTest, self).setUp(__file__)
+        super(FloatingIPTest, self).setUp()
         self.stack = self._get_stack()
         self.fip = self.stackManager.get_output(self.stack, "fip")
         self.unreachable_fip = self.stackManager.get_output(self.stack, "fip2")

tobiko/tests/scenario/test_mtu.py

@@ -27,7 +27,7 @@ class MTUTest(base.ScenarioTestsBase):
     """Tests MTU."""
 
     def setUp(self):
-        super(MTUTest, self).setUp(__file__)
+        super(MTUTest, self).setUp()
         self.stack = self._get_stack()
         self.fip_max_mtu = self.stackManager.get_output(
             self.stack, "fip_max_mtu")

tobiko/tests/scenario/test_security_groups.py

@@ -22,7 +22,7 @@ class SecurityGroupTest(base.ScenarioTestsBase):
     """Tests security groups."""
 
     def setUp(self):
-        super(SecurityGroupTest, self).setUp(__file__)
+        super(SecurityGroupTest, self).setUp()
         self.stack = self._get_stack()
         self.fip = self.stackManager.get_output(self.stack, "fip")
         self.unreachable_fip = self.stackManager.get_output(self.stack, "fip2")