From 5ad60df96aca9d84719179ba5f4941fcfc6a8c67 Mon Sep 17 00:00:00 2001 From: discostu105 Date: Tue, 23 Feb 2016 18:03:04 +0100 Subject: [PATCH] don't compile desktop framework targets on non-windows --- test/dotnet-build.Tests/BuildOutputTests.cs | 16 +++++++++++----- 1 file changed, 11 insertions(+), 5 deletions(-) diff --git a/test/dotnet-build.Tests/BuildOutputTests.cs b/test/dotnet-build.Tests/BuildOutputTests.cs index f91970125..71062f58b 100644 --- a/test/dotnet-build.Tests/BuildOutputTests.cs +++ b/test/dotnet-build.Tests/BuildOutputTests.cs @@ -3,6 +3,7 @@ using System.IO; using System.Linq; +using System.Runtime.InteropServices; using FluentAssertions; using Microsoft.DotNet.ProjectModel; using Microsoft.DotNet.Tools.Test.Utilities; @@ -130,12 +131,17 @@ namespace Microsoft.DotNet.Tools.Builder.Tests } [Theory] - [InlineData("net20", false)] - [InlineData("net40", true)] - [InlineData("net461", true)] - [InlineData("dnxcore50", true)] - public void MultipleFrameworks_ShouldHaveValidTargetFrameworkAttribute(string frameworkName, bool shouldHaveTargetFrameworkAttribute) + [InlineData("net20", false, true)] + [InlineData("net40", true, true)] + [InlineData("net461", true, true)] + [InlineData("dnxcore50", true, false)] + public void MultipleFrameworks_ShouldHaveValidTargetFrameworkAttribute(string frameworkName, bool shouldHaveTargetFrameworkAttribute, bool windowsOnly) { + if (windowsOnly && !RuntimeInformation.IsOSPlatform(OSPlatform.Windows)) + { + return; // don't run test for desktop framework on non-windows + } + var framework = NuGetFramework.Parse(frameworkName); var testInstance = TestAssetsManager.CreateTestInstance("TestLibraryWithMultipleFrameworks")