add vector hardware config popup (hardbyte#774)

zariiii9003 · christiansandberg · web-flow · commit f4c2cff4bf26 · 2020-03-28T19:54:28.000+01:00
Co-authored-by: zariiii9003 &lt;zariiii9003@users.noreply.github.com&gt;
Co-authored-by: Christian Sandberg &lt;christiansandberg@users.noreply.github.com&gt;
diff --git a/can/interfaces/vector/canlib.py b/can/interfaces/vector/canlib.py
@@ -577,6 +577,15 @@ def _detect_available_configs():
             )
         return configs
 
+    @staticmethod
+    def popup_vector_hw_configuration(wait_for_finish: int = 0) -> None:
+        """Open vector hardware configuration window.
+
+        :param int wait_for_finish:
+            Time to wait for user input in milliseconds.
+        """
+        xldriver.xlPopupHwConfig(ctypes.c_char_p(), ctypes.c_uint(wait_for_finish))
+
 
 def get_channel_configs():
     if xldriver is None:
diff --git a/can/interfaces/vector/xldriver.py b/can/interfaces/vector/xldriver.py
@@ -229,3 +229,8 @@ def check_status(result, function, arguments):
 xlCanSetChannelOutput.argtypes = [xlclass.XLportHandle, xlclass.XLaccess, ctypes.c_char]
 xlCanSetChannelOutput.restype = xlclass.XLstatus
 xlCanSetChannelOutput.errcheck = check_status
+
+xlPopupHwConfig = _xlapi_dll.xlPopupHwConfig
+xlPopupHwConfig.argtypes = [ctypes.c_char_p, ctypes.c_uint]
+xlPopupHwConfig.restype = xlclass.XLstatus
+xlPopupHwConfig.errcheck = check_status
diff --git a/test/test_vector.py b/test/test_vector.py
@@ -240,6 +240,14 @@ def test_reset(self) -> None:
         can.interfaces.vector.canlib.xldriver.xlDeactivateChannel.assert_called()
         can.interfaces.vector.canlib.xldriver.xlActivateChannel.assert_called()
 
+    def test_popup_hw_cfg(self) -> None:
+        canlib.xldriver.xlPopupHwConfig = Mock()
+        canlib.VectorBus.popup_vector_hw_configuration(10)
+        assert canlib.xldriver.xlPopupHwConfig.called
+        args, kwargs = canlib.xldriver.xlPopupHwConfig.call_args
+        assert isinstance(args[0], ctypes.c_char_p)
+        assert isinstance(args[1], ctypes.c_uint)
+
     def test_called_without_testing_argument(self) -> None:
         """This tests if an exception is thrown when we are not running on Windows."""
         if os.name != "nt":
