From 563024eea5939081dad682d2e727e938e96cfbdf Mon Sep 17 00:00:00 2001 From: Luca Frosini Date: Wed, 19 Apr 2023 11:38:42 +0200 Subject: [PATCH] Porting code to redesigned E/R format --- .../queries/templates/ContextTest.java | 4 +-- ...sourceRegistryQueryTemplateClientTest.java | 31 ++++++++++--------- 2 files changed, 19 insertions(+), 16 deletions(-) diff --git a/src/test/java/org/gcube/informationsystem/resourceregistry/queries/templates/ContextTest.java b/src/test/java/org/gcube/informationsystem/resourceregistry/queries/templates/ContextTest.java index 1ddcaff..d30cccf 100644 --- a/src/test/java/org/gcube/informationsystem/resourceregistry/queries/templates/ContextTest.java +++ b/src/test/java/org/gcube/informationsystem/resourceregistry/queries/templates/ContextTest.java @@ -15,7 +15,7 @@ import org.gcube.common.authorization.utils.secret.SecretUtility; import org.gcube.common.keycloak.KeycloakClientFactory; import org.gcube.common.keycloak.model.TokenResponse; import org.gcube.common.scope.api.ScopeProvider; -import org.gcube.informationsystem.model.reference.properties.Header; +import org.gcube.informationsystem.model.reference.properties.Metadata; import org.junit.AfterClass; import org.junit.BeforeClass; import org.slf4j.Logger; @@ -117,7 +117,7 @@ public class ContextTest { } public static String getUser() { - String user = Header.UNKNOWN_USER; + String user = Metadata.UNKNOWN_USER; try { user = SecretManagerProvider.instance.get().getUser().getUsername(); } catch(Exception e) { diff --git a/src/test/java/org/gcube/informationsystem/resourceregistry/queries/templates/ResourceRegistryQueryTemplateClientTest.java b/src/test/java/org/gcube/informationsystem/resourceregistry/queries/templates/ResourceRegistryQueryTemplateClientTest.java index 60febd4..845a53e 100644 --- a/src/test/java/org/gcube/informationsystem/resourceregistry/queries/templates/ResourceRegistryQueryTemplateClientTest.java +++ b/src/test/java/org/gcube/informationsystem/resourceregistry/queries/templates/ResourceRegistryQueryTemplateClientTest.java @@ -10,7 +10,7 @@ import org.gcube.com.fasterxml.jackson.databind.ObjectMapper; import org.gcube.com.fasterxml.jackson.databind.node.ObjectNode; import org.gcube.informationsystem.base.reference.IdentifiableElement; import org.gcube.informationsystem.model.reference.entities.Entity; -import org.gcube.informationsystem.model.reference.properties.Header; +import org.gcube.informationsystem.model.reference.properties.Metadata; import org.gcube.informationsystem.queries.templates.impl.entities.QueryTemplateImpl; import org.gcube.informationsystem.queries.templates.impl.properties.TemplateVariableImpl; import org.gcube.informationsystem.queries.templates.reference.entities.QueryTemplate; @@ -45,29 +45,32 @@ public class ResourceRegistryQueryTemplateClientTest extends ContextTest { } } - public static void checkHeader(IdentifiableElement previous, IdentifiableElement got) { - Header gotHeader = got.getHeader(); - Header previousHeader = previous.getHeader(); + public static void checkUUIDAndMetadata(IdentifiableElement previous, IdentifiableElement got) { + Metadata gotMetadata = got.getMetadata(); + Metadata previousMetadata = previous.getMetadata(); - Assert.assertTrue(gotHeader != null); - Assert.assertTrue(gotHeader.getUUID() != null); + Assert.assertTrue(gotMetadata != null); + Assert.assertTrue(got.getUUID() != null); String user = ContextTest.getUser(); - Assert.assertTrue(gotHeader.getLastUpdateBy().compareTo(user) == 0); + Assert.assertTrue(gotMetadata.getLastUpdateBy().compareTo(user) == 0); - if(previousHeader != null) { - Assert.assertTrue(gotHeader.getUUID().compareTo(previousHeader.getUUID()) == 0); - Assert.assertTrue(gotHeader.getCreatedBy().compareTo(user) == 0); - Assert.assertTrue(gotHeader.getCreatedBy().compareTo(previousHeader.getCreatedBy()) == 0); - Assert.assertTrue(gotHeader.getCreationTime().compareTo(previousHeader.getCreationTime()) == 0); + if(previous.getUUID()!=null) { + Assert.assertTrue(got.getUUID().compareTo(previous.getUUID()) == 0); + } + + if(previousMetadata != null) { + Assert.assertTrue(gotMetadata.getCreatedBy().compareTo(user) == 0); + Assert.assertTrue(gotMetadata.getCreatedBy().compareTo(previousMetadata.getCreatedBy()) == 0); + Assert.assertTrue(gotMetadata.getCreationTime().compareTo(previousMetadata.getCreationTime()) == 0); } - Assert.assertFalse(gotHeader.getCreationTime().after(gotHeader.getLastUpdateTime())); + Assert.assertFalse(gotMetadata.getCreationTime().after(gotMetadata.getLastUpdateTime())); } protected void assertions(QueryTemplate expected, QueryTemplate got) { - checkHeader(expected, got); + checkUUIDAndMetadata(expected, got); Assert.assertTrue(expected.getName().compareTo(got.getName()) == 0); Assert.assertTrue(expected.getDescription().compareTo(got.getDescription()) == 0); // Template and TemplateVariable should be compared