Skip to content

Commit

Permalink
Merge pull request #119 from maythamfahmi/jwd/issue107
Browse files Browse the repository at this point in the history
Fix: Workaround the missing example executables on Linux and MacOS, f…
  • Loading branch information
jwdonahue authored Oct 21, 2024
2 parents c5a2435 + d6b3d36 commit 4b1cdbb
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 1 deletion.
4 changes: 3 additions & 1 deletion Examples/Examples.UnitTests/AESExampleTests.cs
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,9 @@ public class AESExampleTests : ExampleTestBase
[Test]
public async Task AESExampleSmokeTest()
{
// ToDo: Missing executables on Linux and MacOS.
if (Environment.OSVersion.Platform != PlatformID.Win32NT) return;

// This provides a human readable temporary directory name prefix.
// If you see a lot of these laying around your temp directory, it's
// probably due to some failures in this test.
Expand All @@ -27,7 +30,6 @@ public async Task AESExampleSmokeTest()
ClassicAssert.IsTrue(Directory.Exists(tmpDir.DirectoryInfo.FullName));
ClassicAssert.IsTrue(File.Exists(AESExampleExeName));


ShowAvailableExecutables();

var result = await Cli.Wrap(AESExampleExeName)
Expand Down
3 changes: 3 additions & 0 deletions Examples/Examples.UnitTests/RSAExampleTests.cs
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,9 @@ public class RSAExampleTests : ExampleTestBase
[Test]
public async Task RSAExampleSmokeTest()
{
// ToDo: Missing executables on Linux and MacOS.
if (Environment.OSVersion.Platform != PlatformID.Win32NT) return;

var tmpDirPrefix = $"{nameof(AESExampleTests)}.{nameof(RSAExampleSmokeTest)}-{Guid.NewGuid().ToString("D")}";

var stdOutBuffer = new StringBuilder();
Expand Down

0 comments on commit 4b1cdbb

Please sign in to comment.