update deposit tree snapshot loading warning (Consensys#8568)

rolfyone · Sep 5, 2024 · 2c60a8f · 2c60a8f
1 parent 99e72b2
commit 2c60a8f
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 4 deletions.
diff --git a/beacon/pow/src/main/java/tech/pegasys/teku/beacon/pow/DepositSnapshotFileLoader.java b/beacon/pow/src/main/java/tech/pegasys/teku/beacon/pow/DepositSnapshotFileLoader.java
@@ -79,7 +79,7 @@ public LoadDepositSnapshotResult loadDepositSnapshot() {
  "Deposit tree snapshot loaded from " + sanitizedUrl + " is not a correct snapshot",
  e);
  } else {
- LOG.warn("Failed to load deposit tree snapshot from " + sanitizedUrl, e);
+ LOG.warn("Failed to load deposit tree snapshot from " + sanitizedUrl, e.getMessage());
  }
 
  if (isRequired) {
@@ -97,7 +97,9 @@ private DepositTreeSnapshot loadFromUrl(final String path) throws IOException {
  ResourceLoader.urlOrFile("application/octet-stream, application/json")
  .loadBytes(path)
  .orElseThrow(
- () -> new FileNotFoundException(String.format("File '%s' not found", path)));
+ () ->
+ new FileNotFoundException(
+ String.format("Failed to load deposit tree snapshot from %s", path)));
 
  try {
  return parseSszDepositSnapshotTreeData(snapshotData);

diff --git a/beacon/pow/src/test/java/tech/pegasys/teku/beacon/pow/DepositSnapshotFileLoaderTest.java b/beacon/pow/src/test/java/tech/pegasys/teku/beacon/pow/DepositSnapshotFileLoaderTest.java
@@ -122,7 +122,7 @@ public void shouldThrowException_whenRequiredResourceInChainFails() {
 
  assertThatThrownBy(() -> depositSnapshotLoader.loadDepositSnapshot())
  .isInstanceOf(InvalidConfigurationException.class)
- .hasMessageContaining("File '/foo/empty2' not found");
+ .hasMessageContaining("Failed to load deposit tree snapshot from /foo/empty2");
  }
 
  @Test
@@ -135,7 +135,7 @@ public void shouldThrowException_whenFirstRequiredResourceInChainFails() {
 
  assertThatThrownBy(() -> depositSnapshotLoader.loadDepositSnapshot())
  .isInstanceOf(InvalidConfigurationException.class)
- .hasMessageContaining("File '/foo/empty1' not found");
+ .hasMessageContaining("Failed to load deposit tree snapshot from /foo/empty1");
  }
 
  @Test