refactor(assertion-tools): ♻️ Fix some linting issues

src/common/parsing/AssertionTools.cpp

@@ -8,29 +8,27 @@
 #include <memory>
 #include <string>
 
-bool doesCommuteEnt(const EntanglementAssertion* assertion,
+bool doesCommuteEnt(const EntanglementAssertion* assertion, // NOLINT
                     const std::string& instruction) {
   const auto targets = parseParameters(instruction);
   return targets.size() < 2; // If the instruction does not target at least two
                              // qubits, it cannot influence entanglement.
   // In theory, even more could be done here, but for now we leave it like this.
 }
 
-bool doesCommuteSup(const SuperpositionAssertion* assertion,
+bool doesCommuteSup(const SuperpositionAssertion* assertion, // NOLINT
                     const std::string& instruction) {
   const auto targets = parseParameters(instruction);
   if (targets.size() >= 2) {
     return false; // For controlled gates, it's hard to say how they would
                   // influence the superposition of the qubits.
   }
   const auto name = splitString(trim(instruction), ' ')[0];
-  if (name == "x" || name == "y" || name == "z" || name == "s" || name == "t" ||
-      name == "sdg" || name == "tdg") {
-    return true; // Most common Single-qubit gates commute with superposition
-                 // assertions.
-  }
-  return false; // For other gates, it's hard to say how they would influence
-                // the superposition of the qubits.
+  return (name == "x" || name == "y" || name == "z" || name == "s" ||
+          name == "t" || name == "sdg" || name == "tdg");
+  // Most common Single-qubit gates commute with superposition assertions.
+  // For other gates, it's hard to say how they would influence
+  // the superposition of the qubits.
 }
 
 bool doesCommute(const std::unique_ptr<Assertion>& assertion,

test/CMakeLists.txt

@@ -1,7 +1,6 @@
 package_add_test(
   mqt_debugger_test
   MQT::Debugger
-  common_fixtures.cpp
   utils_test.cpp
   test_utility.cpp
   test_simulation.cpp

test/test_assertion_creation.cpp

@@ -3,17 +3,16 @@
  * @brief Tests the correctness of the assertion creation diagnosis methods.
  */
 
-#include "backend/dd/DDSimDebug.hpp"
 #include "backend/debug.h"
 #include "backend/diagnostics.h"
 #include "common.h"
-#include "common_fixtures.cpp"
-#include "utils_test.hpp"
+#include "common_fixtures.hpp"
 
+#include <algorithm>
 #include <array>
 #include <cstddef>
 #include <gtest/gtest.h>
-#include <sstream>
+#include <set>
 #include <string>
 #include <utility>
 #include <vector>

test/test_assertion_movement.cpp

@@ -3,18 +3,12 @@
  * @brief Tests the correctness of the assertion movement diagnosis methods.
  */
 
-#include "backend/dd/DDSimDebug.hpp"
-#include "backend/debug.h"
 #include "backend/diagnostics.h"
-#include "common.h"
-#include "common_fixtures.cpp"
-#include "utils_test.hpp"
+#include "common_fixtures.hpp"
 
-#include <array>
 #include <cstddef>
 #include <gtest/gtest.h>
-#include <sstream>
-#include <string>
+#include <set>
 #include <utility>
 #include <vector>
 

test/test_custom_code.cpp

@@ -7,18 +7,15 @@
  * are not covered by the other tests.
  */
 
-#include "backend/dd/DDSimDebug.hpp"
 #include "backend/debug.h"
 #include "backend/diagnostics.h"
 #include "common.h"
-#include "common_fixtures.cpp"
+#include "common_fixtures.hpp"
 #include "utils_test.hpp"
 
 #include <array>
 #include <cstddef>
 #include <gtest/gtest.h>
-#include <sstream>
-#include <string>
 
 /**
  * @brief Fixture for testing the correctness of the debugger on custom code.

test/test_data_retrieval.cpp

@@ -6,10 +6,9 @@
  * vector.
  */
 
-#include "backend/dd/DDSimDebug.hpp"
 #include "backend/debug.h"
 #include "common.h"
-#include "common_fixtures.cpp"
+#include "common_fixtures.hpp"
 #include "utils_test.hpp"
 
 #include <array>

test/test_diagnostics.cpp

@@ -3,12 +3,10 @@
  * @brief Test the functionality of the diagnostics module.
  */
 
-#include "backend/dd/DDSimDebug.hpp"
 #include "backend/debug.h"
 #include "backend/diagnostics.h"
 #include "common.h"
-#include "common_fixtures.cpp"
-#include "utils_test.hpp"
+#include "common_fixtures.hpp"
 
 #include <array>
 #include <cstddef>

test/test_utility.cpp

@@ -2,11 +2,9 @@
  * @file test_utility.cpp
  * @brief Test the functionality of utility functions provided by the debugger.
  */
-#include "backend/dd/DDSimDebug.hpp"
 #include "backend/debug.h"
 #include "common.h"
-#include "common_fixtures.cpp"
-#include "utils_test.hpp"
+#include "common_fixtures.hpp"
 
 #include <cstddef>
 #include <gtest/gtest.h>

test/common_fixtures.cpp → test/utils/common_fixtures.hpp

@@ -7,10 +7,13 @@
 #include "backend/dd/DDSimDiagnostics.hpp"
 #include "backend/debug.h"
 #include "backend/diagnostics.h"
+#include "common.h"
 #include "utils_test.hpp"
 
 #include <cstddef>
 #include <gtest/gtest.h>
+#include <sstream>
+#include <string>
 
 /**
  * @brief Fixture that loads custom code.