docs: update testing-your-application.md for WheelsTest#2128
Merged
Conversation
Editorial follow-up to #2127. Updates the main user-facing testing guide for the WheelsTest rename, plus fixes stale code paths to match the actual tests/runner.cfm implementation. Scope (39 refs → 10 legitimate retentions): Stale code paths corrected to match tests/runner.cfm:99-246: - testbox.system.TestBox → wheels.wheelstest.system.TestBox - testbox.system.reports.JSONReporter → wheels.wheelstest... - testbox.system.reports.TextReporter → wheels.wheelstest... - testbox.system.reports.ANTJUnitReporter → wheels.wheelstest... Framework-identity references renamed: - TestBox 6 → WheelsTest (drops upstream version qualifier) - "TestBox Features" heading → "WheelsTest Features" - "TestBox Test Runner Configuration" → "WheelsTest Test Runner Configuration" - "TestBox testing URLs" → "WheelsTest URLs" - JSON-output section renamed to reference WheelsTest runner - `// load testbox routes` comment → `// load wheelstest routes` - Migration section: "Legacy RocketUnit to TestBox" → "to WheelsTest" Upstream URL references reframed (keeping URLs, clarifying provenance): - "official TestBox documentation" → "upstream TestBox documentation" - Each ortusbooks.com link labeled with "upstream" and where appropriate "(WheelsTest preserves the same API)" Intro paragraph rewritten to establish provenance: "WheelsTest — Wheels' in-house testing framework (a fork of Ortus TestBox)". Legitimate retentions (10): - 1 fork-explanation reference (intro) - 8 upstream URL labels pointing to testbox.ortusbooks.com - 1 code variable name `testBox = new wheels.wheelstest.system. TestBox(...)` — matches actual runner.cfm convention Preserved historical references: - Sections describing pre-Wheels-3.0 RocketUnit legacy - "Wheels 3.0" framing where specifically describing the 3.0 release Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
bpamiri
added a commit
that referenced
this pull request
Apr 16, 2026
Editorial follow-up PR to #2127 and #2128. Covers the last three files deferred from the sweep: WHEELS-CLI-ROADMAP.md (cli/): Added a status banner at the top of the document noting that the testing-strategy sections predate the WheelsTest fork decision. The roadmap originally proposed integrating the external commandbox- testbox-cli module; the actual direction was to fork TestBox in-house as WheelsTest. Rather than rewriting the strategy sections (out of scope for rename), the banner makes the historical context clear and flags for a separate editorial pass. References preserved as-is for historical traceability: - Section 4 "Advanced Testing (via TestBox CLI Module)" - Leverage Existing Modules item 1 - Phase 1 Module Integration - CommandBox Integration Points - box.json dependency listing for commandbox-testbox-cli - Recommended Module Installation section examples/tweet/.claude/commands/wheels_execute.md (10 refs): - TestBox BDD → WheelsTest BDD (descriptive references) - testbox.system.TestBox → wheels.wheelstest.system.TestBox (code) - application.testbox → application.wheelstest (CFML scope) - "Use TestBox's beforeEach/afterEach" → "Use WheelsTest's …" examples/tweet/.opencode/command/wheels_execute.md (11 refs): Same treatment as the .claude/commands variant — this is the mirror opencode command file. Closes the TestBox → WheelsTest rename initiative across all forward- looking doc files. Only historical references, upstream URL labels, and the roadmap's preserved strategy sections remain. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
4 tasks
bpamiri
added a commit
that referenced
this pull request
Apr 16, 2026
) Editorial follow-up PR to #2127 and #2128. Covers the last three files deferred from the sweep: WHEELS-CLI-ROADMAP.md (cli/): Added a status banner at the top of the document noting that the testing-strategy sections predate the WheelsTest fork decision. The roadmap originally proposed integrating the external commandbox- testbox-cli module; the actual direction was to fork TestBox in-house as WheelsTest. Rather than rewriting the strategy sections (out of scope for rename), the banner makes the historical context clear and flags for a separate editorial pass. References preserved as-is for historical traceability: - Section 4 "Advanced Testing (via TestBox CLI Module)" - Leverage Existing Modules item 1 - Phase 1 Module Integration - CommandBox Integration Points - box.json dependency listing for commandbox-testbox-cli - Recommended Module Installation section examples/tweet/.claude/commands/wheels_execute.md (10 refs): - TestBox BDD → WheelsTest BDD (descriptive references) - testbox.system.TestBox → wheels.wheelstest.system.TestBox (code) - application.testbox → application.wheelstest (CFML scope) - "Use TestBox's beforeEach/afterEach" → "Use WheelsTest's …" examples/tweet/.opencode/command/wheels_execute.md (11 refs): Same treatment as the .claude/commands variant — this is the mirror opencode command file. Closes the TestBox → WheelsTest rename initiative across all forward- looking doc files. Only historical references, upstream URL labels, and the roadmap's preserved strategy sections remain. Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Editorial follow-up to #2127 (sweep PR) covering the largest user-facing testing doc:
docs/src/working-with-wheels/testing-your-application.md(39 refs).Applies the same three-bucket rubric from #2126/#2127, plus fixes stale code paths to match the actual
tests/runner.cfmimplementation. Deferred from the sweep because it needed per-line editorial judgment.Changes
Stale code paths corrected (per
tests/runner.cfm:99-246):testbox.system.TestBox→wheels.wheelstest.system.TestBoxtestbox.system.reports.JSONReporter→wheels.wheelstest.system.reports.JSONReportertestbox.system.reports.TextReporter→wheels.wheelstest.system.reports.TextReportertestbox.system.reports.ANTJUnitReporter→wheels.wheelstest.system.reports.ANTJUnitReporterFramework-identity references renamed:
TestBox 6→WheelsTest(drops upstream version qualifier)// load testbox routes→// load wheelstest routesUpstream URL references reframed (URLs kept, provenance clarified):
testbox.ortusbooks.comlink labeled with "upstream" and where appropriate "(WheelsTest preserves the same API)"Intro paragraph rewritten to establish provenance:
Legitimate retentions (10 refs)
Per the rubric's bucket 3:
testbox.ortusbooks.comtestBox = new wheels.wheelstest.system.TestBox(...)— matches actual runner.cfm conventionTest plan
tests/runner.cfm(namespace iswheels.wheelstest.system.*, functionsetTestboxEnvironment()kept as-is since that's the actual function name)Remaining editorial follow-up (separate PR)
Still deferred per the sweep plan:
cli/WHEELS-CLI-ROADMAP.md(15 refs)examples/tweet/.claude/commands/wheels_execute.md(10 refs) +.opencode/command/wheels_execute.md(11 refs)🤖 Generated with Claude Code