From e3fa6b1adbd7fce3b080d75fd0959949b7d3bef4 Mon Sep 17 00:00:00 2001 From: Felix Morgner Date: Tue, 21 Apr 2026 14:13:55 +0200 Subject: acpi: enable test coverage --- .clang-tidy | 15 ++++++++------- .clangd | 13 ------------- CMakePresets.json | 5 ++++- libs/acpi/CMakeLists.txt | 5 +++++ 4 files changed, 17 insertions(+), 21 deletions(-) diff --git a/.clang-tidy b/.clang-tidy index 61ae9c9..26356ea 100644 --- a/.clang-tidy +++ b/.clang-tidy @@ -34,7 +34,7 @@ Checks: - cppcoreguidelines-slicing - cppcoreguidelines-use-enum-class - cppcoreguidelines-virtual-class-destructor - + - misc-definitions-in-headers - misc-include-cleaner - misc-no-recursion @@ -55,18 +55,19 @@ Checks: - readability-magic-numbers CheckOptions: + cppcoreguidelines-avoid-do-while.IgnoreMacros: true cppcoreguidelines-avoid-non-const-global-variables.AllowInternalLinkage: true - modernize-use-std-print.ReplacementPrintFunction: 'kstd::print' - modernize-use-std-print.ReplacementPrintlnFunction: 'kstd::println' - modernize-use-std-print.PrintHeader: 'kstd/print' + modernize-use-std-print.ReplacementPrintFunction: "kstd::print" + modernize-use-std-print.ReplacementPrintlnFunction: "kstd::println" + modernize-use-std-print.PrintHeader: "kstd/print" modernize-use-trailing-return-type.TransformLambdas: none - readability-magic-numbers.IgnoredIntegerValues: '1;2;3;4;5;6;7;10;255' + readability-magic-numbers.IgnoredIntegerValues: "1;2;3;4;5;6;7;10;255" readability-magic-numbers.IgnorePowersOf2IntegerValues: true readability-magic-numbers.IgnoreBitFieldsWidths: true readability-magic-numbers.IgnoreTypeAliases: true FormatStyle: file -HeaderFilterRegex: '(.*/kstd/include/.*)|(arch|kernel|kapi)/.*\.hpp' +HeaderFilterRegex: "(.*/kstd/include/.*)|(arch|kernel|kapi)/.*\\.hpp" SystemHeaders: true RemovedArgs: - - -fcondition-coverage \ No newline at end of file + - -fcondition-coverage diff --git a/.clangd b/.clangd index ce14df5..55f84ae 100644 --- a/.clangd +++ b/.clangd @@ -6,16 +6,3 @@ CompileFlags: - -fcondition-coverage Documentation: CommentFormat: Doxygen - ---- -If: - PathMatch: - - "libs/.*/tests/.*\\.cpp" - - "libs/.*/.*\\.test\\.cpp" - - "kernel/.*\\.tests.cpp" -Diagnostics: - ClangTidy: - Remove: "*" -CompileFlags: - Add: - - -Wno-c2y-extensions diff --git a/CMakePresets.json b/CMakePresets.json index 0e5dd88..d5123a2 100644 --- a/CMakePresets.json +++ b/CMakePresets.json @@ -24,7 +24,10 @@ { "name": "bht", "inherits": "base", - "description": "Build-host Testing" + "description": "Build-host Testing", + "cacheVariables": { + "CATCH_CONFIG_NO_COUNTER": true + } } ], "buildPresets": [ diff --git a/libs/acpi/CMakeLists.txt b/libs/acpi/CMakeLists.txt index b0fc48f..b4d11d9 100644 --- a/libs/acpi/CMakeLists.txt +++ b/libs/acpi/CMakeLists.txt @@ -46,6 +46,7 @@ target_sources("acpi" PRIVATE ) file(GLOB_RECURSE ACPI_HEADERS + CONFIGURE_DEPENDS RELATIVE ${CMAKE_CURRENT_SOURCE_DIR} "acpi/*.hpp" ) @@ -96,6 +97,10 @@ if(BUILD_TESTING) set_source_files_properties("test_data/tables.S" PROPERTIES OBJECT_DEPENDS "${GENERATED_TABLE_BLOBS}") + if(COMMAND "enable_coverage") + enable_coverage("acpi") + endif() + add_executable("acpi_tests") add_executable("acpi::tests" ALIAS "acpi_tests") -- cgit v1.2.3