[WIP] Add unit tests for GUI Widgets.

.coveragerc

@@ -2,4 +2,4 @@
 source =
     asammdf
 omit =
-    */asammdf/gui/*
+    */asammdf/gui/ui/*

.github/workflows/build.yml

@@ -38,6 +38,9 @@ jobs:
             if [ "$RUNNER_OS" == "Linux" ]; then
                  sudo apt update
                  sudo apt install libegl1 libopengl0
+                 sudo apt-get install '^libxcb.*-dev' libx11-xcb-dev libglu1-mesa-dev libxrender-dev libxi-dev libxkbcommon-dev libxkbcommon-x11-dev
+                 sudo apt-get install libxcb-xinerama0
+                 sudo apt-get install libxkbcommon-x11-0
             fi
             pyinstaller asammdf.spec --distpath dist/${RUNNER_OS} --noconfirm --clean
 

.github/workflows/coveralls.yml

@@ -19,11 +19,30 @@ jobs:
           python-version: "3.8"
 
       - name: Install packages
-        run: pip install tox coveralls
+        shell: bash
+        run: |
+          pip install tox coveralls
+          if [ "$RUNNER_OS" == "Linux" ]; then
+             sudo apt update
+             sudo apt-get install libegl1 libopengl0
+             sudo apt-get install '^libxcb.*-dev' libx11-xcb-dev libglu1-mesa-dev libxrender-dev libxi-dev libxkbcommon-dev libxkbcommon-x11-dev
+             sudo apt-get install libxcb-xinerama0
+             sudo apt-get install libxkbcommon-x11-0
+             sudo apt-get install xorg
+             sudo apt-get install -y xvfb
+          fi
+
+      - name: Start Xvfb
+        run: |
+          Xvfb :0 -screen 0 1024x768x24 > /dev/null 2>&1 &
+          export DISPLAY=:0
 
       - name: Run coverage
         run: tox -e py38
 
+      - name: Stop Xvfb
+        run: killall Xvfb
+
       # see: https://coveralls-python.readthedocs.io/en/latest/usage/configuration.html#github-actions-support
       - name: Upload coverage data to coveralls.io
         run: coveralls --service=github

.github/workflows/main.yml

@@ -21,10 +21,30 @@ jobs:
           python-version: ${{ matrix.python-version }}
 
       - name: Install packages
+        shell: bash
         run: |
             pip install tox tox-gh-actions
+            if [ "$RUNNER_OS" == "Linux" ]; then
+               sudo apt update
+               sudo apt-get install libegl1 libopengl0
+               sudo apt-get install '^libxcb.*-dev' libx11-xcb-dev libglu1-mesa-dev libxrender-dev libxi-dev libxkbcommon-dev libxkbcommon-x11-dev
+               sudo apt-get install libxcb-xinerama0
+               sudo apt-get install libxkbcommon-x11-0
+               sudo apt-get install xorg
+               sudo apt-get install -y xvfb
+            fi
+
+      - name: Start Xvfb
+        if: matrix.os == 'ubuntu-22.04'
+        run: |
+            Xvfb :0 -screen 0 1024x768x24 > /dev/null 2>&1 &
+            export DISPLAY=:0
 
       # see: https://github.com/ymyzk/tox-gh-actions
       - name: Run tests
         run: tox
+
+      - name: Stop Xvfb
+        if: matrix.os == 'ubuntu-22.04'
+        run: killall Xvfb
 

.github/workflows/wheels.yml

@@ -23,7 +23,15 @@ jobs:
         run: python -m pip install cibuildwheel
 
       - name: Build wheels
-        run: python -m cibuildwheel --output-dir wheelhouse
+        shell: bash
+        env:
+          CIBW_TEST_COMMAND: "pytest {project}/test --ignore={project}/test/asammdf/gui"
+        run: |
+          if [ "$RUNNER_OS" == "Linux" ]; then
+             sudo apt update
+             sudo apt-get install libegl1 libopengl0
+          fi
+          python -m cibuildwheel --output-dir wheelhouse
 
       - uses: actions/upload-artifact@v3
         with:

.gitignore

@@ -126,6 +126,9 @@ pythonenv*
 # Rope project settings
 .ropeproject
 
+# PyCharm
+.idea/*
+
 # mkdocs documentation
 /site
 

pyproject.toml

@@ -3,6 +3,7 @@ requires = ["numpy", "setuptools", "wheel"]
 build-backend = "setuptools.build_meta"
 
 [tool.black]
+target-version = ['py38']
 extend-exclude = '''
 ^/src/asammdf/gui/ui
 '''

requirements_exe_build.txt

@@ -9,8 +9,8 @@ lxml
 natsort 
 psutil 
 PySide6==6.2.2
-pyqtgraph>=0.12.4; sys_platform=="win32"
-pyqtgraph>=0.12.4; sys_platform=="darwin"
+pyqtgraph==0.12.4; sys_platform=="win32"
+pyqtgraph==0.12.4; sys_platform=="darwin"
 pyqtgraph==0.12.4; sys_platform=="linux"
 pyqtlet2
 pyopengl

src/asammdf/blocks/utils.py

@@ -2450,7 +2450,8 @@ def load_lab(file):
                 sections[section_name] = s
 
             else:
-                s.append(line)
+                if "s" in locals():
+                    s.append(line)
 
     return {name: channels for name, channels in sections.items() if channels}
 

src/asammdf/gui/widgets/file.py

@@ -1039,19 +1039,19 @@ def load_channel_list(self, event=None, file_name=None):
 
             elif extension == ".lab":
                 info = load_lab(file_name)
-                if info:
-                    section, ok = QtWidgets.QInputDialog.getItem(
-                        None,
-                        "Select section",
-                        "Available sections:",
-                        list(info),
-                        0,
-                        False,
-                    )
-                    if ok:
-                        channels = info[section]
-                    else:
-                        return
+                if not info:
+                    return
+                section, ok = QtWidgets.QInputDialog.getItem(
+                    None,
+                    "Select section",
+                    "Available sections:",
+                    list(info),
+                    0,
+                    False,
+                )
+                if not ok:
+                    return
+                channels = info[section]
 
             elif extension in (".cfg", ".txt", ".dspf"):
                 with open(file_name, "r") as infile:

src/asammdf/gui/widgets/plot.py

@@ -3881,7 +3881,7 @@ def __init__(
 
         if x_axis == "time":
             fmt = self._settings.value("plot_xaxis")
-            if fmt == "seconds":
+            if fmt == "seconds" or not fmt:
                 fmt = "phys"
         else:
             fmt = "phys"
@@ -6177,12 +6177,13 @@ def open_menu(self, point):
                 QtCore.QSettings().setValue("plot_cursor_precision", precision)
 
     def update_value(self):
+        cursor_info_text = ""
         if not self.plot.region:
             if self.plot.cursor1 is not None:
                 position = self.plot.cursor1.value()
 
                 fmt = self.plot.x_axis.format
-                if fmt == "phys":
+                if fmt == "phys" or not fmt:
                     if self.precision == -1:
                         cursor_info_text = f"{self.name} = {position}{self.unit}"
                     else:
@@ -6195,7 +6196,9 @@ def update_value(self):
                         seconds=position
                     )
                     cursor_info_text = f"{self.name} = {position_date}"
-                self.setText(cursor_info_text)
+
+                if cursor_info_text:
+                    self.setText(cursor_info_text)
             else:
                 self.setText("")
 

src/asammdf/gui/widgets/tree.py

@@ -303,7 +303,7 @@ def get_data(item):
 
         drag = QtGui.QDrag(self)
         drag.setMimeData(mimeData)
-        drag.exec_(QtCore.Qt.MoveAction)
+        drag.exec(QtCore.Qt.MoveAction)
 
 
 class FileTreeItem(QtWidgets.QTreeWidgetItem):

test/asammdf/gui/__init__.py

@@ -0,0 +1,66 @@
+#!/usr/bin/env python
+import os
+import sys
+
+PYSIDE = "PySide"
+PYSIDE2 = "PySide2"
+PYSIDE6 = "PySide6"
+PYQT4 = "PyQt4"
+PYQT5 = "PyQt5"
+PYQT6 = "PyQt6"
+
+QT_LIB = os.getenv("PYQTGRAPH_QT_LIB")
+
+if QT_LIB is not None:
+    try:
+        __import__(QT_LIB)
+    except ModuleNotFoundError:
+        pass
+        # raise ModuleNotFoundError(
+        #     f"Environment variable PYQTGRAPH_QT_LIB is set to '{os.getenv('PYQTGRAPH_QT_LIB')}',"
+        #     f" but no module with this name was found."
+        # )
+
+## Automatically determine which Qt package to use (unless specified by
+## environment variable).
+## This is done by first checking to see whether one of the libraries
+## is already imported. If not, then attempt to import in the order
+## specified in libOrder.
+libOrder = [PYQT6, PYSIDE6, PYQT5, PYSIDE2]
+if QT_LIB is None:
+    for lib in libOrder:
+        if lib in sys.modules:
+            QT_LIB = lib
+            break
+
+if QT_LIB is None:
+    for lib in libOrder:
+        qt = lib + ".QtCore"
+        try:
+            __import__(qt)
+            QT_LIB = lib
+            break
+        except ImportError:
+            pass
+
+if QT_LIB is None:
+    raise Exception(
+        "PyQtGraph requires one of PyQt5, PyQt6, PySide2 or PySide6; none of these packages could be imported."
+    )
+
+mandatory = ["QtCore", "QtGui", "QtQuick", "QtTest", "QtWidgets"]
+QT_LIB = __import__(f"{QT_LIB}", fromlist=mandatory)
+if not QT_LIB:
+    raise ModuleNotFoundError(
+        f"At least one of the following modules are not present in {QT_LIB}: {mandatory}"
+    )
+
+for m in mandatory:
+    globals()[m] = getattr(QT_LIB, m)
+
+# QtCore = getattr(QT_LIB, "QtCore")
+# QtGui = getattr(QT_LIB, "QtGui")
+# QtTest = getattr(QT_LIB, "QtTest")
+# QtWidgets = getattr(QT_LIB, "QtWidgets")
+
+__all__ = mandatory

test/asammdf/gui/resources/invalid_EmptySection.lab

@@ -0,0 +1 @@
+[lab]