Use 10 characters for medium hash. Use parent context for new id.

NDLANO · Mar 12, 2024 · e96db97 · e96db97
1 parent d0dfe70
commit e96db97
Show file tree

Hide file tree

Showing 4 changed files with 7 additions and 11 deletions.
diff --git a/src/main/java/no/ndla/taxonomy/service/ContextUpdaterServiceImpl.java b/src/main/java/no/ndla/taxonomy/service/ContextUpdaterServiceImpl.java
@@ -71,7 +71,7 @@ private Set<Context> createContexts(Node node) {
                                                 .flatMap(relevance -> Optional.of(
                                                         relevance.getPublicId().toString()))
                                                 .orElse("urn:relevance:core"),
-                                        HashUtil.semiHash(parentContext.rootId() + parentConnection.getPublicId()),
+                                        HashUtil.mediumHash(parentContext.contextId() + parentConnection.getPublicId()),
                                         parentConnection.getRank(),
                                         parentConnection.getPublicId().toString());
                             })

diff --git a/src/main/java/no/ndla/taxonomy/util/HashUtil.java b/src/main/java/no/ndla/taxonomy/util/HashUtil.java
@@ -24,7 +24,7 @@ public static String shortHash(Object original) {
     }
 
     public static String mediumHash(Object original) {
-        return generateHash(original, 8);
+        return generateHash(original, 10);
     }
 
     public static String semiHash(Object original) {

diff --git a/src/test/java/no/ndla/taxonomy/rest/v1/QueryTest.java b/src/test/java/no/ndla/taxonomy/rest/v1/QueryTest.java
@@ -402,14 +402,10 @@ public void that_contexts_can_be_found_by_path() throws Exception {
                                 .contentUri("urn:article:1")
                                 .publicId("urn:resource:1"))));
         Node resource = nodeRepository.getByPublicId(URI.create("urn:resource:1"));
-        String hash = HashUtil.semiHash(root.getPublicId().toString()
-                + resource.getParentConnections().stream()
-                        .findFirst()
-                        .get()
-                        .getPublicId()
-                        .toString());
-
-        var response = testUtils.getResource("/v1/queries/path?path=" + String.format("/one-fine-resource__%s", hash));
+        var response = testUtils.getResource("/v1/queries/path?path="
+                + String.format(
+                "/one-fine-resource__%s",
+                resource.getContexts().stream().findFirst().get().contextId()));
         var result = testUtils.getObject(TaxonomyContextDTO[].class, response);
 
         assertEquals(1, result.length);

diff --git a/src/test/java/no/ndla/taxonomy/util/HashUtilTest.java b/src/test/java/no/ndla/taxonomy/util/HashUtilTest.java
@@ -22,7 +22,7 @@ public void get_different_length_hashes() {
         String fullHash = HashUtil.fullHash("original");
 
         assertEquals(4, shortHash.length());
-        assertEquals(8, mediumHash.length());
+        assertEquals(10, mediumHash.length());
         assertEquals(12, semiHash.length());
         assertEquals(16, longHash.length());
         assertTrue(fullHash.length() > 16);