hostsharing/hs.hsadmin.ng
1
0
Fork 0
Code Activity

add Test canNotAddDebitorRelationWithAnchorThatIsNotAPartner for constraint

Browse Source
This commit is contained in:
Timotheus Pokorra
2025-04-15 12:15:26 +02:00
parent d8dc5655f3
commit 38efe866af
1 changed files with 48 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

53
src/test/java/net/hostsharing/hsadminng/hs/office/relation/HsOfficeRealRelationRepositoryIntegrationTest.java
View File

@@ -1,7 +1,9 @@
package net.hostsharing.hsadminng.hs.office.relation; package net.hostsharing.hsadminng.hs.office.relation;
import net.hostsharing.hsadminng.context.Context; import net.hostsharing.hsadminng.context.Context;
import net.hostsharing.hsadminng.hs.office.person.HsOfficePersonRbacRepository; import net.hostsharing.hsadminng.hs.office.contact.HsOfficeContactRealRepository;
import net.hostsharing.hsadminng.hs.office.person.HsOfficePersonRealEntity;
import net.hostsharing.hsadminng.hs.office.person.HsOfficePersonRealRepository;
import net.hostsharing.hsadminng.hs.office.person.HsOfficePersonType; import net.hostsharing.hsadminng.hs.office.person.HsOfficePersonType;
import net.hostsharing.hsadminng.rbac.test.ContextBasedTestWithCleanup; import net.hostsharing.hsadminng.rbac.test.ContextBasedTestWithCleanup;
import net.hostsharing.hsadminng.rbac.test.JpaAttempt; import net.hostsharing.hsadminng.rbac.test.JpaAttempt;
@@ -21,6 +23,8 @@ import java.util.UUID;
import static net.hostsharing.hsadminng.hs.office.person.HsOfficePersonType.NATURAL_PERSON; import static net.hostsharing.hsadminng.hs.office.person.HsOfficePersonType.NATURAL_PERSON;
import static net.hostsharing.hsadminng.hs.office.relation.HsOfficeRelationType.REPRESENTATIVE; import static net.hostsharing.hsadminng.hs.office.relation.HsOfficeRelationType.REPRESENTATIVE;
import static net.hostsharing.hsadminng.rbac.test.EntityList.one;
import static net.hostsharing.hsadminng.rbac.test.JpaAttempt.attempt;
import static org.assertj.core.api.Assertions.assertThat; import static org.assertj.core.api.Assertions.assertThat;
@DataJpaTest @DataJpaTest
@@ -29,10 +33,13 @@ import static org.assertj.core.api.Assertions.assertThat;
class HsOfficeRealRelationRepositoryIntegrationTest extends ContextBasedTestWithCleanup { class HsOfficeRealRelationRepositoryIntegrationTest extends ContextBasedTestWithCleanup {
@Autowired @Autowired
HsOfficeRelationRealRepository relationRealRepo; HsOfficeRelationRealRepository realRelationRepo;
@Autowired @Autowired
HsOfficePersonRbacRepository personRepo; HsOfficePersonRealRepository realPersonRepo;
@Autowired
HsOfficeContactRealRepository realContactRepo;
@PersistenceContext @PersistenceContext
EntityManager em; EntityManager em;
@@ -49,7 +56,7 @@ class HsOfficeRealRelationRepositoryIntegrationTest extends ContextBasedTestWith
final var personUuid = determinePersonUuid(NATURAL_PERSON, "Smith"); final var personUuid = determinePersonUuid(NATURAL_PERSON, "Smith");
// when // when
final var result = relationRealRepo.findRelationRelatedToPersonUuid(personUuid); final var result = realRelationRepo.findRelationRelatedToPersonUuid(personUuid);
// then // then
context("superuser-alex@hostsharing.net"); // just to be able to access RBAc-entities persons+contact context("superuser-alex@hostsharing.net"); // just to be able to access RBAc-entities persons+contact
@@ -68,7 +75,7 @@ class HsOfficeRealRelationRepositoryIntegrationTest extends ContextBasedTestWith
final var personUuid = determinePersonUuid(NATURAL_PERSON, "Smith"); final var personUuid = determinePersonUuid(NATURAL_PERSON, "Smith");
// when: // when:
final var result = relationRealRepo.findRelationRelatedToPersonUuidRelationTypeMarkPersonAndContactData(personUuid, REPRESENTATIVE, null, null, null); final var result = realRelationRepo.findRelationRelatedToPersonUuidRelationTypeMarkPersonAndContactData(personUuid, REPRESENTATIVE, null, null, null);
// then: // then:
context("superuser-alex@hostsharing.net"); // just to be able to access RBAc-entities persons+contact context("superuser-alex@hostsharing.net"); // just to be able to access RBAc-entities persons+contact
@@ -79,6 +86,34 @@ class HsOfficeRealRelationRepositoryIntegrationTest extends ContextBasedTestWith
} }
} }
@Nested
class CreateRelation {
@Test
public void canNotAddDebitorRelationWithAnchorThatIsNotAPartner() {
// given
context("superuser-alex@hostsharing.net");
final var givenPartnerPerson = hsOfficePersonRealEntity("Non-Partner Person");
final var givenContact = one(realContactRepo.findContactByOptionalCaptionLike("eleventh contact"));
// when
final var result = attempt(em, () -> {
final var newRelation = HsOfficeRelationRealEntity.builder()
.type(HsOfficeRelationType.DEBITOR)
.anchor(givenPartnerPerson)
.holder(givenPartnerPerson)
.contact(givenContact)
.build();
final var entity = realRelationRepo.save(newRelation);
return toCleanup(entity.load());
});
// then
result.assertExceptionWithRootCauseMessage(org.postgresql.util.PSQLException.class, "ERROR: [400] invalid debitor relation: anchor person must have a PARTNER relation");
}
}
private UUID determinePersonUuid(final HsOfficePersonType type, final String familyName) { private UUID determinePersonUuid(final HsOfficePersonType type, final String familyName) {
return (UUID) em.createNativeQuery(""" return (UUID) em.createNativeQuery("""
SELECT uuid FROM hs_office.person p SELECT uuid FROM hs_office.person p
@@ -97,4 +132,12 @@ class HsOfficeRealRelationRepositoryIntegrationTest extends ContextBasedTestWith
.extracting(HsOfficeRelation::toString) .extracting(HsOfficeRelation::toString)
.containsExactlyInAnyOrder(relationNames); .containsExactlyInAnyOrder(relationNames);
} }
public HsOfficePersonRealEntity hsOfficePersonRealEntity(final String tradeName) {
return realPersonRepo.save(HsOfficePersonRealEntity.builder()
.personType(HsOfficePersonType.NATURAL_PERSON)
.tradeName(tradeName)
.build());
}
} }