Merge lp:~charlesk/keeper/get-coverage-reports-working into lp:keeper

[Charles Kerr]

Get coverage reports working.

Does what it says on the tin: coverage reports now work.

1. Added cmake variable COVERAGE_TEST_TARGETS, which gets appended to by the subdirectories in src/ with parent scope so that it's visible by the call to enable_coverage_report() in the toplevel CMakeLists.txt.

2. Added cmake variable UNIT_TEST_TARGETS which gets appended to by the subdirectories in tests/ with parent scope so so it, too, is visible to enable_coverage_report()

3. Recursively building UNIT_TEST_TARGETS means we no longer have to have a single 'unit-tests' executable, so break the unit tests into standalone executables. This was the secondary goal of this branch -- making it easier to run a single unit test again and again during a debugging cycle

4. Minor copyediting: use more consistent spacing (eg 2 spaces vs 4) in the CMakeLists.txt files, use target_link_libraries(Qt5::Core) instead of qt5_use_libraries(Core)

[lp:~charlesk/keeper/get-coverage-reports-working]

46. By Charles Kerr on 2016-07-22

copyediting: whitespace

47. By Charles Kerr on 2016-07-22

sync with trunk

[Xavi Garcia (xavi-garcia-mena)]

I think you forgot to add/commit a couple of files:

CMake Error at tests/unit/CMakeLists.txt:33 (add_subdirectory):
  The source directory

    /tmp/get-coverage-reports-working/tests/unit/metadata-providers

  does not contain a CMakeLists.txt file.

CMake Error at tests/unit/CMakeLists.txt:34 (add_subdirectory):
  The source directory

    /tmp/get-coverage-reports-working/tests/unit/tar

  does not contain a CMakeLists.txt file.

review: Needs Fixing

[lp:~charlesk/keeper/get-coverage-reports-working]

48. By Charles Kerr on 2016-07-22

fix oops -- forgot to bzr add a couple of CMakeLists.txt

[Xavi Garcia (xavi-garcia-mena)]

Looks good now, thanks!

review: Approve

