From e7ace6cc999b9da25429e9197d5294cf4d0d2d82 Mon Sep 17 00:00:00 2001 From: Ian Chua Date: Tue, 15 Sep 2026 16:46:28 +0800 Subject: [PATCH] fix: unit tests & unused variables --- src/slic3r/GUI/CameraPopup.cpp | 2 +- src/slic3r/GUI/StatusPanel.cpp | 2 +- tests/CMakeLists.txt | 16 +++++++++++++--- tests/slic3rutils/CMakeLists.txt | 4 ++-- 4 files changed, 17 insertions(+), 7 deletions(-) diff --git a/src/slic3r/GUI/CameraPopup.cpp b/src/slic3r/GUI/CameraPopup.cpp index 4a52cb4b2f..099edd7de0 100644 --- a/src/slic3r/GUI/CameraPopup.cpp +++ b/src/slic3r/GUI/CameraPopup.cpp @@ -81,7 +81,7 @@ CameraPopup::CameraPopup(wxWindow *parent) top_sizer->Add(m_text_liveview_retry, 0, wxALIGN_CENTER_VERTICAL | wxALIGN_LEFT | wxALL, FromDIP(5)); top_sizer->Add(m_switch_liveview_retry, 0, wxALIGN_CENTER_VERTICAL | wxALIGN_RIGHT | wxALL, FromDIP(5)); - m_switch_liveview_retry->Bind(wxEVT_TOGGLEBUTTON, [this](wxCommandEvent &e) { + m_switch_liveview_retry->Bind(wxEVT_TOGGLEBUTTON, [](wxCommandEvent &e) { wxGetApp().app_config->set("liveview", "auto_retry", e.IsChecked()); e.Skip(); }); diff --git a/src/slic3r/GUI/StatusPanel.cpp b/src/slic3r/GUI/StatusPanel.cpp index 7099f3fea9..b97d6210e1 100644 --- a/src/slic3r/GUI/StatusPanel.cpp +++ b/src/slic3r/GUI/StatusPanel.cpp @@ -1463,7 +1463,7 @@ wxBoxSizer *StatusBasePanel::create_monitoring_page() #if !BBL_RELEASE_TO_PUBLIC m_staticText_timelapse->Show(); m_bmToggleBtn_timelapse->Show(); - m_bmToggleBtn_timelapse->Bind(wxEVT_TOGGLEBUTTON, [this](wxCommandEvent &e) { + m_bmToggleBtn_timelapse->Bind(wxEVT_TOGGLEBUTTON, [](wxCommandEvent &e) { if (e.IsChecked()) wxGetApp().getAgent()->start_subscribe("tunnel"); else diff --git a/tests/CMakeLists.txt b/tests/CMakeLists.txt index cd9bfea1c2..7539369c64 100644 --- a/tests/CMakeLists.txt +++ b/tests/CMakeLists.txt @@ -57,18 +57,28 @@ function(orcaslicer_copy_test_dlls) # into it). Copy them next to the executable and give it an $ORIGIN # rpath so the loader finds them when the tests run on the CI unit-test # runner, which only receives this build/tests tree. - get_target_property(_linked_libs ${_TEST_NAME}_tests LINK_LIBRARIES) + # + # Optional first arg: the target to patch, for directories that define + # more than one test executable. Defaults to ${_TEST_NAME}_tests so + # existing single-executable callers don't need to pass it. + if (ARGC GREATER 0) + set(_target ${ARGV0}) + else() + set(_target ${_TEST_NAME}_tests) + endif() + + get_target_property(_linked_libs ${_target} LINK_LIBRARIES) if (NOT "libslic3r_gui" IN_LIST _linked_libs) return() endif() - set_property(TARGET ${_TEST_NAME}_tests PROPERTY BUILD_RPATH "$ORIGIN") + set_property(TARGET ${_target} PROPERTY BUILD_RPATH "$ORIGIN") set(_configs ${CMAKE_CONFIGURATION_TYPES}) if (NOT _configs) set(_configs "${CMAKE_BUILD_TYPE}") endif() foreach(_cfg IN LISTS _configs) - orcaslicer_copy_sos(${_TEST_NAME}_tests "${_cfg}" "" _unused_sos) + orcaslicer_copy_sos(${_target} "${_cfg}" "" _unused_sos) endforeach() endif() endfunction() diff --git a/tests/slic3rutils/CMakeLists.txt b/tests/slic3rutils/CMakeLists.txt index fbf0eaa901..11aca7dac6 100644 --- a/tests/slic3rutils/CMakeLists.txt +++ b/tests/slic3rutils/CMakeLists.txt @@ -31,7 +31,7 @@ endif () target_link_libraries(${_TEST_NAME}_tests test_common libslic3r_gui libslic3r pybind11::embed Catch2::Catch2WithMain) set_property(TARGET ${_TEST_NAME}_tests PROPERTY FOLDER "tests") -orcaslicer_copy_test_dlls() +orcaslicer_copy_test_dlls(${_TEST_NAME}_tests) orcaslicer_stage_test_python_runtime(${_TEST_NAME}_tests) @@ -52,7 +52,7 @@ set_property(TARGET printer_agent_plugin_tests PROPERTY FOLDER "tests") # beside the shared test output dir; serialize them so the copies can't race. add_dependencies(printer_agent_plugin_tests ${_TEST_NAME}_tests) -orcaslicer_copy_test_dlls() +orcaslicer_copy_test_dlls(printer_agent_plugin_tests) orcaslicer_stage_test_python_runtime(printer_agent_plugin_tests)