Fix minor issues and create a .clangd config for different builddirs

.clangd

@@ -0,0 +1,11 @@
+---
+If:
+  PathMatch: tests/.*
+CompileFlags:
+  CompilationDatabase: builddir-debug/
+
+---
+If:
+  PathMatch: src/.*
+CompileFlags:
+  CompilationDatabase: builddir/

.vscode/c_cpp_properties.json

@@ -5,9 +5,10 @@
       "compilerPath": "/usr/bin/clang",
       "cStandard": "c11",
       "cppStandard": "c++20",
-      "compileCommands": "${workspaceFolder}/builddir/compile_commands.json",
       "browse": {
-        "path": ["${workspaceFolder}"]
+        "path": [
+          "${workspaceFolder}"
+        ]
       }
     }
   ],

.vscode/settings.json

@@ -1,7 +1,6 @@
 {
   "clangd.arguments": [
     "-background-index",
-    "-compile-commands-dir=builddir/"
   ],
   "editor.formatOnType": false,
   "editor.formatOnSave": true,

tests/label_test/label_test.cpp

@@ -25,8 +25,8 @@
 #include <memory>
 #include <vector>
 
-#include "../../tests/mocks/MockCairoWrapper.hpp"
 #include "graphics/interface/ILabel.hpp"
+#include "mocks/MockCairoWrapper.hpp"
 
 using ::testing::_;
 using ::testing::DoAll;

tests/monochrome_test/monochrome_test.cpp

@@ -57,9 +57,6 @@ TEST(basic_test, Monochrome_convertWithCustomThreshhold_yieldsMonochromeRespecti
 }
 
 TEST(basic_test, Monochrome_convertNonAlignedPixels_spillsOverIntoNewByte) {
-    // TODO: We need to find to access the vector without the possiblity of out-of-bounds access
-    // Ideas: constexpr? compile time check?
-    GTEST_SKIP() << "Skipping this test, as ASAN will halt as this is an out-of-bounds access";
     const std::vector<uint8_t> pixels(
         {0xFF, 0x00, 0xFF, 0x00, 0xFF, 0x00, 0xFF, 0x00, 0xFF, 0x00, 0xFF, 0x00, 0xFF, 0x00, 0xFF, 0x00, 0xFF});
 

tests/p700_printer_test/p700_printer_test.cpp

@@ -22,9 +22,9 @@
 
 #include <memory>
 
-#include "../../tests/mocks/MockUsbDevice.hpp"
 #include "graphics/Bitmap.hpp"
 #include "graphics/Monochrome.hpp"
+#include "mocks/MockUsbDevice.hpp"
 #include "printers/P700Printer.hpp"
 
 using ::testing::_;
@@ -39,8 +39,8 @@ namespace ptprnt::printer {
 class P700PrinterTest : public ::testing::Test {
    protected:
     void SetUp() override {
-        printer     = std::make_unique<P700Printer>();
-        mockUsbDev  = std::make_shared<NiceMock<libusbwrap::MockUsbDevice>>();
+        printer    = std::make_unique<P700Printer>();
+        mockUsbDev = std::make_shared<NiceMock<libusbwrap::MockUsbDevice>>();
 
         // Default mock behaviors
         ON_CALL(*mockUsbDev, open()).WillByDefault(Return(true));

tests/printer_service_test/printer_service_test.cpp

@@ -23,8 +23,6 @@
 #include <memory>
 #include <vector>
 
-#include "../../tests/mocks/MockPrinterDriver.hpp"
-#include "../../tests/mocks/MockUsbDevice.hpp"
 #include "core/PrinterDriverFactory.hpp"
 #include "core/PrinterService.hpp"