makes the usb test more conditional so the build will pass on CI

TheNotary · Sep 10, 2024 · 7197f76 · 7197f76
1 parent 6c511e4
commit 7197f76
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 2 deletions.
diff --git a/docs/env_setup_and_building.md b/docs/env_setup_and_building.md
@@ -7,6 +7,8 @@ As that the project is my only repo demonstrating sourcing dependencies in a C++
 
 #### Building on Terminal ("Developer Command Prompt")
 
+Note:  There's an integration test that can be enabled by setting the env variable "KEYBOARD_ATTACHED" to 1.  
+
 At the root of the repo, run:
 ```
 # Skip these, they were for gitbash but there are pathing issues that ruined it

diff --git a/test/keyboard_test.cpp b/test/keyboard_test.cpp
@@ -8,7 +8,6 @@
 #include "../src/demo/blink_loader.h"
 
 
-
 namespace KeyboardTest
 {
     using namespace rk84;
@@ -77,7 +76,11 @@ namespace KeyboardTest
             was_reasonable = true;
 
         EXPECT_TRUE(was_reasonable);
-        EXPECT_EQ(list.size(), 1);
+
+        // Only do this if an KEYBOARD_ATTACHED is set to 1
+        const char* keyboardAttachedEnv = std::getenv("KEYBOARD_ATTACHED");
+        if (keyboardAttachedEnv != nullptr && std::string(keyboardAttachedEnv) == "1")
+            EXPECT_EQ(list.size(), 1);
     }
 
 }