test

Signed-off-by: Xiangce Liu <xiangceliu@redhat.com>

rh-pre-commit.version: 2.3.1
rh-pre-commit.check-secrets: ENABLED

.github/workflows/main.yml

@@ -62,7 +62,7 @@ jobs:
       run: |
         pip install urllib3
         pip install -e .[testing]
-        pytest
+        pytest --log-cli-level=WARN
 
   python26-test:
 
@@ -112,7 +112,7 @@ jobs:
         export PATH=$PATH:/github/home/.local/bin
         pip install --user -e .[testing] -f ./pips/slave26/pip_packages
         export PYTHONPATH=${PYTHONPATH}:./../collections_module
-        pytest
+        pytest --log-cli-level=WARN
 
   docs-test:
 

insights/parsers/uname.py

@@ -414,10 +414,16 @@ def _best_lv_release(self, other):
         o_release_parts = other._lv_release.vstring.split(".")
         is_s_with_dist = s_release_parts[-1].startswith(dist_opts)
         is_o_with_dist = o_release_parts[-1].startswith(dist_opts)
+        import logging
+        logger = logging.getLogger(__name__)
         if not (is_s_with_dist ^ is_o_with_dist):
             return self._lv_release, other._lv_release
-        import warnings
-        warnings.warn('Comparison of distribution part will be ingored.')
+        try:
+            import warnings
+            warnings.warn('Comparison of distribution part will be ignored.')
+        except Exception as ex:
+            print(str(ex))
+        logger.warning('_best_lv_release: after warnings')
         s_release = (
             self._lv_release.vstring if not is_s_with_dist
             else pad_release(".".join(s_release_parts[:-1]), len(s_release_parts))
@@ -586,6 +592,7 @@ def fixed_by(self, *fixes, **kwargs):
         # If there is a fix in your kernel release stream
         # See if you are fixed by it
         for i, fixed in enumerate(fix_unames):
+            print('FIXED BY:', self._rel_maj, fixed._rel_maj)
             if self._rel_maj == fixed._rel_maj:
                 if self._less_than(fixed):
                     return fix_kernels[i:]

insights/tests/parsers/test_uname.py

@@ -243,7 +243,7 @@ def test_fixed_by():
 def test_fixed_by_warning():
     # For all remaining tests, cause the comparation warnings
     # to always be caught, and ignore other warnings.
-    warning_msg = "Comparison of distribution part will be ingored"
+    warning_msg = "Comparison of distribution part will be ignored."
     warnings.simplefilter("ignore")
     warnings.filterwarnings("always", message=warning_msg)
     with warnings.catch_warnings(record=True) as w:
@@ -253,11 +253,13 @@ def test_fixed_by_warning():
         assert [] == u.fixed_by('2.6.32-220.1.el6', '2.6.32-504.el6')
         assert len(w) == 0
 
+        print('---------------------------------')
         # fixes without distribution part
         assert [] == u.fixed_by('2.6.32-504')
         assert len(w) == 1
+        print('---------------------------------')
 
-        # fixes only has kernel version part
+        # frnings.warn('Comparison of distribution part will be ingored.')ixes only has kernel version part
         assert [] == u.fixed_by('2.6.32-')
         assert len(w) == 2