=== modified file 'CMakeLists.txt'
--- CMakeLists.txt	2016-07-05 21:09:03 +0000
+++ CMakeLists.txt	2016-07-22 13:14:22 +0000
@@ -254,9 +254,10 @@
 
 enable_coverage_report(
     TARGETS
+        ${COVERAGE_REPORT_TARGETS}
     FILTER
         ${CMAKE_SOURCE_DIR}/tests/*
         ${CMAKE_BINARY_DIR}/*
     TESTS
-        unit-tests
+        ${UNIT_TEST_TARGETS}
 )

=== added file 'HACKING'
--- HACKING	1970-01-01 00:00:00 +0000
+++ HACKING	2016-07-22 13:14:22 +0000
@@ -0,0 +1,64 @@
+Building the code
+-----------------
+
+By default, the code is built in release mode. To build a debug version, use
+
+    $ mkdir builddebug
+    $ cd builddebug
+    $ cmake -DCMAKE_BUILD_TYPE=debug ..
+    $ make
+
+For a release version, use -DCMAKE_BUILD_TYPE=release
+
+
+Running the tests
+-----------------
+
+    $ make
+    $ make test
+
+Note that "make test" alone is dangerous because it does not rebuild
+any tests if either the library or the test files themselves need
+rebuilding. It's not possible to fix this with cmake because cmake cannot
+add build dependencies to built-in targets. To make sure that everything
+is up-to-date, run "make" before running "make test"!
+
+
+Coverage
+--------
+
+To build with the flags for coverage testing enabled and get coverage:
+
+    $ mkdir buildcoverage
+    $ cd buildcoverage
+    $ cmake -DCMAKE_BUILD_TYPE=coverage ..
+    $ make
+    $ make test
+    $ make coverage
+    $ make coverage-html
+
+After "make coverage-html" a report will be viewable in
+./buildcoverage/coveragereport/index.html
+
+Unfortunately, it is not possible to get 100% coverage for some files,
+mainly due to gcc's generation of two destructors for dynamic and non-
+dynamic instances. For abstract base classes and for classes that
+prevent stack and static allocation, this causes one of the destructors
+to be reported as uncovered.
+
+There are also issues with some functions in header files that are
+incorrectly reported as uncovered due to inlining, as well as
+the impossibility of covering defensive assert(false) statements,
+such as an assert in the default branch of a switch, where the
+switch is meant to handle all possible cases explicitly.
+
+If you run a binary and get lots of warnings about a "merge mismatch for summaries",
+this is caused by having made changes to the source that add or remove code
+that was previously run, so the new coverage output cannot sensibly be merged
+into the old coverage output. You can get rid of this problem by running
+
+    $ make clean-coverage
+
+This deletes all the .gcda files, allowing the merge to (sometimes) succeed again.
+If this doesn't work either, the only remedy is to do a clean build.
+

=== modified file 'src/CMakeLists.txt'
--- src/CMakeLists.txt	2016-07-13 16:09:04 +0000
+++ src/CMakeLists.txt	2016-07-22 13:14:22 +0000
@@ -11,3 +11,9 @@
 add_subdirectory(storage-framework)
 add_subdirectory(simple-helper)
 add_subdirectory(tar)
+
+set(
+  COVERAGE_REPORT_TARGETS
+  ${COVERAGE_REPORT_TARGETS}
+  PARENT_SCOPE
+)

=== modified file 'src/helper/CMakeLists.txt'
--- src/helper/CMakeLists.txt	2016-07-12 20:17:22 +0000
+++ src/helper/CMakeLists.txt	2016-07-22 13:14:22 +0000
@@ -1,9 +1,13 @@
-
 include(FindPkgConfig)
-pkg_check_modules(UAL REQUIRED ubuntu-app-launch-2)
-include_directories(SYSTEM ${UAL_INCLUDE_DIRS})
-pkg_check_modules(PROP-CPP REQUIRED properties-cpp)
-include_directories(SYSTEM ${PROP-CPP_INCLUDE_DIRS})
+pkg_check_modules(BACKUP_HELPER_DEPENDENCIES REQUIRED
+  ubuntu-app-launch-2
+  properties-cpp
+)
+
+include_directories(
+  SYSTEM
+  ${BACKUP_HELPER_DEPENDENCIES_INCLUDE_DIRS}
+)
 
 add_library(
   backup-helper
@@ -14,11 +18,19 @@
   ${CMAKE_SOURCE_DIR}/include/helper/backup-helper.h
 )
 
-qt5_use_modules(backup-helper Core DBus Network)
-set_target_properties(backup-helper PROPERTIES AUTOMOC TRUE)
+set_target_properties(
+  backup-helper
+  PROPERTIES
+  AUTOMOC TRUE
+)
 
-target_link_libraries(backup-helper
-                      ${UAL_LIBRARIES})
+target_link_libraries(
+  backup-helper
+  ${BACKUP_HELPER_DEPENDENCIES_LIBRARIES}
+  Qt5::Core
+  Qt5::DBus
+  Qt5::Network
+)
 
 install(
   FILES ${CMAKE_SOURCE_DIR}/src/helper/exec-tool

=== modified file 'src/service/CMakeLists.txt'
--- src/service/CMakeLists.txt	2016-07-13 16:09:04 +0000
+++ src/service/CMakeLists.txt	2016-07-22 13:14:22 +0000
@@ -19,6 +19,13 @@
   ${SERVICE_LIB_SOURCES}
 )
 
+set(
+  COVERAGE_REPORT_TARGETS
+  ${COVERAGE_REPORT_TARGETS}
+  ${SERVICE_LIB}
+  PARENT_SCOPE
+)
+
 set(SERVICE_EXEC_SOURCES
   main.cpp
 )
@@ -49,3 +56,10 @@
     ${SERVICE_EXEC}
   RUNTIME DESTINATION ${CMAKE_INSTALL_FULL_PKGLIBEXECDIR}
 )
+
+set(
+  COVERAGE_REPORT_TARGETS
+  ${COVERAGE_REPORT_TARGETS}
+  ${SERVICE_EXEC}
+  PARENT_SCOPE
+)

=== modified file 'src/storage-framework/CMakeLists.txt'
--- src/storage-framework/CMakeLists.txt	2016-06-25 19:34:21 +0000
+++ src/storage-framework/CMakeLists.txt	2016-07-22 13:14:22 +0000
@@ -1,13 +1,30 @@
+set(
+  LIB_NAME
+  storage-framework
+)
 
 add_library(
-  storage-framework
+  ${LIB_NAME}
   STATIC
   storage_framework_client.cpp
   storage_framework_client.h
 )
 
-qt5_use_modules(storage-framework Core Network)
-set_target_properties(backup-helper PROPERTIES AUTOMOC TRUE)
+set_target_properties(
+  ${LIB_NAME}
+  PROPERTIES
+  AUTOMOC TRUE
+)
+target_link_libraries(
+  ${LIB_NAME}
+  -lstorage-framework-qt-local-client
+  Qt5::Core
+  Qt5::Network
+)
 
-target_link_libraries(storage-framework
-                      -lstorage-framework-qt-local-client)
+set(
+  COVERAGE_REPORT_TARGETS
+  ${COVERAGE_REPORT_TARGETS}
+  ${LIB_NAME}
+  PARENT_SCOPE
+)

=== modified file 'src/tar/CMakeLists.txt'
--- src/tar/CMakeLists.txt	2016-07-07 12:35:22 +0000
+++ src/tar/CMakeLists.txt	2016-07-22 13:14:22 +0000
@@ -2,45 +2,53 @@
 set(APP_NAME "keeper-tar-create")
 
 set(LIB_SOURCES
-    tar-creator.cpp
+  tar-creator.cpp
 )
 add_library(
-    ${LIB_NAME}
-    STATIC
-    ${LIB_SOURCES}
+  ${LIB_NAME}
+  STATIC
+  ${LIB_SOURCES}
 )
 
 set_property(
-    SOURCE main.cpp
-    PROPERTIES APPEND_STRING PROPERTY COMPILE_DEFINITIONS APP_NAME=\"${APP_NAME}\"
+  SOURCE main.cpp
+  PROPERTIES APPEND_STRING PROPERTY COMPILE_DEFINITIONS APP_NAME=\"${APP_NAME}\"
 )
 
 set(APP_SOURCES
-    main.cpp
+  main.cpp
 )
 
 add_executable(
-    ${APP_NAME}
-    ${APP_SOURCES}
+  ${APP_NAME}
+  ${APP_SOURCES}
 )
 
 link_directories(
-    ${SERVICE_DEPS_LIBRARY_DIRS}
+  ${SERVICE_DEPS_LIBRARY_DIRS}
 )
 
 target_link_libraries(
-    ${APP_NAME}
-    ${LIB_NAME}
-    qdbus-stubs
-    storage-framework
-    backup-helper
-    ${SERVICE_DEPS_LIBRARIES}
-    Qt5::Core
-    Qt5::DBus
+  ${APP_NAME}
+  ${LIB_NAME}
+  qdbus-stubs
+  storage-framework
+  backup-helper
+  ${SERVICE_DEPS_LIBRARIES}
+  Qt5::Core
+  Qt5::DBus
 )
 
 install(
-    TARGETS
-        ${APP_NAME}
-    RUNTIME DESTINATION ${CMAKE_INSTALL_FULL_PKGLIBEXECDIR}
+  TARGETS
+    ${APP_NAME}
+  RUNTIME DESTINATION ${CMAKE_INSTALL_FULL_PKGLIBEXECDIR}
+)
+
+set(
+  COVERAGE_REPORT_TARGETS
+  ${COVERAGE_REPORT_TARGETS}
+  ${LIB_NAME}
+  ${APP_NAME}
+  PARENT_SCOPE
 )

=== modified file 'tests/CMakeLists.txt'
--- tests/CMakeLists.txt	2016-07-08 01:59:54 +0000
+++ tests/CMakeLists.txt	2016-07-22 13:14:22 +0000
@@ -52,3 +52,9 @@
 add_subdirectory(dbusmock)
 add_subdirectory(unit)
 add_subdirectory(utils)
+
+set(
+  UNIT_TEST_TARGETS
+  ${UNIT_TEST_TARGETS}
+  PARENT_SCOPE
+)
\ No newline at end of file

=== modified file 'tests/unit/CMakeLists.txt'
--- tests/unit/CMakeLists.txt	2016-07-21 04:43:41 +0000
+++ tests/unit/CMakeLists.txt	2016-07-22 13:14:22 +0000
@@ -1,69 +1,40 @@
-
-
-add_definitions (-DCMAKE_SOURCE_DIR="${CMAKE_CURRENT_SOURCE_DIR}")
-add_definitions (-DTEST_SIMPLE_HELPER="${CMAKE_BINARY_DIR}/src/simple-helper/simple-helper")
-add_definitions (-DKEEPER_SERVICE_BIN="${CMAKE_BINARY_DIR}/src/service/keeper-service")
-add_definitions (-DKEEPER_CLIENT_BIN="${CMAKE_BINARY_DIR}/src/cli/keeper")
-add_definitions (-DTEST_SIMPLE_HELPER_SH="${CMAKE_SOURCE_DIR}/src/simple-helper/helper-test.sh")
-add_definitions (-DKEEPER_TAR_CREATE_BIN="${CMAKE_BINARY_DIR}/src/tar/keeper-tar-create")
-
-pkg_check_modules(HELPERS_TEST_DEPS REQUIRED
-    ubuntu-app-launch-2
-    properties-cpp
-    dbustest-1>=14.04.0
-    glib-2.0>=2.26
-    gio-2.0>=2.26
-    gio-unix-2.0
-)
-
-include_directories(
-    "${CMAKE_SOURCE_DIR}/src"
-)
-
-include_directories(
-    SYSTEM
-    ${HELPERS_TEST_DEPS_INCLUDE_DIRS}
-)
-
-add_executable(
-    unit-tests
-
-    tar/tar-creator-test.cpp
-
-    storage/test-factory.cpp
-
-    helper/fake-helper.h
-    helper/speed-test.cpp
-
-    helper/helpers-test.cc
-    helper/mir-mock.cpp
-
-    metadata-providers/user-dirs-test.cpp
-)
-
-qt5_use_modules(
-    unit-tests
-    Core
-    DBus
-    Test
-)
-
-target_link_libraries(
-    unit-tests
-    backup-helper
-    test-utils
-    keeperservice
-    keepertar
-    ${SERVICE_DEPS_LDFLAGS}
-    ${HELPERS_TEST_DEPS_LDFLAGS}
-    ${TEST_DEPENDENCIES_LDFLAGS}
-    ${GTEST_LIBRARIES}
-    ${GMOCK_LIBRARIES}
-)
-
-add_test(
-    unit-tests
-    unit-tests
+add_definitions(
+  -DCMAKE_SOURCE_DIR="${CMAKE_CURRENT_SOURCE_DIR}"
+  -DTEST_SIMPLE_HELPER="${CMAKE_BINARY_DIR}/src/simple-helper/simple-helper"
+  -DKEEPER_SERVICE_BIN="${CMAKE_BINARY_DIR}/src/service/keeper-service"
+  -DKEEPER_CLIENT_BIN="${CMAKE_BINARY_DIR}/src/cli/keeper"
+  -DTEST_SIMPLE_HELPER_SH="${CMAKE_SOURCE_DIR}/src/simple-helper/helper-test.sh"
+  -DKEEPER_TAR_CREATE_BIN="${CMAKE_BINARY_DIR}/src/tar/keeper-tar-create"
+)
+
+include_directories(
+  "${CMAKE_SOURCE_DIR}/src"
+)
+
+include_directories(
+  SYSTEM
+  ${HELPERS_TEST_DEPS_INCLUDE_DIRS}
+)
+
+set(
+  UNIT_TEST_LIBRARIES
+  backup-helper
+  test-utils
+  keeperservice
+  keepertar
+  ${SERVICE_DEPS_LDFLAGS}
+  ${HELPERS_TEST_DEPS_LDFLAGS}
+  ${TEST_DEPENDENCIES_LDFLAGS}
+  ${GTEST_LIBRARIES}
+  ${GMOCK_LIBRARIES}
 )
 
 add_subdirectory(helper)
+add_subdirectory(metadata-providers)
+add_subdirectory(tar)
+
+set(
+  UNIT_TEST_TARGETS
+  ${UNIT_TEST_TARGETS}
+  PARENT_SCOPE
+)

=== modified file 'tests/unit/helper/CMakeLists.txt'
--- tests/unit/helper/CMakeLists.txt	2016-07-22 12:30:46 +0000
+++ tests/unit/helper/CMakeLists.txt	2016-07-22 13:14:22 +0000
@@ -1,45 +1,103 @@
-add_definitions ( -DCMAKE_SOURCE_DIR="${CMAKE_CURRENT_SOURCE_DIR}" )
-add_definitions ( -DCMAKE_BINARY_DIR="${CMAKE_CURRENT_BINARY_DIR}" )
-
-include(FindPkgConfig)
-pkg_check_modules(UAL REQUIRED ubuntu-app-launch-2)
-pkg_check_modules(PROP-CPP REQUIRED properties-cpp)
-
-pkg_check_modules(DBUSTEST REQUIRED dbustest-1>=14.04.0)
-include_directories(${DBUSTEST_INCLUDE_DIRS})
-
-pkg_check_modules(GIO2 REQUIRED gio-2.0 gio-unix-2.0)
-include_directories(${GIO2_INCLUDE_DIRS})
-
-# Google Test
-
-include_directories(${GTEST_INCLUDE_DIR})
-
-#add_library (gtest2 STATIC
-#             ${GTEST_SOURCE_DIR}/gtest-all.cc
-#             ${GTEST_SOURCE_DIR}/gtest_main.cc)
-
-# libUAL Test
-include_directories(${UAL_INCLUDE_DIRS})
-include_directories(${PROP-CPP_INCLUDE_DIRS})
-
-add_executable (helpers-test
-    helpers-test.cc
-    mir-mock.cpp
-)
-
-set_target_properties(helpers-test PROPERTIES COMPILE_FLAGS -fPIC)
-set_target_properties(helpers-test PROPERTIES AUTOMOC TRUE)
-qt5_use_modules(helpers-test Core DBus Test)
-
-target_link_libraries(helpers-test
-                        gtest
-                        backup-helper
-                        test-utils
-                        ${GTEST_LIBS}
-                        ${LIBUPSTART_LIBRARIES}
-                        ${DBUSTEST_LIBRARIES}
-                        ${UAL_LIBRARIES}
-                        ${GIO2_LIBRARIES})
-
-add_test (NAME helpers-test COMMAND helpers-test)
+add_definitions(
+  -DCMAKE_SOURCE_DIR="${CMAKE_CURRENT_SOURCE_DIR}"
+  -DCMAKE_BINARY_DIR="${CMAKE_CURRENT_BINARY_DIR}"
+)
+
+#
+# helpers-test
+#
+
+pkg_check_modules(HELPERS_TEST_DEPS REQUIRED
+  ubuntu-app-launch-2
+  properties-cpp
+  dbustest-1>=14.04.0
+  glib-2.0>=2.26
+  gio-2.0>=2.26
+  gio-unix-2.0
+)
+
+include_directories(
+  SYSTEM
+  ${UAL_INCLUDE_DIRS}
+  ${PROP-CPP_INCLUDE_DIRS}
+  ${HELPERS_TEST_DEPS_INCLUDE_DIRS}
+)
+
+set(
+  HELPERS_TEST
+  helpers-test
+)
+
+add_executable(
+  ${HELPERS_TEST}
+  helpers-test.cc
+  mir-mock.cpp
+)
+
+set_target_properties(
+  ${HELPERS_TEST}
+  PROPERTIES
+  COMPILE_FLAGS -fPIC
+  AUTOMOC TRUE
+)
+
+target_link_libraries(
+  ${HELPERS_TEST}
+  ${HELPERS_TEST_DEPS_LDFLAGS}
+  ${UNIT_TEST_LIBRARIES}
+  Qt5::Core
+  Qt5::DBus
+  Qt5::Test
+)
+
+add_test(
+  NAME ${HELPERS_TEST}
+  COMMAND ${HELPERS_TEST}
+)
+
+#
+# speed-test
+#
+
+set(
+  SPEED_TEST
+  helper-speed-test
+)
+
+add_executable(
+  ${SPEED_TEST}
+  speed-test.cpp
+  fake-helper.h
+)
+
+set_target_properties(
+  ${SPEED_TEST}
+  PROPERTIES
+  COMPILE_FLAGS -fPIC
+  AUTOMOC TRUE
+)
+
+target_link_libraries(
+  ${SPEED_TEST}
+  ${UNIT_TEST_LIBRARIES}
+  Qt5::Core
+  Qt5::DBus
+  Qt5::Test
+)
+
+add_test(
+  NAME ${SPEED_TEST}
+  COMMAND ${SPEED_TEST}
+)
+
+#
+#
+#
+
+set(
+  UNIT_TEST_TARGETS
+  ${UNIT_TEST_TARGETS}
+  ${HELPERS_TEST}
+  ${SPEED_TEST}
+  PARENT_SCOPE
+)

=== added file 'tests/unit/metadata-providers/CMakeLists.txt'
--- tests/unit/metadata-providers/CMakeLists.txt	1970-01-01 00:00:00 +0000
+++ tests/unit/metadata-providers/CMakeLists.txt	2016-07-22 13:14:22 +0000
@@ -0,0 +1,29 @@
+set(
+  TEST_NAME
+  user-dirs-test
+)
+
+add_executable(
+  ${TEST_NAME}
+  user-dirs-test.cpp
+)
+
+target_link_libraries(
+  ${TEST_NAME}
+  ${UNIT_TEST_LIBRARIES}
+  Qt5::Core
+  Qt5::DBus
+  Qt5::Test
+)
+
+add_test(
+  ${TEST_NAME}
+  ${TEST_NAME}
+)
+
+set(
+  UNIT_TEST_TARGETS
+  ${UNIT_TEST_TARGETS}
+  ${TEST_NAME}
+  PARENT_SCOPE
+)

=== removed directory 'tests/unit/storage'
=== removed file 'tests/unit/storage/test-factory.cpp'
--- tests/unit/storage/test-factory.cpp	2016-05-28 01:52:46 +0000
+++ tests/unit/storage/test-factory.cpp	1970-01-01 00:00:00 +0000
@@ -1,29 +0,0 @@
-/*
- * Copyright (C) 2015 Canonical, Ltd.
- *
- * This program is free software: you can redistribute it and/or modify it
- * under the terms of the GNU General Public License version 3, as published
- * by the Free Software Foundation.
- *
- * This program is distributed in the hope that it will be useful, but
- * WITHOUT ANY WARRANTY; without even the implied warranties of
- * MERCHANTABILITY, SATISFACTORY QUALITY, or FITNESS FOR A PARTICULAR
- * PURPOSE.  See the GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License along
- * with this program.  If not, see <http://www.gnu.org/licenses/>.
- *
- * Authors:
- *   Charles Kerr <[e-mail]>
- */
-
-#include <gmock/gmock.h>
-#include <gtest/gtest.h>
-
-class TestFactory : public ::testing::Test
-{
-};
-
-TEST_F(TestFactory, HelloWorld)
-{
-}

=== added file 'tests/unit/tar/CMakeLists.txt'
--- tests/unit/tar/CMakeLists.txt	1970-01-01 00:00:00 +0000
+++ tests/unit/tar/CMakeLists.txt	2016-07-22 13:14:22 +0000
@@ -0,0 +1,29 @@
+set(
+  TEST_NAME
+  tar-creator-test
+)
+
+add_executable(
+  ${TEST_NAME}
+  tar-creator-test.cpp
+)
+
+target_link_libraries(
+  ${TEST_NAME}
+  ${UNIT_TEST_LIBRARIES}
+  Qt5::Core
+  Qt5::DBus
+  Qt5::Test
+)
+
+add_test(
+  ${TEST_NAME}
+  ${TEST_NAME}
+)
+
+set(
+  UNIT_TEST_TARGETS
+  ${UNIT_TEST_TARGETS}
+  ${TEST_NAME}
+  PARENT_SCOPE
+)

=== modified file 'tests/utils/CMakeLists.txt'
--- tests/utils/CMakeLists.txt	2016-07-13 19:09:59 +0000
+++ tests/utils/CMakeLists.txt	2016-07-22 13:14:22 +0000
@@ -1,23 +1,23 @@
-
 include_directories(
-    ${CMAKE_SOURCE_DIR}/src
-    ${CMAKE_CURRENT_BINARY_DIR}
+  ${CMAKE_SOURCE_DIR}/src
+  ${CMAKE_CURRENT_BINARY_DIR}
 )
 
 set(
-    LIB_NAME
-    test-utils
+  LIB_NAME
+  test-utils
 )
 
 add_library(
-    ${LIB_NAME}
-    STATIC
-    main.cpp
-    dummy-file.cpp
-    xdg-user-dirs-sandbox.cpp
+  ${LIB_NAME}
+  STATIC
+  main.cpp
+  dummy-file.cpp
+  xdg-user-dirs-sandbox.cpp
 )
 
-qt5_use_modules(
-    ${LIB_NAME}
-    Core
+target_link_libraries(
+  ${LIB_NAME}
+  backup-helper
+  Qt5::Core
 )