Borrowing more code from the aspire repo to try and make tests easier

Author: aaronpowell

tests/Aspire.CommunityToolkit.Hosting.Java.Tests/JavaHostingComponentTests.cs

@@ -1,14 +1,14 @@
 using Aspire.CommunityToolkit.Testing;
 using FluentAssertions;
-using Microsoft.TestUtilities;
+using Aspire.Components.Common.Tests;
 
 namespace Aspire.CommunityToolkit.Hosting.Java.Tests;
 
 #pragma warning disable CTASPIRE001
+[RequiresDocker]
 public class JavaHostingComponentTests(AspireIntegrationTestFixture<Projects.Aspire_CommunityToolkit_Hosting_Java_AppHost> fixture) : IClassFixture<AspireIntegrationTestFixture<Projects.Aspire_CommunityToolkit_Hosting_Java_AppHost>>
 {
-    [ConditionalTheory]
-    [OSSkipCondition(OperatingSystems.Windows)]
+    [Theory]
     [InlineData("containerapp")]
     [InlineData("executableapp")]
     public async Task AppResourceWillRespondWithOk(string resourceName)

tests/Aspire.CommunityToolkit.Hosting.Ollama.Tests/AppHostTests.cs

@@ -1,13 +1,13 @@
 using Aspire.CommunityToolkit.Testing;
+using Aspire.Components.Common.Tests;
 using FluentAssertions;
-using Microsoft.TestUtilities;
 
 namespace Aspire.CommunityToolkit.Hosting.Ollama.Tests;
 
+[RequiresDocker]
 public class AppHostTests(AspireIntegrationTestFixture<Projects.Aspire_CommunityToolkit_Hosting_Ollama_AppHost> fixture) : IClassFixture<AspireIntegrationTestFixture<Projects.Aspire_CommunityToolkit_Hosting_Ollama_AppHost>>
 {
-    [ConditionalTheory]
-    [OSSkipCondition(OperatingSystems.Windows)]
+    [Theory]
     [InlineData("ollama")]
     [InlineData("ollama-openwebui")]
     public async Task ResourceStartsAndRespondsOk(string resourceName)

tests/Aspire.CommunityToolkit.Testing/AspireIntegrationTest.cs

@@ -1,4 +1,5 @@
-using Aspire.Hosting;
+using Aspire.Components.Common.Tests;
+using Aspire.Hosting;
 using Aspire.Hosting.ApplicationModel;
 using Microsoft.Extensions.DependencyInjection;
 using Microsoft.Extensions.Logging;
@@ -32,7 +33,11 @@ protected override void OnBuilderCreated(DistributedApplicationBuilder applicati
         base.OnBuilderCreated(applicationBuilder);
     }
 
-    public async Task InitializeAsync() => await StartAsync();
+    public async Task InitializeAsync()
+    {
+        if (RequiresDockerAttribute.IsSupported)
+            await StartAsync();
+    }
 
     async Task IAsyncLifetime.DisposeAsync()
     {
@@ -42,7 +47,7 @@ async Task IAsyncLifetime.DisposeAsync()
         }
         catch (Exception)
         {
-            if (!RuntimeInformation.IsOSPlatform(OSPlatform.Windows) || RuntimeInformation.IsOSPlatform(OSPlatform.Windows) && !string.IsNullOrEmpty(Environment.GetEnvironmentVariable("CI")))
+            if (RequiresDockerAttribute.IsSupported)
             {
                 // GitHub Actions Windows runners don't support Linux Docker containers, which can result in a bunch of false errors, even if we try to skip the test run, so we only really want to throw
                 // if we're on a non-Windows runner or if we're on a Windows runner but not in a CI environment

tests/Aspire.CommunityToolkit.Testing/PlatformDetection.cs

@@ -0,0 +1,6 @@
+namespace Aspire.Components.Common.Tests;
+
+public static class PlatformDetection
+{
+    public static bool IsRunningOnCI => Environment.GetEnvironmentVariable("CI") is not null;
+}

tests/Aspire.CommunityToolkit.Testing/RequiresDockerAttribute.cs

@@ -0,0 +1,34 @@
+// Licensed to the .NET Foundation under one or more agreements.
+// The .NET Foundation licenses this file to you under the MIT license.
+
+// Copied from: https://github.com/dotnet/aspire/blob/d31331d6132aeb22940dcd8834344956ba811373/tests/Aspire.Components.Common.Tests/RequiresDockerAttribute.cs
+
+using Xunit.Sdk;
+
+namespace Aspire.Components.Common.Tests;
+
+[TraitDiscoverer("Aspire.Components.Common.Tests.RequiresDockerDiscoverer", "Aspire.Components.Common.Tests")]
+[AttributeUsage(AttributeTargets.Class | AttributeTargets.Method, AllowMultiple = false)]
+public class RequiresDockerAttribute : Attribute, ITraitAttribute
+{
+    // This property is `true` when docker is *expected* to be available.
+    //
+    // A hard-coded *expected* value is used here to ensure that docker
+    // dependent tests *fail* if docker is not available/usable in an environment
+    // where it is expected to be available. A run-time check would allow tests
+    // to fail silently, which is not desirable.
+    //
+    // scenarios:
+    // - Windows: assume installed only for *local* runs as docker isn't supported on CI yet
+    //                - https://github.com/dotnet/aspire/issues/4291
+    // - Linux - Local, or CI: always assume that docker is installed
+    public static bool IsSupported =>
+        !OperatingSystem.IsWindows() ||
+        !PlatformDetection.IsRunningOnCI;
+
+    public string? Reason { get; init; }
+    public RequiresDockerAttribute(string? reason = null)
+    {
+        Reason = reason;
+    }
+}

tests/Aspire.CommunityToolkit.Testing/RequiresDockerDiscoverer.cs

@@ -0,0 +1,20 @@
+// Licensed to the .NET Foundation under one or more agreements.
+// The .NET Foundation licenses this file to you under the MIT license.
+
+// Copied from: https://github.com/dotnet/aspire/blob/d31331d6132aeb22940dcd8834344956ba811373/tests/Aspire.Components.Common.Tests/RequiresDockerDiscoverer.cs
+
+using Xunit.Abstractions;
+using Xunit.Sdk;
+
+namespace Aspire.Components.Common.Tests;
+
+public class RequiresDockerDiscoverer : ITraitDiscoverer
+{
+    public IEnumerable<KeyValuePair<string, string>> GetTraits(IAttributeInfo traitAttribute)
+    {
+        if (!RequiresDockerAttribute.IsSupported)
+        {
+            yield return new KeyValuePair<string, string>("category", "failing");
+        }
+    }
+}