tests: cargo only test couldn't run because of missing symbol compile…

…d by cmake project

.github/workflows/github-cxx-qt-tests.yml

@@ -91,7 +91,7 @@ jobs:
             # FIXME: many tests fail to link
             # https://github.com/KDAB/cxx-qt/issues/111
             # cxx_qt_lib_cargo_tests fails to run with a missing DLL error.
-            ctest_args: --exclude-regex '^(test_basic_cxx_only_cargo_tests|reuse_lint|cpp_clang_format|.*valgrind)$'
+            ctest_args: --exclude-regex '^(reuse_lint|cpp_clang_format|.*valgrind)$'
             exe_suffix: .exe
             qt_qpa_platform: windows
             compiler_cache_path: C:\Users\runneradmin\AppData\Local\Mozilla\sccache\cache
@@ -106,7 +106,7 @@ jobs:
             # FIXME: many tests fail to link
             # https://github.com/KDAB/cxx-qt/issues/111
             # cxx_qt_lib_cargo_tests fails to run with a missing DLL error.
-            ctest_args: --exclude-regex '^(test_basic_cxx_only_cargo_tests|reuse_lint|cpp_clang_format|.*valgrind)$'
+            ctest_args: --exclude-regex '^(reuse_lint|cpp_clang_format|.*valgrind)$'
             exe_suffix: .exe
             qt_qpa_platform: windows
             compiler_cache_path: C:\Users\runneradmin\AppData\Local\Mozilla\sccache\cache

tests/basic_cxx_only/cpp/cxx_test.cpp

@@ -0,0 +1,25 @@
+// clang-format off
+// SPDX-FileCopyrightText: 2023 Klarälvdalens Datakonsult AB, a KDAB Group company <[email protected]>
+// clang-format on
+// SPDX-FileContributor: Andrew Hayzen <[email protected]>
+// SPDX-FileContributor: Gerhard de Clercq <[email protected]>
+//
+// SPDX-License-Identifier: MIT OR Apache-2.0
+
+#include "cxx_test.h"
+
+namespace {
+int hidden_num = 100;
+}
+
+int
+get_cpp_number()
+{
+  return hidden_num;
+}
+
+void
+set_cpp_number(int num)
+{
+  hidden_num = num;
+}

tests/basic_cxx_only/cpp/cxx_test.h

@@ -9,3 +9,6 @@
 
 int
 get_cpp_number();
+
+void
+set_cpp_number(int num);

tests/basic_cxx_only/cpp/main.cpp

@@ -8,14 +8,7 @@
 #include <QtTest/QTest>
 
 #include "cxx-qt-gen/ffi.cxx.h"
-
-int hidden_num = 100;
-
-int
-get_cpp_number()
-{
-  return hidden_num;
-}
+#include "cxx_test.h"
 
 class CxxTest : public QObject
 {
@@ -26,7 +19,7 @@ private Q_SLOTS:
   void test_cxx_interaction()
   {
     QCOMPARE(get_numbers_sum(), 102);
-    hidden_num = 200;
+    set_cpp_number(200);
     QCOMPARE(get_numbers_sum(), 202);
   }
 };

tests/basic_cxx_only/rust/build.rs

@@ -10,6 +10,10 @@ fn main() {
         .file("src/lib.rs")
         .cc_builder(|cc| {
             cc.include("../cpp");
+            // cxx_test.cpp need to be compiled by cargo rather than CMakeLists.txt,
+            // otherwise linking cargo tests fails because the symbols from those files are not found.
+            // This to make cargo only tests work.
+            cc.file("../cpp/cxx_test.cpp");
         })
         .build();
 }

tests/basic_cxx_only/rust/src/lib.rs

@@ -19,3 +19,13 @@ mod ffi {
 fn get_numbers_sum() -> i32 {
     ffi::get_cpp_number() + 2
 }
+
+#[cfg(test)]
+mod tests {
+    use super::ffi::get_cpp_number;
+
+    #[test]
+    fn test_get_numbers_sum() {
+        assert_eq!(get_cpp_number(), 100);
+    }
+}