diff --git a/.github/workflows/qemu-test.yml b/.github/workflows/qemu-test.yml index a55de0f..3436409 100644 --- a/.github/workflows/qemu-test.yml +++ b/.github/workflows/qemu-test.yml @@ -25,7 +25,6 @@ jobs: ovmf \ mtools \ xorriso \ - scrot \ imagemagick - name: Build bootloader diff --git a/.github/workflows/unit-tests.yml b/.github/workflows/unit-tests.yml index ec69dcd..d751701 100644 --- a/.github/workflows/unit-tests.yml +++ b/.github/workflows/unit-tests.yml @@ -33,10 +33,4 @@ jobs: - name: Test summary if: always() run: | - echo "Unit tests completed" - if [ $? -eq 0 ]; then - echo "✓ All tests passed" - else - echo "✗ Some tests failed" - exit 1 - fi + echo "Unit test execution completed" diff --git a/tests/include/test_framework.h b/tests/include/test_framework.h index af358b3..1d9591a 100644 --- a/tests/include/test_framework.h +++ b/tests/include/test_framework.h @@ -50,7 +50,7 @@ static TestStats test_stats = {0, 0, 0}; do { \ if ((a) != (b)) { \ printf(COLOR_RED "[FAILED] " COLOR_RESET "%s:%d: Expected %lld, got %lld\n", \ - __FILE__, __LINE__, (long long)(a), (long long)(b)); \ + __FILE__, __LINE__, (long long)(b), (long long)(a)); \ test_stats.failed++; \ return; \ } \ @@ -59,8 +59,8 @@ static TestStats test_stats = {0, 0, 0}; #define ASSERT_NE(a, b) \ do { \ if ((a) == (b)) { \ - printf(COLOR_RED "[FAILED] " COLOR_RESET "%s:%d: Expected not equal: %lld == %lld\n", \ - __FILE__, __LINE__, (long long)(a), (long long)(b)); \ + printf(COLOR_RED "[FAILED] " COLOR_RESET "%s:%d: Values should not be equal: %lld\n", \ + __FILE__, __LINE__, (long long)(a)); \ test_stats.failed++; \ return; \ } \ diff --git a/tests/unit/test_console.c b/tests/unit/test_console.c index faa3db8..5bb51c5 100644 --- a/tests/unit/test_console.c +++ b/tests/unit/test_console.c @@ -19,8 +19,8 @@ typedef struct { uint32_t bg_color; } Console; -// Mock framebuffer for testing -static uint32_t test_framebuffer[1920 * 1080]; +// Mock framebuffer for testing (using smaller size to avoid stack issues) +static uint32_t test_framebuffer[800 * 600]; // Simplified console functions for unit testing // In real implementation, these would be in a testable module