Merged
Conversation
- Added new `wheels test coverage` command to run tests with coverage reporting. - Implemented the core functionality to execute the test suite and generate coverage output. - Options and flags (e.g., --html, --threshold) are in progress and will be completed in upcoming commits. - Command is functional but not yet finalized for all available options.
Refactored test command components to eliminate code duplication and improve maintainability: - Extracted shared test execution logic into base.cfc component - Added executeTestRequest() for HTTP test requests - Added buildTestUrl() to construct test URLs with proper server configuration - Added parseTestResponse() for handling different response formats - Added getServerConfig() to retrieve actual running server host/port - Added helper functions for coverage processing and formatting - Updated coverage.cfc to use base functions - Replaced inline HTTP execution with executeTestRequest() - Replaced URL building logic with buildTestUrl() - Maintained all existing coverage reporting functionality - Updated run.cfc (test runner) to use base functions - Replaced duplicate HTTP and URL building code - Added proper test output formatting matching legacy format - Fixed response structure compatibility issues - Added support for different TestBox response formats - Updated legacy run.cfc to use base functions - Maintained backward compatibility - Replaced inline implementations with base function calls This refactoring reduces code duplication across test commands while maintaining full backward compatibility and existing functionality.
Updated test directory path to ensure new test files are detected and executed.
- Added real time processor for "wheels analyze code" - Updated guides for "wheels assets" commands
- Removed watch options from run command as its already a sub command available "wheels test watch" - Deleted test.cfc as its now converted into new command "wheels test run", existence of this file was creating issue to run sub commands in test folder. - Guides updated according to current working of commands
- Removed extra functions in base.cfc - Restored test.cfc as the latest code for "wheels test run" not functional.
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.
No description provided.