Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Add owner by email #21

Merged
merged 1 commit into from
Nov 17, 2024
Merged

Add owner by email #21

merged 1 commit into from
Nov 17, 2024

Conversation

frankbille
Copy link
Member

No description provided.

@frankbille frankbille added the enhancement New feature or request label Nov 17, 2024
@frankbille frankbille self-assigned this Nov 17, 2024
@github-actions GitHub Actions
Copy link

Test Results

34 tests  +3   34 ✅ +3   1m 8s ⏱️ +6s
19 suites +2    0 💤 ±0 
19 files   +2    0 ❌ ±0 

Results for commit 4fb95ee. ± Comparison against base commit 7e3a785.

This pull request removes 1 and adds 4 tests. Note that renamed tests count towards both. 
info.jotajoti.jampuz.location.LocationControllerIntegrationTests$AddOwnerTests ‑ should not be allowed to add self to not-owned location

info.jotajoti.jampuz.location.LocationServiceTests$AddOwnerTests ‑ should add owner to location
info.jotajoti.jampuz.location.LocationServiceTests$AddOwnerTests ‑ should fail if admin not found by email
info.jotajoti.jampuz.location.LocationServiceTests$AddOwnerTests ‑ should fail if location not found
info.jotajoti.jampuz.location.LocationServiceTests$AddOwnerTests ‑ should fail if trying to add self to location

@frankbille frankbille merged commit 5016dca into main Nov 17, 2024
4 checks passed
@frankbille frankbille deleted the add-owner-by-email branch November 17, 2024 20:26
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers
No reviews
Assignees

@frankbille frankbille

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@frankbille