From 7664059073dd0691428864deb37c3861521e2cd8 Mon Sep 17 00:00:00 2001
From: jardellos <thierry.hardy@orange.com>
Date: Fri, 28 Aug 2020 11:27:47 +0200
Subject: Add logger in python-sdktests

Adding of logger to onaptests setting the stream and file handlers based
on settings.py
onapsdk should evolve to include the same code
This would avoid to add it in run.py
Simplification of the proposal based on remarks from Michal and Morgan
Issue-ID: TEST-252

Signed-off-by: jardellos <thierry.hardy@orange.com>
Change-Id: I5b7baf77580df916c4f5e62965a47d8462b44c43
---
 src/onaptests/configuration/settings.py | 32 ++++++++++++++++++++++++++++++++
 src/onaptests/steps/base.py             | 18 +++++++++++++++++-
 2 files changed, 49 insertions(+), 1 deletion(-)

(limited to 'src')

diff --git a/src/onaptests/configuration/settings.py b/src/onaptests/configuration/settings.py
index f6959de..0e69b4e 100644
--- a/src/onaptests/configuration/settings.py
+++ b/src/onaptests/configuration/settings.py
@@ -6,6 +6,38 @@
 #                    #
 ######################
 
+
+# Variables to set logger information
+# Possible values for logging levels in onapsdk: INFO, DEBUG , WARNING, ERROR
+LOG_CONFIG = {
+    "version": 1,
+    "disable_existing_loggers": False,
+    "formatters": {
+        "default": {
+            "class": "logging.Formatter",
+            "format": "%(asctime)s %(levelname)s %(lineno)d:%(filename)s(%(process)d) - %(message)s"
+        }
+    },
+    "handlers": {
+        "console": {
+            "level": "DEBUG",
+            "class": "logging.StreamHandler",
+            "formatter": "default"
+        },
+        "file": {
+            "level": "DEBUG",
+            "class": "logging.FileHandler",
+            "formatter": "default",
+            "filename": "./pythonsdk.debug.log",
+            "mode": "w"
+        }
+    },
+    "root": {
+        "level": "DEBUG",
+        "handlers": ["console", "file"]
+    }
+}
+
 VENDOR_NAME = "sdktests_vendor"
 VSP_NAME = "sdktests_vsp"
 VSP_FILE_PATH = "vfw.zip"
diff --git a/src/onaptests/steps/base.py b/src/onaptests/steps/base.py
index b32e6d3..5f66935 100644
--- a/src/onaptests/steps/base.py
+++ b/src/onaptests/steps/base.py
@@ -1,10 +1,24 @@
+import logging
+import logging.config
+
 from abc import ABC, abstractmethod
 from typing import List
-
+from onapsdk.configuration import settings
 
 class BaseStep(ABC):
     """Base step class."""
 
+    _logger: logging.Logger = logging.getLogger("")
+
+    def __init_subclass__(cls):
+        """Subclass initialization.
+
+        Add _logger property for any BaseStep subclass
+        """
+        super().__init_subclass__()
+        cls._logger: logging.Logger = logging.getLogger("")
+        logging.config.dictConfig(settings.LOG_CONFIG)
+
     def __init__(self, cleanup: bool = False) -> None:
         """Step initialization.
 
@@ -16,6 +30,8 @@ class BaseStep(ABC):
         self._cleanup: bool = cleanup
         self._parent: "BaseStep" = None
 
+
+
     def add_step(self, step: "BaseStep") -> None:
         """Add substep.
 
-- 
cgit 1.2.3-korg