Skip to content

docs: correct testing harness paths - #351

Merged
codelion merged 1 commit into
algorithmicsuperintelligence:mainfrom
original4422:docs/testing-harness-location
Nov 2, 2026
Merged

codelion merged 1 commit into
algorithmicsuperintelligence:mainfrom
original4422:docs/testing-harness-location

Conversation

@original4422

Copy link
Copy Markdown
Contributor

Summary

-python test.py [options]
+python tests/test.py [options]

Correct the four root-directory commands in CLAUDE.md and the contradictory harness-location bullet in tests/README.md. The existing harness and its arguments are unchanged.

Evidence

  • Before: from the project root, python test.py --help exits 2 because the file does not exist.
  • After: the original python tests/test.py --help succeeds, as do the documented --approaches, --model/--base-url, and --single-test forms with --help appended. Verified using Python 3.13.14 and the original proxy-only requirements; all four invocations made zero outbound network attempts.
  • git diff --check passes. This validates script location/imports/arguments only; model inference, server/provider integration, and the full test suite were not run.

Merge Danger

Door: two-way

Blast Radius: documentation

Prepared with AI assistance.

@CLAassistant

CLAassistant commented Oct 31, 2026 •

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@CLAassistant

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@original4422

Copy link
Copy Markdown
Contributor Author

@codelion, when convenient, could you review this small documentation correction? It updates the four testing commands and the harness-location note to tests/test.py. The original help/argument checks pass with no outbound requests; no model/server tests were run. This was prepared with AI assistance. I understand the CLA remains pending before acceptance. Thank you!

@codelion

Copy link
Copy Markdown
Member

You will need to sign the CLA and then I can merge it.

@original4422

Copy link
Copy Markdown
Contributor Author

Hi @codelion , I have signed the CLA

@codelion
codelion merged commit b63ce5c into algorithmicsuperintelligence:main Nov 2, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants