Combining tests to see if this makes them more consistent

CommunityToolkit · Sep 27, 2024 · 4c4c648 · 4c4c648
1 parent 1e9461d
commit 4c4c648
Showing 1 changed file with 4 additions and 17 deletions.
diff --git a/tests/Aspire.CommunityToolkit.Hosting.Java.Tests/JavaHostingComponentTests.cs b/tests/Aspire.CommunityToolkit.Hosting.Java.Tests/JavaHostingComponentTests.cs
@@ -7,25 +7,12 @@ namespace Aspire.CommunityToolkit.Hosting.Java.Tests;
 #pragma warning disable CTASPIRE001
 public class JavaHostingComponentTests(AspireIntegrationTestFixture<Projects.Aspire_CommunityToolkit_Hosting_Java_AppHost> fixture) : IClassFixture<AspireIntegrationTestFixture<Projects.Aspire_CommunityToolkit_Hosting_Java_AppHost>>
 {
-    [ConditionalFact]
+    [ConditionalTheory]
     [OSSkipCondition(OperatingSystems.Windows)]
-    public async Task ContainerAppResourceWillRespondWithOk()
+    [InlineData("containerapp")]
+    [InlineData("executableapp")]
+    public async Task AppResourceWillRespondWithOk(string resourceName)
     {
-        var resourceName = "containerapp";
-        var httpClient = fixture.CreateHttpClient(resourceName);
-
-        await fixture.App.WaitForTextAsync("Started SpringMavenApplication", resourceName).WaitAsync(TimeSpan.FromMinutes(5));
-
-        var response = await httpClient.GetAsync("/");
-
-        response.StatusCode.Should().Be(HttpStatusCode.OK);
-    }
-
-    [ConditionalFact]
-    [OSSkipCondition(OperatingSystems.Windows)]
-    public async Task ExecutableAppResourceWillRespondWithOk()
-    {
-        var resourceName = "executableapp";
         var httpClient = fixture.CreateHttpClient(resourceName);
 
         await fixture.App.WaitForTextAsync("Started SpringMavenApplication", resourceName).WaitAsync(TimeSpan.FromMinutes(5));