test/wip: BdvMetadataIoTests clearer assertions

saalfeldlab · Oct 4, 2024 · 989ec72 · 989ec72
1 parent 2bd2533
commit 989ec72
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/src/test/java/org/janelia/saalfeldlab/n5/bdv/BdvMetadataIoTests.java b/src/test/java/org/janelia/saalfeldlab/n5/bdv/BdvMetadataIoTests.java
@@ -99,11 +99,11 @@ public <T extends NumericType<T> & NativeType<T>, V extends Volatile<T> & Numeri
 					Arrays.asList(N5ViewerCreator.n5vGroupParsers));
 
 			final N5TreeNode root = datasetDiscoverer.discoverAndParseRecursive("");
-			assertNotNull("root is null", root);
+			assertNotNull("Root node is null", root);
 
 			final Optional<N5TreeNode> metaOpt = root.getDescendant(readerDataset);
-			if (!metaOpt.isPresent())
-				fail("could not find metadata at: " + readerDataset);
+			assertTrue( String.format( "[%s] Could not find descendant node: %s ", outputPath, readerDataset ),
+					metaOpt.isPresent());
 
 			final List<ConverterSetup> converterSetups = new ArrayList<>();
 			final List<SourceAndConverter<T>> sourcesAndConverters = new ArrayList<>();
@@ -148,6 +148,8 @@ public <T extends NumericType<T> & NativeType<T>, V extends Volatile<T> & Numeri
 				final List<Source<T>> srcList = sourcesAndConverters.stream().map(sac -> sac.getSpimSource()).collect(Collectors.toList());
 				assertTrue(String.format("%s data ", dataset), sourceDataIdentical(imp, srcList));
 			}
+
+			n5.close();
 		}
 	}