test: add tests for WorkspaceTeamService #66

Open

Charon wants to merge 0 commits from feat/test-workspace-team-service into dev

Charon commented

2026-03-24 13:36:07 +00:00

Member

Fixes #15

Summary

Adds tests/Feature/WorkspaceTeamServiceTest.php with 50 test methods
Covers team CRUD (create, read, update, delete)
Covers member management (add/remove from team, custom permissions)
Covers permission checks (hasPermission, hasAny, hasAll, isOwner, isAdmin)
Covers member queries (getMembersWithPermission, count methods)
Covers seeding (seedDefaultTeams, ensureDefaultTeams, idempotency)
Covers member migration (role-based team assignment)
Covers edge cases (no workspace context, cross-workspace isolation, system team protection)
Verifies default team permission sets (owner=all, admin=no delete/billing, viewer=read-only)

Test Plan

Run composer test to verify all tests pass
Verify no regressions in existing test suite

Fixes #15 ## Summary - Adds `tests/Feature/WorkspaceTeamServiceTest.php` with 50 test methods - Covers team CRUD (create, read, update, delete) - Covers member management (add/remove from team, custom permissions) - Covers permission checks (hasPermission, hasAny, hasAll, isOwner, isAdmin) - Covers member queries (getMembersWithPermission, count methods) - Covers seeding (seedDefaultTeams, ensureDefaultTeams, idempotency) - Covers member migration (role-based team assignment) - Covers edge cases (no workspace context, cross-workspace isolation, system team protection) - Verifies default team permission sets (owner=all, admin=no delete/billing, viewer=read-only) ## Test Plan - [ ] Run `composer test` to verify all tests pass - [ ] Verify no regressions in existing test suite

This branch is already included in the target branch. There is nothing to merge.

View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.

git fetch -u origin feat/test-workspace-team-service:feat/test-workspace-team-service

git checkout feat/test-workspace-team-service

Merge

Merge the changes and update on Forgejo.

Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.

git checkout dev

git merge --no-ff feat/test-workspace-team-service

git checkout feat/test-workspace-team-service

git rebase dev

git checkout dev

git merge --ff-only feat/test-workspace-team-service

git checkout feat/test-workspace-team-service

git rebase dev

git checkout dev

git merge --no-ff feat/test-workspace-team-service

git checkout dev

git merge --squash feat/test-workspace-team-service