refactor: rename context accessors to use "get" prefix for consistency

Author: Pfeil

src/main/java/edu/kit/datamanager/ro_crate/Crate.java

@@ -57,19 +57,19 @@ public interface Crate {
    * @param key the key to be searched
    * @return the value of the key, null if not found
    */
-  String readMetadataContextValueOf(String key);
+  String getMetadataContextValueOf(String key);
 
   /**
    * Get an immutable collection of the keys in the metadata context.
    * @return the keys in the metadata context
    */
-  Set<String> readMetadataContextKeys();
+  Set<String> getMetadataContextKeys();
 
   /**
    * Get an immutable map of the context.
    * @return an immutable map containing the context key-value pairs
    */
-  Map<String, String> readMetadataContextPairs();
+  Map<String, String> getMetadataContextPairs();
 
   RootDataEntity getRootDataEntity();
 

src/main/java/edu/kit/datamanager/ro_crate/RoCrate.java

@@ -62,18 +62,18 @@ public void setMetadataContext(CrateMetadataContext metadataContext) {
         this.metadataContext = metadataContext;
     }
 
-    public String readMetadataContextValueOf(String key) {
-        return this.metadataContext.readValueOf(key);
+    public String getMetadataContextValueOf(String key) {
+        return this.metadataContext.getValueOf(key);
     }
 
     @Override
-    public Set<String> readMetadataContextKeys() {
-        return this.metadataContext.readKeys();
+    public Set<String> getMetadataContextKeys() {
+        return this.metadataContext.getKeys();
     }
 
     @Override
-    public Map<String, String> readMetadataContextPairs() {
-        return this.metadataContext.readPairs();
+    public Map<String, String> getMetadataContextPairs() {
+        return this.metadataContext.getPairs();
     }
 
     public ContextualEntity getJsonDescriptor() {

src/main/java/edu/kit/datamanager/ro_crate/context/CrateMetadataContext.java

@@ -30,19 +30,19 @@ public interface CrateMetadataContext {
    * @param key the key to be searched
    * @return the value of the key, null if not found
    */
-  String readValueOf(String key);
+  String getValueOf(String key);
 
   /**
    * Get an immutable collection of the keys in the metadata context.
    * @return the keys in the metadata context
    */
-  Set<String> readKeys();
+  Set<String> getKeys();
 
   /**
   * Get an immutable map of the context.
   * @return an immutable map containing the context key-value pairs
   */
-  Map<String, String> readPairs();
+  Map<String, String> getPairs();
 
   void deleteValuePairFromContext(String key);
 

src/main/java/edu/kit/datamanager/ro_crate/context/RoCrateMetadataContext.java

@@ -201,21 +201,21 @@ public void addToContext(String key, String value) {
   }
 
   @Override
-  public String readValueOf(String key) {
+  public String getValueOf(String key) {
     return Optional.ofNullable(this.contextMap.get(key))
             .orElseGet(() -> this.other.get(key));
   }
 
   @Override
-  public Set<String> readKeys() {
+  public Set<String> getKeys() {
     List<String> merged = new ArrayList<>();
     merged.addAll(this.contextMap.keySet());
     merged.addAll(this.other.keySet());
     return Set.copyOf(merged);
   }
 
   @Override
-  public Map<String, String> readPairs() {
+  public Map<String, String> getPairs() {
     Map<String, String> merged = new HashMap<>();
     merged.putAll(this.contextMap);
     merged.putAll(this.other);

src/test/java/edu/kit/datamanager/ro_crate/context/ContextTest.java

@@ -238,26 +238,26 @@ void testSetDeleteGetPair() {
     String key = "key";
     String value = "value";
     context.addToContext(key, value);
-    assertEquals(value, context.readValueOf(key));
+    assertEquals(value, context.getValueOf(key));
     context.deleteValuePairFromContext(key);
-    assertNull(context.readValueOf(key));
+    assertNull(context.getValueOf(key));
   }
 
   @Test
   void testReadDeleteGetPair() {
     String key = "custom";
     String value = "_:";
-    assertEquals(value, this.complexContext.readValueOf(key));
+    assertEquals(value, this.complexContext.getValueOf(key));
     this.complexContext.deleteValuePairFromContext(key);
-    assertNull(this.complexContext.readValueOf(key));
+    assertNull(this.complexContext.getValueOf(key));
     this.complexContext.addToContext(key, value);
-    assertEquals(value, this.complexContext.readValueOf(key));
+    assertEquals(value, this.complexContext.getValueOf(key));
   }
 
   @Test
   void testReadKeys() {
     var expected = Set.of("custom", "owl", "datacite", "xsd", "rdfs");
-    var given = this.complexContext.readKeys();
+    var given = this.complexContext.getKeys();
     for (String key : expected) {
       assertTrue(given.contains(key), "Key " + key + " not found in the context");
     }
@@ -268,7 +268,7 @@ void testReadKeys() {
   @Test
   void testReadPairs() {
     var expected = Set.of("custom", "owl", "datacite", "xsd", "rdfs");
-    var given = this.complexContext.readPairs();
+    var given = this.complexContext.getPairs();
     var keys = given.keySet();
     var values = given.values();
     for (String key : expected) {

src/test/java/edu/kit/datamanager/ro_crate/crate/TestRemoveAddContext.java

@@ -50,15 +50,15 @@ void testAddRemoveUrl() throws JsonProcessingException {
   void testReadDeleteGetContextPair() {
     String key = "custom";
     String value = "_:";
-    assertEquals(value, this.crateWithComplexContext.readMetadataContextValueOf(key));
+    assertEquals(value, this.crateWithComplexContext.getMetadataContextValueOf(key));
     this.crateWithComplexContext.deleteValuePairFromContext(key);
-    assertNull(this.crateWithComplexContext.readMetadataContextValueOf(key));
+    assertNull(this.crateWithComplexContext.getMetadataContextValueOf(key));
   }
 
   @Test
   void testReadContextKeys() {
     var expected = Set.of("custom", "owl", "datacite", "xsd", "rdfs");
-    var given = this.crateWithComplexContext.readMetadataContextKeys();
+    var given = this.crateWithComplexContext.getMetadataContextKeys();
     for (String key : expected) {
         assertTrue(given.contains(key), "Key " + key + " not found in the context");
     }
@@ -69,7 +69,7 @@ void testReadContextKeys() {
   @Test
   void testReadContextPairs() {
     var expected = Set.of("custom", "owl", "datacite", "xsd", "rdfs");
-    var given = this.crateWithComplexContext.readMetadataContextPairs();
+    var given = this.crateWithComplexContext.getMetadataContextPairs();
     var keys = given.keySet();
     var values = given.values();
     for (String key : expected) {