save

ledgerwatch · Sep 14, 2023 · d72b94c · d72b94c
1 parent ffd19ea
commit d72b94c
Showing 1 changed file with 12 additions and 6 deletions.
diff --git a/downloader/downloader_test.go b/downloader/downloader_test.go
@@ -40,12 +40,18 @@ func TestChangeInfoHashOfSameFile(t *testing.T) {
 	require.Equal("a.seg", tt.Name())
 
 	// allow adding files only if they are insidesnapshots dir
-	require.NoError(BuildTorrentIfNeed(ctx, "a.seg", dirs.Snap))
-	require.NoError(BuildTorrentIfNeed(ctx, "b/a.seg", dirs.Snap))
-	require.NoError(BuildTorrentIfNeed(ctx, filepath.Join(dirs.Snap, "a.seg"), dirs.Snap))
-	require.NoError(BuildTorrentIfNeed(ctx, filepath.Join(dirs.Snap, "b", "a.seg"), dirs.Snap))
+	_, err = BuildTorrentIfNeed(ctx, "a.seg", dirs.Snap)
+	require.NoError(err)
+	_, err = BuildTorrentIfNeed(ctx, "b/a.seg", dirs.Snap)
+	require.NoError(err)
+	_, err = BuildTorrentIfNeed(ctx, filepath.Join(dirs.Snap, "a.seg"), dirs.Snap)
+	require.NoError(err)
+	_, err = BuildTorrentIfNeed(ctx, filepath.Join(dirs.Snap, "b", "a.seg"), dirs.Snap)
+	require.NoError(err)
 
 	// reject escaping snapshots dir
-	require.Error(BuildTorrentIfNeed(ctx, filepath.Join(dirs.Chaindata, "b", "a.seg"), dirs.Snap))
-	require.Error(BuildTorrentIfNeed(ctx, "./../a.seg", dirs.Snap))
+	_, err = BuildTorrentIfNeed(ctx, filepath.Join(dirs.Chaindata, "b", "a.seg"), dirs.Snap)
+	require.Error(err)
+	_, err = BuildTorrentIfNeed(ctx, "./../a.seg", dirs.Snap)
+	require.Error(err)
 }