Revert "Enable qInfo tests for PySide6 (#593)"

This reverts commit ce1a689.

This change is incorrect as it breaks things with PySide < 6.8.2, but we support
running against older versions.

The next commit will fix this properly without breaking compatibility.

See #232

Author: The-Compiler

src/pytestqt/qt_compat.py

@@ -111,7 +111,14 @@ def _import_module(module_name):
 
         self._check_qt_api_version()
 
-        self.qInfo = QtCore.qInfo
+        # qInfo is not exposed in PySide6 (#232)
+        if hasattr(QtCore, "QMessageLogger"):
+            self.qInfo = lambda msg: QtCore.QMessageLogger().info(msg)
+        elif hasattr(QtCore, "qInfo"):
+            self.qInfo = QtCore.qInfo
+        else:
+            self.qInfo = None
+
         self.qDebug = QtCore.qDebug
         self.qWarning = QtCore.qWarning
         self.qCritical = QtCore.qCritical

tests/test_basics.py

@@ -613,7 +613,6 @@ class Mock:
     qtcore = Mock()
     for method_name in (
         "qInstallMessageHandler",
-        "qInfo",
         "qDebug",
         "qWarning",
         "qCritical",

tests/test_logging.py

@@ -26,7 +26,8 @@ def print_msg(msg_type, context, message):
         qt_api.QtCore.qInstallMessageHandler(print_msg)
 
         def test_types():
-            qt_api.qInfo('this is an INFO message')
+            # qInfo is not exposed by the bindings yet (#225)
+            # qt_api.qInfo('this is an INFO message')
             qt_api.qDebug('this is a DEBUG message')
             qt_api.qWarning('this is a WARNING message')
             qt_api.qCritical('this is a CRITICAL message')
@@ -44,7 +45,8 @@ def test_types():
             res.stdout.fnmatch_lines(
                 [
                     "*-- Captured Qt messages --*",
-                    "*QtInfoMsg: this is an INFO message*",
+                    # qInfo is not exposed by the bindings yet (#232)
+                    # '*QtInfoMsg: this is an INFO message*',
                     "*QtDebugMsg: this is a DEBUG message*",
                     "*QtWarningMsg: this is a WARNING message*",
                     "*QtCriticalMsg: this is a CRITICAL message*",
@@ -54,25 +56,43 @@ def test_types():
             res.stdout.fnmatch_lines(
                 [
                     "*-- Captured stderr call --*",
-                    "this is an INFO message*",
+                    # qInfo is not exposed by the bindings yet (#232)
+                    # '*QtInfoMsg: this is an INFO message*',
+                    # 'this is an INFO message*',
                     "this is a DEBUG message*",
                     "this is a WARNING message*",
                     "this is a CRITICAL message*",
                 ]
             )
 
 
+def test_qinfo(qtlog):
+    """Test INFO messages when we have means to do so. Should be temporary until bindings
+    catch up and expose qInfo (or at least QMessageLogger), then we should update
+    the other logging tests properly. #232
+    """
+
+    if qt_api.is_pyside:
+        assert (
+            qt_api.qInfo is None
+        ), "pyside6 does not expose qInfo. If it does, update this test."
+        return
+
+    qt_api.qInfo("this is an INFO message")
+    records = [(m.type, m.message.strip()) for m in qtlog.records]
+    assert records == [(qt_api.QtCore.QtMsgType.QtInfoMsg, "this is an INFO message")]
+
+
 def test_qtlog_fixture(qtlog):
     """
     Test qtlog fixture.
     """
-    qt_api.qInfo("this is an INFO message")
+    # qInfo is not exposed by the bindings yet (#232)
     qt_api.qDebug("this is a DEBUG message")
     qt_api.qWarning("this is a WARNING message")
     qt_api.qCritical("this is a CRITICAL message")
     records = [(m.type, m.message.strip()) for m in qtlog.records]
     assert records == [
-        (qt_api.QtCore.QtMsgType.QtInfoMsg, "this is an INFO message"),
         (qt_api.QtCore.QtMsgType.QtDebugMsg, "this is a DEBUG message"),
         (qt_api.QtCore.QtMsgType.QtWarningMsg, "this is a WARNING message"),
         (qt_api.QtCore.QtMsgType.QtCriticalMsg, "this is a CRITICAL message"),