Skip to content

Document Test Suites#94

Draft
Lightning11wins wants to merge 5 commits into
masterfrom
doc-testing
Draft

Document Test Suites#94
Lightning11wins wants to merge 5 commits into
masterfrom
doc-testing

Conversation

@Lightning11wins
Copy link
Copy Markdown
Contributor

Add Testing.md to centrallix-sysdoc to document the various test suites and testing methods in Centrallix, including where to find them, how to run them, and how to add additional tests.

Information may be missing or inaccurate, especially regarding the Selenium test suite that I still have never run on my own.

@Lightning11wins Lightning11wins self-assigned this Apr 7, 2026
@Lightning11wins Lightning11wins added testing Includes testing, either new tests or updates to existing tests. ai-review Request AI review for PRs. labels Apr 7, 2026
@Lightning11wins Lightning11wins marked this pull request as draft April 7, 2026 17:23
@Lightning11wins Lightning11wins added the documentation Changes, improvements, or fixes to documentation files. label Apr 7, 2026
@greptile-apps
Copy link
Copy Markdown

greptile-apps Bot commented Apr 7, 2026

Greptile Summary

This PR adds Testing.md to centrallix-sysdoc, documenting the test-obj, centrallix-lib, and Selenium UI test suites — covering how to find, run, and extend each suite. It also corrects an invalid C function signature in centrallix/tests/README (**char namechar** name). All remaining findings are P2.

Confidence Score: 5/5

Safe to merge — documentation-only PR with accurate content and a correct README fix.

Both findings are P2 (a one-word typo and an informal phrasing note). All cross-referenced files (UITestCoverage.md, centrallix-lib/tests/README) resolve correctly. The C signature fix addresses the previously raised P0 issue.

No files require special attention.

Important Files Changed

Filename Overview
centrallix-sysdoc/Testing.md New 185-line testing guide covering test-obj, centrallix-lib, and Selenium UI suites; minor typo on line 59 and informal voice on line 118
centrallix/tests/README One-line fix correcting C function signature from **char name to char** name

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[Developer runs tests] --> B{Which suite?}
    B --> C[test-obj suite]
    B --> D[centrallix-lib suite]
    B --> E[Selenium UI suite]
    C --> C1[make test in centrallix/]
    C1 --> C2[Standard .to / .cmp tests]
    C1 --> C3[Native C .c tests]
    D --> D1[make test in centrallix-lib/]
    D1 --> D2[C binaries via t_driver.c]
    D1 --> D3[Optional: make valtest / gcov]
    E --> E1[Start Centrallix server]
    E1 --> E2[python3 tests/X_test.py]
    E2 --> E3[Chrome via ChromeDriver]
Loading

Greploops — Automatically fix all review issues by running /greploops in Claude Code. It iterates: fix, push, re-review, repeat until 5/5 confidence.
Use the Greptile plugin for Claude Code to query reviews, search comments, and manage custom context directly from your terminal.

Reviews (2): Last reviewed commit: "Fix comments from greptile." | Re-trigger Greptile

Comment thread centrallix-sysdoc/Testing.md Outdated
@Lightning11wins
Copy link
Copy Markdown
Contributor Author

@greptileai I've resolved your comments, can you rereview? Remember, I'd like to make MINIMAL changes to that README file in centrallix/tests even though I now have changes there because it's outside the tight scope of this small documentation project.

@Lightning11wins
Copy link
Copy Markdown
Contributor Author

Oops, forgot about the blocking issues in our questions thread. My bad.

@Lightning11wins Lightning11wins added size: medium Might be hard to review, usually less than ~5000 lines. blocked No more work can be done until other work is completed. labels Apr 27, 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. blocked No more work can be done until other work is completed. documentation Changes, improvements, or fixes to documentation files. 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