fixing code-review-issues
This commit is contained in:
parent
ead3fa2053
commit
3356868bf0
@ -154,8 +154,11 @@ class HsHostingAssetRepositoryIntegrationTest extends ContextBasedTestWithCleanu
|
|||||||
}
|
}
|
||||||
|
|
||||||
private void assertThatAssetIsPersisted(final HsHostingAssetEntity saved) {
|
private void assertThatAssetIsPersisted(final HsHostingAssetEntity saved) {
|
||||||
final var found = assetRepo.findByUuid(saved.getUuid());
|
attempt(em, () -> {
|
||||||
assertThat(found).isNotEmpty().map(HsHostingAssetEntity::getVersion).get().isEqualTo(saved.getVersion());
|
context("superuser-alex@hostsharing.net");
|
||||||
|
final var found = assetRepo.findByUuid(saved.getUuid());
|
||||||
|
assertThat(found).isNotEmpty().map(HsHostingAssetEntity::toString).get().isEqualTo(saved.toString());
|
||||||
|
});
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
@ -178,14 +181,6 @@ class HsHostingAssetRepositoryIntegrationTest extends ContextBasedTestWithCleanu
|
|||||||
"HsHostingAssetEntity(MANAGED_WEBSPACE, thi01, some Webspace, MANAGED_SERVER:vm1013, D-1000313:D-1000313 default project:separate ManagedWebspace)");
|
"HsHostingAssetEntity(MANAGED_WEBSPACE, thi01, some Webspace, MANAGED_SERVER:vm1013, D-1000313:D-1000313 default project:separate ManagedWebspace)");
|
||||||
}
|
}
|
||||||
|
|
||||||
public <R> R stopWatch(final String caption, final Supplier<R> operation) {
|
|
||||||
long start = System.nanoTime();
|
|
||||||
final R result = operation.get();
|
|
||||||
System.out.printf("StopWatch %s: %dms\n",
|
|
||||||
caption, TimeUnit.NANOSECONDS.toMillis(System.nanoTime() - start));
|
|
||||||
return result;
|
|
||||||
}
|
|
||||||
|
|
||||||
@Test
|
@Test
|
||||||
public void normalUser_canViewOnlyRelatedAsset() {
|
public void normalUser_canViewOnlyRelatedAsset() {
|
||||||
// given:
|
// given:
|
||||||
@ -194,9 +189,7 @@ class HsHostingAssetRepositoryIntegrationTest extends ContextBasedTestWithCleanu
|
|||||||
.findAny().orElseThrow().getUuid();
|
.findAny().orElseThrow().getUuid();
|
||||||
|
|
||||||
// when:
|
// when:
|
||||||
final var result = stopWatch("findAllByCriteria", () ->
|
final var result = assetRepo.findAllByCriteria(projectUuid, null, null);
|
||||||
assetRepo.findAllByCriteria(projectUuid, null, null)
|
|
||||||
);
|
|
||||||
|
|
||||||
// then:
|
// then:
|
||||||
exactlyTheseAssetsAreReturned(
|
exactlyTheseAssetsAreReturned(
|
||||||
|
Loading…
Reference in New Issue
Block a user