diff --git a/tests/SharpCompress.Test/LargeArchiveTests.cs b/tests/SharpCompress.Test/LargeArchiveTests.cs index fc94c935..761abda6 100644 --- a/tests/SharpCompress.Test/LargeArchiveTests.cs +++ b/tests/SharpCompress.Test/LargeArchiveTests.cs @@ -26,7 +26,7 @@ public class LargeArchiveTests : TestBase [InlineData("Large/Large.7z")] public void OpenArchive_ShouldStreamLargeEntry(string fixtureName) { - using var stream = File.OpenRead(GetArchiveFixturePath(fixtureName)); + using var stream = File.OpenRead(GetMaterializedFixturePath(fixtureName)); using var archive = ArchiveFactory.OpenArchive(stream); VerifyArchive(archive); @@ -41,7 +41,7 @@ public class LargeArchiveTests : TestBase public async Task OpenAsyncArchive_ShouldStreamLargeEntry(string fixtureName) { await using var stream = new AsyncOnlyStream( - File.OpenRead(await GetArchiveFixturePathAsync(fixtureName)) + File.OpenRead(await GetMaterializedFixturePathAsync(fixtureName)) ); await using var archive = await ArchiveFactory.OpenAsyncArchive(stream); @@ -58,7 +58,7 @@ public class LargeArchiveTests : TestBase [InlineData("Large/Large.tar.gz")] public void OpenReader_ShouldStreamLargeEntry(string fixtureName) { - using var stream = File.OpenRead(GetFixturePath(fixtureName)); + using var stream = File.OpenRead(GetMaterializedFixturePath(fixtureName)); using var reader = ReaderFactory.OpenReader(stream); VerifyReader(reader); @@ -72,7 +72,9 @@ public class LargeArchiveTests : TestBase [InlineData("Large/Large.tar.gz")] public async Task OpenAsyncReader_ShouldStreamLargeEntry(string fixtureName) { - await using var stream = new AsyncOnlyStream(File.OpenRead(GetFixturePath(fixtureName))); + await using var stream = new AsyncOnlyStream( + File.OpenRead(await GetMaterializedFixturePathAsync(fixtureName)) + ); await using var reader = await ReaderFactory.OpenAsyncReader(stream); Assert.True(await reader.MoveToNextEntryAsync()); @@ -85,10 +87,10 @@ public class LargeArchiveTests : TestBase private static string GetFixturePath(string fixtureName) => Path.Combine(TEST_ARCHIVES_PATH, fixtureName); - private string GetArchiveFixturePath(string fixtureName) => + private string GetMaterializedFixturePath(string fixtureName) => fixtureName == "Large/Large.tar" ? MaterializeTarFixture() : GetFixturePath(fixtureName); - private async Task GetArchiveFixturePathAsync(string fixtureName) => + private async Task GetMaterializedFixturePathAsync(string fixtureName) => fixtureName == "Large/Large.tar" ? await MaterializeTarFixtureAsync() : GetFixturePath(fixtureName); diff --git a/tests/TestArchives/Archives/Large/README.md b/tests/TestArchives/Archives/Large/README.md index 4992f31b..0797db5d 100644 --- a/tests/TestArchives/Archives/Large/README.md +++ b/tests/TestArchives/Archives/Large/README.md @@ -7,16 +7,15 @@ deterministic, repeated text pattern with a 2 KiB `0xFF` prefix. The entry CRC-3 | Fixture | Archive API | Reader API | | --- | --- | --- | | `Large.zip` | Yes | Yes | -| Generated `Large.tar` | Yes | N/A | +| Generated `Large.tar` | Yes | Yes | | `Large.gz` | Yes | Yes | | `Large.rar` | Yes | Yes | | `Large.7z` | Yes | No | | `Large.tar.gz` | No | Yes | The compressible payload keeps the compressed fixtures small while requiring a full -64 MiB decompression to validate each API. The Archive API test expands `Large.tar.gz` -to a scratch `Large.tar` before exercising raw TAR support, so the 64 MiB TAR file is -not committed. +64 MiB decompression to validate each API. The tests expand `Large.tar.gz` to a scratch +`Large.tar` before exercising raw TAR support, so the 64 MiB TAR file is not committed. ## Regenerating