webui: tests: don't apply step_logger decorator for non permian tests

The latter introduces quite some noice and the log view for cockpit tests does not nicely integrate with the extra screenshots.
rhinstaller · Jun 19, 2023 · fe0192b · fe0192b
1 parent 8484637
commit fe0192b
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 0 deletions.
diff --git a/ui/webui/test/helpers/permian.py b/ui/webui/test/helpers/permian.py
@@ -49,6 +49,7 @@ def setUp(self):
  self.logs_dir = os.path.join('./test_logs', self.__class__.__name__)
  if not os.path.isdir(self.logs_dir):
  os.makedirs(self.logs_dir)
+ os.environ["PERMIAN"] = True
 
  def __add_public_key(self):
  with open(self.machine.identity_file + '.pub', 'r') as pub:

diff --git a/ui/webui/test/helpers/step_logger.py b/ui/webui/test/helpers/step_logger.py
@@ -1,3 +1,4 @@
+import os
 
 class BrowserSnapshot():
  SNAPSHOT_NUMBER = 0
@@ -23,6 +24,8 @@ def log_step(snapshots=False, snapshot_before=False, snapshot_after=False, docst
  :type docstring: bool, optional
  """
  def decorator(function):
+ if not os.environ.get('PERMIAN', False):
+ return function
 
  def wrapper(*args, **kwargs):
  nice_args = ', '.join(str(a) for a in args[1:])