cameleer/cameleer-saas
2
0
Fork 0
Code Issues 22 Pull Requests Actions Packages Projects Releases Wiki Activity

fix: update VendorTenantServiceTest for async provisioning
All checks were successful
CI / build (push) Successful in 50s
Details
CI / docker (push) Successful in 45s
Details

Browse Source 
Tests now mock tenantRepository.findById() since provisionAsync re-loads
the tenant entity, and assert on the entity directly rather than the
return value of createAndProvision().

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This commit is contained in:
hsiegeln
2026-04-10 17:28:51 +02:00
parent 4447d79c92
commit a3a1643b37
1 changed files with 11 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

18
src/test/java/net/siegeln/cameleer/saas/vendor/VendorTenantServiceTest.java vendored
View File

@@ -135,13 +135,15 @@ class VendorTenantServiceTest {
when(licenseService.generateLicense(eq(tenant), any(Duration.class), eq(actorId))).thenReturn(license); when(licenseService.generateLicense(eq(tenant), any(Duration.class), eq(actorId))).thenReturn(license);
when(tenantProvisioner.isAvailable()).thenReturn(true); when(tenantProvisioner.isAvailable()).thenReturn(true);
when(tenantProvisioner.provision(any())).thenReturn(ProvisionResult.ok("http://server:8080")); when(tenantProvisioner.provision(any())).thenReturn(ProvisionResult.ok("http://server:8080"));
when(tenantRepository.findById(tenant.getId())).thenReturn(Optional.of(tenant));
when(tenantRepository.save(any(TenantEntity.class))).thenAnswer(inv -> inv.getArgument(0)); when(tenantRepository.save(any(TenantEntity.class))).thenAnswer(inv -> inv.getArgument(0));
var result = vendorTenantService.createAndProvision(request, actorId); vendorTenantService.createAndProvision(request, actorId);
assertThat(result.getStatus()).isEqualTo(TenantStatus.ACTIVE); // provisionAsync modifies the tenant entity in-place (runs synchronously in unit tests)
assertThat(result.getServerEndpoint()).isEqualTo("http://server:8080"); assertThat(tenant.getStatus()).isEqualTo(TenantStatus.ACTIVE);
assertThat(result.getProvisionError()).isNull(); assertThat(tenant.getServerEndpoint()).isEqualTo("http://server:8080");
assertThat(tenant.getProvisionError()).isNull();
verify(tenantRepository).save(tenant); verify(tenantRepository).save(tenant);
} }
@@ -156,12 +158,14 @@ class VendorTenantServiceTest {
when(licenseService.generateLicense(eq(tenant), any(Duration.class), eq(actorId))).thenReturn(license); when(licenseService.generateLicense(eq(tenant), any(Duration.class), eq(actorId))).thenReturn(license);
when(tenantProvisioner.isAvailable()).thenReturn(true); when(tenantProvisioner.isAvailable()).thenReturn(true);
when(tenantProvisioner.provision(any())).thenReturn(ProvisionResult.fail("Docker failure")); when(tenantProvisioner.provision(any())).thenReturn(ProvisionResult.fail("Docker failure"));
when(tenantRepository.findById(tenant.getId())).thenReturn(Optional.of(tenant));
when(tenantRepository.save(any(TenantEntity.class))).thenAnswer(inv -> inv.getArgument(0)); when(tenantRepository.save(any(TenantEntity.class))).thenAnswer(inv -> inv.getArgument(0));
var result = vendorTenantService.createAndProvision(request, actorId); vendorTenantService.createAndProvision(request, actorId);
assertThat(result.getProvisionError()).isEqualTo("Docker failure"); // provisionAsync modifies the tenant entity in-place (runs synchronously in unit tests)
assertThat(result.getStatus()).isEqualTo(TenantStatus.PROVISIONING); assertThat(tenant.getProvisionError()).isEqualTo("Docker failure");
assertThat(tenant.getStatus()).isEqualTo(TenantStatus.PROVISIONING);
verify(tenantRepository).save(tenant); verify(tenantRepository).save(tenant);
} }