Skip to content

Upgrade centrallix-lib Tests - #170

Open
Lightning11wins wants to merge 106 commits into
masterfrom
add-lib-tests
Open

Upgrade centrallix-lib Tests#170
Lightning11wins wants to merge 106 commits into
masterfrom
add-lib-tests

Conversation

@Lightning11wins

@Lightning11wins Lightning11wins commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

This PR will:

  • Improves the centrallix-lib test driver.
    • Lockups now take 5 seconds without Valgrind and 10 with valgrind.
    • Tests that "fail" due to running too fast now print PASS ??? and give a clear warning.
  • Add test_utils.h to make tests easier to read and write.
    • Includes EXPECT functions for clear assertions.
    • Includes a function to automatically loop tests until a time limit is reached.
  • Rewrite tests to only loop as many times as needed.
    • This change alone reduced time required to run the test suite by 40%.
    • When using Valgrind, this reduced the time from 20-25m down to 95 seconds.
  • Tests run in parallel for large performance improvements.
    • Outputs are accumulated in the background so they can be printed in order.
    • To disable, export TESTJOBS=1.
  • Fix make valtest to allow using valgrind on tests.
  • Fix some tests that previously failed (e.g. memstr-00).
  • Add several tests for centrallix-lib modules and functions that did not previously have tests.

GitHub Relationships

Greptile Summary

@Lightning11wins Lightning11wins mentioned this pull request Sep 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ai-review Request AI review for PRs. size: medium Might be hard to review, usually less than ~5000 lines. testing Includes testing, either new tests or updates to existing tests.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant