Merge pull request #231 from seqan/actions/iwyu

[MISC] IWYU
seqan · Sep 16, 2024 · 6efa1fc · 6efa1fc
2 parents 169cafe + 5dbadd0
commit 6efa1fc
Show file tree

Hide file tree

Showing 7 changed files with 44 additions and 28 deletions.
diff --git a/include/hibf/layout/hierarchical_binning.hpp b/include/hibf/layout/hierarchical_binning.hpp
@@ -10,10 +10,9 @@
 #include <utility> // for pair
 #include <vector>  // for vector
 
-#include <hibf/build/bin_size_in_bits.hpp> // for bin_size_in_bits
-#include <hibf/config.hpp>                 // for config
-#include <hibf/layout/data_store.hpp>      // for data_store
-#include <hibf/platform.hpp>               // for HIBF_WORKAROUND_GCC_BOGUS_MEMCPY
+#include <hibf/config.hpp>            // for config
+#include <hibf/layout/data_store.hpp> // for data_store
+#include <hibf/platform.hpp>          // for HIBF_WORKAROUND_GCC_BOGUS_MEMCPY
 
 namespace seqan::hibf::layout
 {

diff --git a/src/layout/data_store.cpp b/src/layout/data_store.cpp
@@ -2,7 +2,11 @@
 // SPDX-FileCopyrightText: 2016-2024, Knut Reinert & MPI für molekulare Genetik
 // SPDX-License-Identifier: BSD-3-Clause
 
-#include <hibf/layout/data_store.hpp> // for data_store
+#include <stdexcept> // for invalid_argument
+#include <vector>    // for vector
+
+#include <hibf/layout/data_store.hpp>  // for data_store
+#include <hibf/sketch/hyperloglog.hpp> // for hyperloglog
 
 namespace seqan::hibf::layout
 {

diff --git a/src/layout/hierarchical_binning.cpp b/src/layout/hierarchical_binning.cpp
@@ -2,12 +2,13 @@
 // SPDX-FileCopyrightText: 2016-2024, Knut Reinert & MPI für molekulare Genetik
 // SPDX-License-Identifier: BSD-3-Clause
 
-#include <algorithm> // for copy, max, fill_n, min
+#include <algorithm> // for max, min
 #include <cassert>   // for assert
 #include <cmath>     // for log, ceil
 #include <cstddef>   // for size_t
 #include <limits>    // for numeric_limits
-#include <string>    // for allocator, operator+, string
+#include <stdexcept> // for invalid_argument
+#include <string>    // for operator+, string
 #include <utility>   // for pair
 #include <vector>    // for vector
 

diff --git a/test/include/hibf/test/expect_throw_msg.hpp b/test/include/hibf/test/expect_throw_msg.hpp
@@ -9,7 +9,12 @@
 
 #pragma once
 
-#include <gtest/gtest.h>
+#include <gtest/gtest.h> // for GTEST_NONFATAL_FAILURE_, AssertionResult, GetTypeName, EqHelper
+
+#include <exception>   // for exception
+#include <stdexcept>   // for invalid_argument
+#include <string>      // for string
+#include <string_view> // for string_view
 
 #include <hibf/platform.hpp>
 

diff --git a/test/unit/hibf/config_test.cpp b/test/unit/hibf/config_test.cpp
@@ -2,19 +2,18 @@
 // SPDX-FileCopyrightText: 2016-2024, Knut Reinert & MPI für molekulare Genetik
 // SPDX-License-Identifier: BSD-3-Clause
 
-#include <gtest/gtest.h> // for Message, TestPartResult, AssertionResult, Test, EXPECT_EQ, CaptureStderr
-
-#include <cstddef>     // for size_t
-#include <functional>  // for function
-#include <sstream>     // for basic_stringstream, stringstream
-#include <stdexcept>   // for invalid_argument
-#include <string>      // for allocator, char_traits, string
-#include <string_view> // for string_view
-#include <utility>     // for move
-
-#include <hibf/config.hpp>      // for config, insert_iterator
-#include <hibf/test/cereal.hpp> // for test_serialisation
-#include <hibf/test/expect_throw_msg.hpp>
+#include <gtest/gtest.h> // for Message, TestPartResult, AssertionResult, Test, EXPECT_EQ, Capture...
+
+#include <cstddef>    // for size_t
+#include <functional> // for function
+#include <sstream>    // for basic_stringstream, stringstream
+#include <stdexcept>  // for invalid_argument
+#include <string>     // for allocator, char_traits, string
+#include <utility>    // for move
+
+#include <hibf/config.hpp>                // for config, insert_iterator
+#include <hibf/test/cereal.hpp>           // for test_serialisation
+#include <hibf/test/expect_throw_msg.hpp> // for EXPECT_THROW_MSG
 
 TEST(config_test, write_to)
 {

diff --git a/test/unit/hibf/layout/data_store_test.cpp b/test/unit/hibf/layout/data_store_test.cpp
@@ -2,10 +2,17 @@
 // SPDX-FileCopyrightText: 2016-2024, Knut Reinert & MPI für molekulare Genetik
 // SPDX-License-Identifier: BSD-3-Clause
 
-#include <gtest/gtest.h>
+#include <gtest/gtest.h> // for Message, TestPartResult, Test, TestInfo, EXPECT_NO_THROW, TEST
 
-#include <hibf/layout/data_store.hpp>
-#include <hibf/test/expect_throw_msg.hpp>
+#include <cstddef>   // for size_t
+#include <stdexcept> // for invalid_argument
+#include <string>    // for basic_string
+#include <vector>    // for vector
+
+#include <hibf/layout/data_store.hpp>     // for data_store
+#include <hibf/layout/layout.hpp>         // for layout
+#include <hibf/sketch/hyperloglog.hpp>    // for hyperloglog
+#include <hibf/test/expect_throw_msg.hpp> // for EXPECT_THROW_MSG
 
 TEST(data_store_test, validate)
 {

diff --git a/test/unit/hibf/layout/hierarchical_binning_test.cpp b/test/unit/hibf/layout/hierarchical_binning_test.cpp
@@ -2,10 +2,11 @@
 // SPDX-FileCopyrightText: 2016-2024, Knut Reinert & MPI für molekulare Genetik
 // SPDX-License-Identifier: BSD-3-Clause
 
-#include <gtest/gtest.h> // for Test, Message, TestPartResult, TestInfo, EXPECT_EQ
+#include <gtest/gtest.h> // for Test, Message, TestPartResult, TestInfo, TEST, EXP...
 
-#include <cstddef> // for size_t
-#include <vector>  // for vector, allocator
+#include <cstddef>   // for size_t
+#include <stdexcept> // for invalid_argument
+#include <vector>    // for vector
 
 #include <hibf/config.hpp>                                // for config
 #include <hibf/layout/compute_fpr_correction.hpp>         // for compute_fpr_correction
@@ -14,7 +15,7 @@
 #include <hibf/layout/hierarchical_binning.hpp>           // for hierarchical_binning
 #include <hibf/layout/layout.hpp>                         // for layout
 #include <hibf/test/expect_range_eq.hpp>                  // for expect_range_eq, EXPECT_RANGE_EQ
-#include <hibf/test/expect_throw_msg.hpp>
+#include <hibf/test/expect_throw_msg.hpp>                 // for EXPECT_THROW_MSG
 
 TEST(hierarchical_binning_test, missing_sketches)
 {