slapos_accounting: Makes Pylint happy

master/bt5/slapos_accounting/TestTemplateItem/portal_components/test.erp5.testSlapOSAccountingAlarm.py

@@ -28,7 +28,7 @@ class Simulator:
     """Simulation Method"""
     old = open(self.outfile, 'r').read()
     if old:
-      l = eval(old)
+      l = eval(old) #pylint: disable=eval-used
     else:
       l = []
     l.append({'recmethod': self.method,
@@ -782,8 +782,8 @@ class TestSlapOSTriggerBuildAlarm(SlapOSTestCaseMixin):
 
       simulation_movement.SimulationMovement_buildSlapOS(tag='root_tag')
 
-      build_value = eval(open(build_simulator).read())
-      activate_value = eval(open(activate_simulator).read())
+      build_value = eval(open(build_simulator).read()) #pylint: disable=eval-used
+      activate_value = eval(open(activate_simulator).read()) #pylint: disable=eval-used
 
       self.assertEqual([{
         'recmethod': 'build',
@@ -804,8 +804,8 @@ class TestSlapOSTriggerBuildAlarm(SlapOSTestCaseMixin):
       open(activate_simulator, 'w').truncate()
 
       lower_simulation_movement.SimulationMovement_buildSlapOS(tag='lower_tag')
-      build_value = eval(open(build_simulator).read())
-      activate_value = eval(open(activate_simulator).read())
+      build_value = eval(open(build_simulator).read()) #pylint: disable=eval-used
+      activate_value = eval(open(activate_simulator).read()) #pylint: disable=eval-used
 
       self.assertEqual([{
         'recmethod': 'build',
@@ -870,8 +870,8 @@ class TestSlapOSTriggerBuildAlarm(SlapOSTestCaseMixin):
 
       simulation_movement.SimulationMovement_buildSlapOS(tag='root_tag')
 
-      build_value = eval(open(build_simulator).read())
-      activate_value = eval(open(activate_simulator).read())
+      build_value = eval(open(build_simulator).read()) #pylint: disable=eval-used
+      activate_value = eval(open(activate_simulator).read()) #pylint: disable=eval-used
 
       self.assertEqual([], build_value)
       self.assertEqual([], activate_value)
@@ -880,8 +880,8 @@ class TestSlapOSTriggerBuildAlarm(SlapOSTestCaseMixin):
       open(activate_simulator, 'w').write(repr([]))
 
       lower_simulation_movement.SimulationMovement_buildSlapOS(tag='lower_tag')
-      build_value = eval(open(build_simulator).read())
-      activate_value = eval(open(activate_simulator).read())
+      build_value = eval(open(build_simulator).read()) #pylint: disable=eval-used
+      activate_value = eval(open(activate_simulator).read()) #pylint: disable=eval-used
 
       self.assertEqual([], build_value)
       self.assertEqual([], activate_value)
@@ -944,8 +944,8 @@ class TestSlapOSManageBuildingCalculatingDeliveryAlarm(SlapOSTestCaseMixin):
 
       delivery.Delivery_manageBuildingCalculatingDelivery()
 
-      updateCausalityState_value = eval(open(updateCausalityState_simulator).read())
-      updateSimulation_value = eval(open(updateSimulation_simulator).read())
+      updateCausalityState_value = eval(open(updateCausalityState_simulator).read()) #pylint: disable=eval-used
+      updateSimulation_value = eval(open(updateSimulation_simulator).read()) #pylint: disable=eval-used
 
       if empty:
         self.assertEqual([], updateCausalityState_value)
@@ -1330,7 +1330,7 @@ class TestSlapOSReindexOpenSaleOrder(SlapOSTestCaseMixin):
     )
     return open_order
 
-  def _simulateScript(self, script_name):
+  def _simulateScript(self, script_name, fake_return="False"):
     if script_name in self.portal.portal_skins.custom.objectIds():
       raise ValueError('Precondition failed: %s exists in custom' % script_name)
     createZODBPythonScript(self.portal.portal_skins.custom,

master/bt5/slapos_accounting/TestTemplateItem/portal_components/test.erp5.testSlapOSAccountingAlarm.xml

@@ -45,19 +45,7 @@
         <item>
             <key> <string>text_content_warning_message</string> </key>
             <value>
-              <tuple>
-                <string>W: 31, 10: Use of eval (eval-used)</string>
-                <string>W:785, 20: Use of eval (eval-used)</string>
-                <string>W:786, 23: Use of eval (eval-used)</string>
-                <string>W:807, 20: Use of eval (eval-used)</string>
-                <string>W:808, 23: Use of eval (eval-used)</string>
-                <string>W:873, 20: Use of eval (eval-used)</string>
-                <string>W:874, 23: Use of eval (eval-used)</string>
-                <string>W:883, 20: Use of eval (eval-used)</string>
-                <string>W:884, 23: Use of eval (eval-used)</string>
-                <string>W:947, 35: Use of eval (eval-used)</string>
-                <string>W:948, 31: Use of eval (eval-used)</string>
-              </tuple>
+              <tuple/>
             </value>
         </item>
         <item>