IWYU

Author: TrentHouliston

src/threading/ReactionHandle.hpp

@@ -20,8 +20,8 @@
  * OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
  */
 
-#ifndef NUCLEAR_THREADING_REACTIONHANDLE_HPP
-#define NUCLEAR_THREADING_REACTIONHANDLE_HPP
+#ifndef NUCLEAR_THREADING_REACTION_HANDLE_HPP
+#define NUCLEAR_THREADING_REACTION_HANDLE_HPP
 
 #include <memory>
 
@@ -100,4 +100,4 @@ namespace threading {
 }  // namespace threading
 }  // namespace NUClear
 
-#endif  // NUCLEAR_THREADING_REACTIONHANDLE_HPP
+#endif  // NUCLEAR_THREADING_REACTION_HANDLE_HPP

tests/tests/api/LogLevel.cpp

@@ -28,6 +28,7 @@
 #include <sstream>
 #include <string>
 #include <tuple>
+#include <vector>
 
 SCENARIO("LogLevel smart enum values can be constructed and converted appropriately") {
     GIVEN("A LogLevel and a corresponding string representation") {

tests/tests/api/PriorityLevel.cpp

@@ -25,9 +25,9 @@
 #include <catch2/catch_test_macros.hpp>
 #include <catch2/generators/catch_generators.hpp>
 #include <catch2/generators/catch_generators_range.hpp>
-#include <ostream>
 #include <sstream>
 #include <tuple>
+#include <vector>
 
 SCENARIO("PriorityLevel smart enum values can be constructed and converted appropriately") {
     GIVEN("A PriorityLevel and a corresponding string representation") {
@@ -92,7 +92,7 @@ SCENARIO("PriorityLevel smart enum values can be constructed and converted appro
 
 SCENARIO("PriorityLevel comparison operators work correctly") {
     GIVEN("Two PriorityLevel enum values") {
-        std::vector<NUClear::PriorityLevel::Value> levels = {
+        const std::vector<NUClear::PriorityLevel::Value> levels = {
             NUClear::PriorityLevel::IDLE,
             NUClear::PriorityLevel::LOW,
             NUClear::PriorityLevel::LOWEST,