MokoConsulting/MokoSuiteCross
1
0
Fork 0
Code Issues 2 Pull Requests Releases 5 Wiki Activity

feat(#132): PHPUnit test suite #185

Merged
jmiller merged 1 commits from feature/132-phpunit-tests into dev 2026-06-28 16:12:19 +00:00
Conversation 0 Commits 1 Files Changed 6
+378 -1
jmiller commented 2026-06-27 19:51:40 +00:00
Owner
Copy Link
Copy Source

Summary

  • Add phpunit.xml.dist, tests/bootstrap.php, and PSR-4 autoload config in composer.json
  • ServiceIconHelperTest: 14 tests for icon mapping and HTML rendering
  • PreviewHelperTest: 14 tests for platform mockup rendering (skipped until #156 merges)
  • ServiceInterfaceContractTest: 7 reflection tests verifying interface methods/types, plus 15 plugin implementation checks (skipped outside Joomla runtime)

21 tests pass immediately, 29 skip gracefully.

Test plan

  • Run composer install && vendor/bin/phpunit --testdox
  • Verify all ServiceIconHelper tests pass
  • Verify all ServiceInterfaceContract reflection tests pass
  • Verify PreviewHelper tests skip cleanly (class not yet on dev)

Closes #132

## Summary - Add `phpunit.xml.dist`, `tests/bootstrap.php`, and PSR-4 autoload config in `composer.json` - **ServiceIconHelperTest**: 14 tests for icon mapping and HTML rendering - **PreviewHelperTest**: 14 tests for platform mockup rendering (skipped until #156 merges) - **ServiceInterfaceContractTest**: 7 reflection tests verifying interface methods/types, plus 15 plugin implementation checks (skipped outside Joomla runtime) 21 tests pass immediately, 29 skip gracefully. ## Test plan - [ ] Run `composer install && vendor/bin/phpunit --testdox` - [ ] Verify all ServiceIconHelper tests pass - [ ] Verify all ServiceInterfaceContract reflection tests pass - [ ] Verify PreviewHelper tests skip cleanly (class not yet on dev) Closes #132
jmiller force-pushed feature/132-phpunit-tests from 3f33afd6b2 to d3a58a8dfd 2026-06-28 16:09:47 +00:00 Compare
jmiller added 1 commit 2026-06-28 16:12:09 +00:00
feat(#132): add PHPUnit test suite with unit tests
Universal: PR Check / Branch Policy (pull_request) Successful in 2s
Details
Universal: PR Check / Secret Scan (pull_request) Successful in 4s
Details
Universal: PR Check / Validate PR (pull_request) Failing after 5s
Details
RC Revert / Rename rc/ back to dev/ (pull_request) Has been skipped
Details
Universal: Auto Version Bump / Version Bump (push) Successful in 10s
Details
Branch Cleanup / Delete merged branch (pull_request) Failing after 1s
Details
Joomla: Metadata Validation / Validate Joomla Metadata (pull_request) Successful in 39s
Details
Universal: PR Check / Build RC Package (pull_request) Has been cancelled
Details
Universal: PR Check / Report Issues (pull_request) Has been cancelled
Details
a6dc736787 
Add phpunit.xml.dist, bootstrap, and PSR-4 autoload config. Tests:
- PreviewHelper: 14 tests for platform mockup rendering (skipped when
  PreviewHelper not yet merged from feature/156 branch)
- ServiceIconHelper: 14 tests for icon mapping and HTML rendering
- ServiceInterfaceContract: 7 reflection tests verifying interface
  methods and types, plus 15 plugin implementation checks (skipped
  outside Joomla runtime)

21 tests pass immediately, 29 skip gracefully.

Authored-by: Moko Consulting
jmiller force-pushed feature/132-phpunit-tests from 7528074622 to a6dc736787 2026-06-28 16:12:09 +00:00 Compare
jmiller merged commit db061e2b75 into dev 2026-06-28 16:12:19 +00:00
jmiller deleted branch feature/132-phpunit-tests 2026-06-28 16:12:19 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
No items
No labels
Priority -
Type -
Milestone
No items
No Milestone
Assignees
Clear assignees
jmiller (Jonathan Miller)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: MokoConsulting/MokoSuiteCross#185
Delete Branch "feature/132-phpunit-tests"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?