Introduce FireHubTestCase base abstraction#2
Merged
Conversation
…on-and-initial-tests` branch **Details added:** - Replaced `develop` branch references with `feature/phpunit-integration-and-initial-tests` to align README with the active branch name. - Updated badges to display commit activity, differences, and last commit info for the new branch. - Introduced a section detailing the purpose of the branch, highlighting the abstraction of `FireHubTestCase` as the base for ecosystem-wide testing. - Added architectural goals for standardizing and decoupling FireHub tests from PHPUnit. - Enhanced descriptions around the design of testing layers, unifying test suite structures. - Updated the license reference from MIT to Apache-2.0 for consistency with repository changes.
1 task
**Details added:** - Created the abstract `FireHubTestCase` class extending PHPUnit's `TestCase` to standardize testing across FireHub's Core, Runtime, and ecosystem packages. - Configured PSR-4 autoloading for the `FireHub\Testing` namespace in `composer.json`. - Included metadata, licensing, and versioning details for clarity and future maintainability. - Enhanced test structure extensibility for shared utilities and consistent architecture.
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
2 participants
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.
💢 Feature / Enhancement Pull Request
Related Issue
Closes FireHubTestCase base abstraction #1
Description
This PR introduces the
FireHubTestCasebase abstraction as the foundational testing class for the FireHub ecosystem.It is needed to establish a standardized and reusable testing entry point across all FireHub repositories, removing direct dependency usage of
PHPUnit\Framework\TestCasein higher-level test suites and enabling a unified testing structure.This improves the FireHub ecosystem by:
Context / design notes:
FireHubTestCaseacts as a thin abstraction over PHPUnit’sTestCaseCoreTestCase,IntegrationTestCase,ArchitectureTestCase)Screenshots (if applicable):
Feature Overview
Functional Changes
FireHub\Testing\FireHubTestCasebase abstractionPHPUnit\Framework\TestCaseas a unified testing entry point for the FireHub ecosystemAPI / Behavior Changes
Public API changes:
FireHubTestCaseInternal behavior changes:
Dependencies
phpunit/phpunit(dev dependency, if not already present)Compatibility
Testing
Validation
Automated Tests
Documentation
Checklist
Risks / Impact
Potential Impacts
Side effects:
Affected modules:
QA Focus Areas
Notes
Technical Notes
FireHubTestCaseis intentionally kept as a thin abstraction over PHPUnitQA / Reviewer Notes