Change name of get graffiti in manager

courtneyeh · Apr 19, 2024 · ba2311a · ba2311a
1 parent b24311c
commit ba2311a
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 6 deletions.
diff --git a/validator/api/src/main/java/tech/pegasys/teku/validator/api/GraffitiManager.java b/validator/api/src/main/java/tech/pegasys/teku/validator/api/GraffitiManager.java
@@ -76,7 +76,7 @@ private Optional<String> updateGraffiti(final BLSPublicKey publicKey, final Stri
     return Optional.empty();
   }
 
-  public Optional<Bytes32> getGraffitiFromStorage(final BLSPublicKey publicKey) {
+  public Optional<Bytes32> getGraffiti(final BLSPublicKey publicKey) {
     final Path filePath = graffitiPath.resolve(resolveFileName(publicKey));
     try {
       return Optional.of(GraffitiParser.loadFromFile(filePath)).filter(this::graffitiNotEmpty);

diff --git a/validator/api/src/test/java/tech/pegasys/teku/validator/api/GraffitiManagerTest.java b/validator/api/src/test/java/tech/pegasys/teku/validator/api/GraffitiManagerTest.java
@@ -170,7 +170,7 @@ void getGraffitiFromStorage_shouldGetGraffitiFromStorage(@TempDir final Path tem
     Files.writeString(filePath, graffiti);
 
     final Bytes32 expectedBytes = Bytes32Parser.toBytes32(graffiti);
-    assertThat(manager.getGraffitiFromStorage(publicKey)).hasValue(expectedBytes);
+    assertThat(manager.getGraffiti(publicKey)).hasValue(expectedBytes);
   }
 
   @Test
@@ -183,7 +183,7 @@ void getGraffitiFromStorage_shouldReturnEmptyWhenFileTooBig(@TempDir final Path
     final Path filePath = getGraffitiManagementDir().resolve(getFileName(publicKey));
     Files.writeString(filePath, invalidGraffiti);
 
-    assertThat(manager.getGraffitiFromStorage(publicKey)).isEmpty();
+    assertThat(manager.getGraffiti(publicKey)).isEmpty();
   }
 
   @Test
@@ -196,7 +196,7 @@ void getGraffitiFromStorage_shouldReturnEmptyWhenNotReadableFile(@TempDir final
     Files.writeString(filePath, graffiti);
     assertThat(filePath.toFile().setReadable(false)).isTrue();
 
-    assertThat(manager.getGraffitiFromStorage(publicKey)).isEmpty();
+    assertThat(manager.getGraffiti(publicKey)).isEmpty();
   }
 
   @Test
@@ -207,7 +207,7 @@ void getGraffitiFromStorage_shouldReturnEmptyWhenFileEmpty(@TempDir final Path t
     final Path filePath = getGraffitiManagementDir().resolve(getFileName(publicKey));
     assertThat(filePath.toFile().createNewFile()).isTrue();
 
-    assertThat(manager.getGraffitiFromStorage(publicKey)).isEmpty();
+    assertThat(manager.getGraffiti(publicKey)).isEmpty();
   }
 
   private Path getGraffitiManagementDir() {

diff --git a/...dator/client/src/main/java/tech/pegasys/teku/validator/client/ValidatorClientService.java b/...dator/client/src/main/java/tech/pegasys/teku/validator/client/ValidatorClientService.java
@@ -165,7 +165,7 @@ public static ValidatorClientService create(
             config,
             asyncRunner,
             validatorApiConfig.isRestApiEnabled()
-                ? graffitiManager::getGraffitiFromStorage
+                ? graffitiManager::getGraffiti
                 : (publicKey) -> Optional.empty());
     final ValidatorStatusProvider validatorStatusProvider =
         new OwnedValidatorStatusProvider(