Skip to content

Add tool script for testing individual drivers - #3227

Open
hcarter-775 wants to merge 1 commit into
mainfrom
add/run-driver-tests-file
Open

hcarter-775 wants to merge 1 commit into
mainfrom
add/run-driver-tests-file

Conversation

@hcarter-775

@hcarter-775 hcarter-775 commented Sep 10, 2026

Copy link
Copy Markdown
Contributor

Description of Change

Add a test_driver.sh script, along with a one-time setup script, to the tools directory, so that developers and agents have access to a slightly more ergonomic driver unit testing framework.

Some context: I have been using basically this script for a couple years now, and am trying to bring this to the more generic experience (like for AI tools). I had AI mostly generate the AGENTS documentation, and the QOL script for setting up an alias. The actual test_driver.sh script was pre-existing for me

@hcarter-775 hcarter-775 added the CGAI Contains code that was generated by AI label Sep 10, 2026

@cjswedes cjswedes left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I see why you have been using this. However, rather than having two different test scripts, I think I would prefer to see new functionality integrated into the existing script (luacheck, installable to PATH, output file option, etc.).

If there is a different format for output that you think works better for agents (to save tokens or whatever the reason), we could add a new option that changes output to the existing script.

I dont love jamming everything in one script, but it would be useful for development to have luacheck/coverage and test running possible with one command. But I dont think we want to maintain this script along the existing one.

Comment thread tools/test_driver.sh

# ---------------------------------------------------------------------------
# -r: luacheck (mirrors the repo's .github/workflows/.luacheckrc — keep the
# two in sync if that file changes)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We should not have to update 2 places to use this

Comment thread tools/test_driver.sh

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

IMO this is a somewhat complex bash script and python is more maintainable. That aside, it is another barrier for developers on windows, which is the main reason why we should not add this script.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CGAI Contains code that was generated by AI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants