test(cli): add TestBox test suite for LuCLI module#2093
Merged
Conversation
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Tests for Admin.generateAdmin() (controller + 5 views, form column filtering, FK loaders, route injection, force flag, noRoutes flag) and form helper type mapping covered indirectly via generated _form.cfm content (mapColumnToFormHelper is private).
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Add /wheels/cli/tests route, clitests view, Public.cfc handler, and /cli mapping so TestBox can discover cli.lucli.tests.specs. Remove standalone runner.cfm approach (Wheels Application.cfc intercepts all requests).
…li tests - fix reFindNoCase -> reFind for uppercase detection in Helpers.cfc pluralize/singularize, preventing case-insensitive match from corrupting all-lowercase words like "user" - scaffold temp project with all required dirs, config files, placeholder migration and test spec so Doctor health checks report HEALTHY instead of CRITICAL - fix TestHelper to avoid cfscript/tag parser confusion in string literals - add recommendation for missing tests/ directory in Doctor.buildRecommendations - use modelData.tableName for Admin view/controller paths to match spec expectations - fix CodeGenSpec: pass array for actions param, add required type to validateName - fix AdminSpec: rename form -> formContent to avoid CFML form scope collision - fix DoctorSpec: use < 10 char content to trigger routes.cfm minimal warning Result: 70/70 CLI tests pass (was 37 pass, 14 fail, 19 error)
bpamiri
force-pushed
the
peter/cli-testing
branch
from
65562e8 to
1260a16
Compare
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
1 participant
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
Adds a TestBox BDD test suite for the CLI module services and commands, integrated into the CI pipeline.
/wheels/cli/tests?format=jsonroute, reusing the same server instance as core testsTest coverage
Bug fixes found during testing
Helpers.cfc:reFindNoCasevsreFind— case-sensitive regex needed for uppercase detection in pluralize/singularizeAdmin.cfc: UsetableNamefrom introspection data for consistent path derivationDoctor.cfc: Add recommendation for missing tests directoryTest plan
🤖 Generated with Claude Code