Skip to content
This repository was archived by the owner on Jan 23, 2023. It is now read-only.
/ corefx Public archive

Disable Unicode compression tests failing in CI #1537

Merged
merged 1 commit into from
Apr 28, 2015
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
Expand Up @@ -34,6 +34,7 @@
<Compile Include="$(CommonTestPath)\Compression\Utilities\LocalMemoryStream.cs" />
<Compile Include="$(CommonTestPath)\Compression\Utilities\StreamHelpers.cs" />
<Compile Include="$(CommonTestPath)\Compression\Utilities\ZipTestHelper.cs" />
<Compile Include="$(CommonPath)\Interop\Interop.PlatformDetection.cs" />
</ItemGroup>
<Import Project="$([MSBuild]::GetDirectoryNameOfFileAbove($(MSBuildThisFileDirectory), dir.targets))\dir.targets" />
<!-- Temporary until we have new work in build tools to
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,10 @@ public partial class ZipTest
public static async Task CreateFromDirectoryNormal()
{
await TestCreateDirectory(zfolder("normal"), true);
await TestCreateDirectory(zfolder("unicode"), true);
if (Interop.IsWindows) // [ActiveIssue(846, PlatformID.Linux | PlatformID.OSX)]
{
await TestCreateDirectory(zfolder("unicode"), true);
}
}

private static async Task TestCreateDirectory(String folderName, Boolean testWithBaseDir)
Expand Down Expand Up @@ -62,7 +65,10 @@ private static void SameExceptForBaseDir(String zipNoBaseDir, String zipBaseDir,
public static void ExtractToDirectoryNormal()
{
TestExtract(zfile("normal.zip"), zfolder("normal"));
TestExtract(zfile("unicode.zip"), zfolder("unicode"));
if (Interop.IsWindows) // [ActiveIssue(846, PlatformID.Linux | PlatformID.OSX)]
{
TestExtract(zfile("unicode.zip"), zfolder("unicode"));
}
TestExtract(zfile("empty.zip"), zfolder("empty"));
TestExtract(zfile("explicitdir1.zip"), zfolder("explicitdir"));
TestExtract(zfile("explicitdir2.zip"), zfolder("explicitdir"));
Expand Down Expand Up @@ -137,12 +143,15 @@ public static void ExtractToDirectoryTest()
DirsEqual(tempFolder, zfolder("normal"));
}

using (ZipArchive archive = ZipFile.OpenRead(zfile("unicode.zip")))
if (Interop.IsWindows) // [ActiveIssue(846, PlatformID.Linux | PlatformID.OSX)]
{
String tempFolder = StreamHelpers.GetTmpPath(false);
archive.ExtractToDirectory(tempFolder);
using (ZipArchive archive = ZipFile.OpenRead(zfile("unicode.zip")))
{
String tempFolder = StreamHelpers.GetTmpPath(false);
archive.ExtractToDirectory(tempFolder);

DirsEqual(tempFolder, zfolder("unicode"));
DirsEqual(tempFolder, zfolder("unicode"));
}
}
}

Expand Down
7 changes: 5 additions & 2 deletions src/System.IO.Compression/tests/ZipArchive/zip_CreateTests.cs
Original file line number Diff line number Diff line change
Expand Up @@ -15,8 +15,11 @@ public static async Task CreateNormal()
await testCreate("small", false);
await testCreate("normal", true);
await testCreate("normal", false);
await testCreate("unicode", true);
await testCreate("unicode", false);
if (Interop.IsWindows) // [ActiveIssue(846, PlatformID.Linux | PlatformID.OSX)]
{
await testCreate("unicode", true);
await testCreate("unicode", false);
}
await testCreate("empty", true);
await testCreate("empty", false);
await testCreate("emptydir", true);
Expand Down
12 changes: 9 additions & 3 deletions src/System.IO.Compression/tests/ZipArchive/zip_ReadTests.cs
Original file line number Diff line number Diff line change
Expand Up @@ -25,8 +25,11 @@ await ZipTest.IsZipSameAsDirAsync(
ZipTest.zfile("emptydir.zip"), ZipTest.zfolder("emptydir"), ZipArchiveMode.Read);
await ZipTest.IsZipSameAsDirAsync(
ZipTest.zfile("small.zip"), ZipTest.zfolder("small"), ZipArchiveMode.Read);
await ZipTest.IsZipSameAsDirAsync(
ZipTest.zfile("unicode.zip"), ZipTest.zfolder("unicode"), ZipArchiveMode.Read);
if (Interop.IsWindows) // [ActiveIssue(846, PlatformID.Linux | PlatformID.OSX)]
{
await ZipTest.IsZipSameAsDirAsync(
ZipTest.zfile("unicode.zip"), ZipTest.zfolder("unicode"), ZipArchiveMode.Read);
}
}

[Fact]
Expand All @@ -36,7 +39,10 @@ public static async Task ReadStreaming()

await TestStreamingRead(ZipTest.zfile("normal.zip"), ZipTest.zfolder("normal"));
await TestStreamingRead(ZipTest.zfile("fake64.zip"), ZipTest.zfolder("small"));
await TestStreamingRead(ZipTest.zfile("unicode.zip"), ZipTest.zfolder("unicode"));
if (Interop.IsWindows) // [ActiveIssue(846, PlatformID.Linux | PlatformID.OSX)]
{
await TestStreamingRead(ZipTest.zfile("unicode.zip"), ZipTest.zfolder("unicode"));
}
await TestStreamingRead(ZipTest.zfile("empty.zip"), ZipTest.zfolder("empty"));
await TestStreamingRead(ZipTest.zfile("appended.zip"), ZipTest.zfolder("small"));
await TestStreamingRead(ZipTest.zfile("prepended.zip"), ZipTest.zfolder("small"));
Expand Down
12 changes: 9 additions & 3 deletions src/System.IO.Compression/tests/ZipArchive/zip_UpdateTests.cs
Original file line number Diff line number Diff line change
Expand Up @@ -15,8 +15,11 @@ public static async Task UpdateReadNormal()
await StreamHelpers.CreateTempCopyStream(ZipTest.zfile("normal.zip")), ZipTest.zfolder("normal"), ZipArchiveMode.Update, false, false);
ZipTest.IsZipSameAsDir(
await StreamHelpers.CreateTempCopyStream(ZipTest.zfile("fake64.zip")), ZipTest.zfolder("small"), ZipArchiveMode.Update, false, false);
ZipTest.IsZipSameAsDir(
await StreamHelpers.CreateTempCopyStream(ZipTest.zfile("unicode.zip")), ZipTest.zfolder("unicode"), ZipArchiveMode.Update, false, false);
if (Interop.IsWindows)
{
ZipTest.IsZipSameAsDir(
await StreamHelpers.CreateTempCopyStream(ZipTest.zfile("unicode.zip")), ZipTest.zfolder("unicode"), ZipArchiveMode.Update, false, false);
}
ZipTest.IsZipSameAsDir(
await StreamHelpers.CreateTempCopyStream(ZipTest.zfile("empty.zip")), ZipTest.zfolder("empty"), ZipArchiveMode.Update, false, false);
ZipTest.IsZipSameAsDir(
Expand Down Expand Up @@ -49,7 +52,10 @@ public static async Task UpdateCreate()
{
await testFolder("normal");
await testFolder("empty");
await testFolder("unicode");
if (Interop.IsWindows)
{
await testFolder("unicode");
}
}

public static async Task testFolder(String s)
Expand Down