Introduce Infrastructure testing - #4
Merged
Merged
Conversation
Detailed description: - Updated README to introduce the `the-firehub-project/testing` package as a centralized library for the FireHub ecosystem's shared testing components. - Added new badges (PHPStan, PHPUnit, Codecov) for workflow and coverage tracking. - Replaced `develop` branch references with `feature/testing-infrastructure` and updated badge links accordingly. - Documented branch purpose, architectural goals, and core concepts for the Testing package, ensuring clarity on its framework-agnostic design and separation from runtime dependencies. - Updated license reference from MIT to Apache-2.0.
Closed
2 tasks
dieselxxx
added
the
type: Architecture
dieselxxx
changed the title
Detailed description: - Introduced stubs: `CountableClass`, `EmptyClass`, `EmptyEnum`, `EmptyInterface`, and `EmptyTrait` for use in generic testing scenarios across the FireHub ecosystem. - Added `ArrDataProvider`, `NumDataProvider`, `StrDataProvider`, `DataDataProvider`, `DateDataProvider`, `ResourceDataProvider`, and `ConstantDataProvider` to supply structured data for PHPUnit tests. - Ensured comprehensive coverage of core data types (scalars, arrays, objects, resources, and dates) to streamline test development and enhance reusability. - Included necessary metadata and licensing information in all newly created classes.
Detailed description: - Replaced the `Apache License, Version 2.0` with the `MIT License` in all stub and data provider classes within the `Testing` package. - Ensured all affected files include updated licensing metadata in their docblocks. - Maintained compliance with FireHub's licensing changes while ensuring clarity and consistency across the package.
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.
🧱 System Design / Architecture Pull Request
Related Issue
Closes Infrastructure testing #3
Description
This architectural decision introduces the
the-firehub-project/testingpackage as a shared verification infrastructure layer for the FireHub ecosystem.The package defines a centralized location for reusable testing components, including Data Providers, Stubs, Fixtures, and common testing utilities that can be consumed by all FireHub repositories.
Architectural Decision
The FireHub Testing package establishes a dedicated layer responsible only for test support and verification.
Architecture Overview
System Design
Design Decisions
System Boundaries
Included
Excluded
Implementation
Core Changes
Contracts
Dependencies
Validation
Architecture Compliance
Build & Stability
Impact Analysis
Core Impact
Runtime Impact
Future Extensions
Risks / Constraints
Risk Level
Potential Risks
Documentation
Checklist
Notes
Technical Notes
Future Work
Reviewer Guidance