Unit test "Indentify_Windows_Platform" fails on Linux about mailmergelib

[axunonb]

Did you mean to make the Indentify_Windows_Platform test conditional - fails on linux.

Actually the test should work also without ExcludePlatform. The intention is to in include the new Platform class in unit tests.

[axunonb]

@thegruffalo Could you elaborate what exactly fails with the test, please?

[thegruffalo]

Here is the test output:

---- Test Execution Summary -----

MailMergeLib.Tests.Platform.Indentify_Windows_Platform:
Outcome: Failed
Error Message:
MailMergeLib.UnsupportedPlatformException : Unknown
Stack Trace:
at MailMergeLib.Platform.DeterminePlatform() in /home/thill/tdev/MMLO/MailMergeLib/MailMergeLib/Platform.cs:line 65
at MailMergeLib.Tests.Platform.Indentify_Windows_Platform() in /home/thill/tdev/MMLO/MailMergeLib/MailMergeLib.Tests/Platform.cs:line 24

Total tests: 1. Passed: 0. Failed: 1. Skipped: 0

As an alternative, maybe, something like this would be easier to implement https://docs.microsoft.com/en-us/dotnet/api/system.runtime.interopservices.runtimeinformation.isosplatform?view=netcore-2.2

[axunonb]

With commit 1b08f36 we're using System.Runtime.InteropServices.OSPlatform to detect the platform. Does the test pass now on Linux?

[axunonb]

Tests for commit 1b08f36 complete successfully on Ubuntu 18.04 LTS and Windows.