Add more integration tests and remove inaccurate benchmarks (#29)

* more integration tests * Remove benchmarks as they were inaccurate
JonathanBout · Dec 7, 2024 · 629dc70 · 629dc70
1 parent 8acf52d
commit 629dc70
Show file tree

Hide file tree

Showing 7 changed files with 24 additions and 81 deletions.
diff --git a/SimpleCDN.Benchmarks/Benchmarks.cs b/SimpleCDN.Benchmarks/Benchmarks.cs
diff --git a/SimpleCDN.Benchmarks/GlobalSuppressions.cs b/SimpleCDN.Benchmarks/GlobalSuppressions.cs
diff --git a/SimpleCDN.Benchmarks/Program.cs b/SimpleCDN.Benchmarks/Program.cs
diff --git a/SimpleCDN.Benchmarks/Properties/launchSettings.json b/SimpleCDN.Benchmarks/Properties/launchSettings.json
diff --git a/SimpleCDN.Benchmarks/SimpleCDN.Benchmarks.csproj b/SimpleCDN.Benchmarks/SimpleCDN.Benchmarks.csproj
diff --git a/SimpleCDN.Tests.Integration/EndpointTests.cs b/SimpleCDN.Tests.Integration/EndpointTests.cs
@@ -1,4 +1,7 @@
-namespace SimpleCDN.Tests.Integration
+using System.Net;
+using System.Net.Http.Headers;
+
+namespace SimpleCDN.Tests.Integration
 {
 	public class EndpointTests : IClassFixture<CustomWebApplicationFactory>
 	{
@@ -58,5 +61,25 @@ public async Task Test_FileExists_WithContent(string endpoint, string expectedTe
 			var content = await response.Content.ReadAsStringAsync();
 			Assert.Contains(expectedText, content);
 		}
+
+		[Theory]
+		[InlineData("/test.txt", "text/plain", false)]
+		[InlineData("/test.txt", "application/json",true)]
+		[InlineData("/data/test.json", "application/json", false)]
+		[InlineData("/data/test.json", "text/plain", true)]
+		public async Task Test_UnsupportedMediaType_WhenWrongAcceptHeader(string endpoint, string supportedMediaType, bool shouldFail)
+		{
+			var client = _webApplicationFactory.CreateClient();
+			var request = new HttpRequestMessage(HttpMethod.Get, endpoint);
+			request.Headers.Accept.Add(new MediaTypeWithQualityHeaderValue(supportedMediaType));
+			var response = await client.SendAsync(request);
+			if (shouldFail)
+			{
+				Assert.Equal(HttpStatusCode.NotAcceptable, response.StatusCode);
+			} else
+			{
+				Assert.True(response.IsSuccessStatusCode, response.StatusCode.ToString());
+			}
+		}
 	}
 }
diff --git a/SimpleCDN.sln b/SimpleCDN.sln
@@ -8,8 +8,6 @@ Project("{E53339B2-1760-4266-BCC7-CA923CBCF16C}") = "docker-compose", "docker-co
 EndProject
 Project("{FAE04EC0-301F-11D3-BF4B-00C04F79EFBC}") = "SimpleCDN.Tests", "SimpleCDN.Tests\SimpleCDN.Tests.csproj", "{9F41AC4B-73A5-AED7-2FAD-13532040B2E1}"
 EndProject
-Project("{FAE04EC0-301F-11D3-BF4B-00C04F79EFBC}") = "SimpleCDN.Benchmarks", "SimpleCDN.Benchmarks\SimpleCDN.Benchmarks.csproj", "{01B35836-CD1D-0631-A280-04996E3E2058}"
-EndProject
 Project("{FAE04EC0-301F-11D3-BF4B-00C04F79EFBC}") = "SimpleCDN.Tests.Integration", "SimpleCDN.Tests.Integration\SimpleCDN.Tests.Integration.csproj", "{748DF05A-61E9-3214-6751-2FB8D9C75A1C}"
 EndProject
 Global
@@ -30,10 +28,6 @@ Global
 		{9F41AC4B-73A5-AED7-2FAD-13532040B2E1}.Debug|Any CPU.Build.0 = Debug|Any CPU
 		{9F41AC4B-73A5-AED7-2FAD-13532040B2E1}.Release|Any CPU.ActiveCfg = Release|Any CPU
 		{9F41AC4B-73A5-AED7-2FAD-13532040B2E1}.Release|Any CPU.Build.0 = Release|Any CPU
-		{01B35836-CD1D-0631-A280-04996E3E2058}.Debug|Any CPU.ActiveCfg = Debug|Any CPU
-		{01B35836-CD1D-0631-A280-04996E3E2058}.Debug|Any CPU.Build.0 = Debug|Any CPU
-		{01B35836-CD1D-0631-A280-04996E3E2058}.Release|Any CPU.ActiveCfg = Release|Any CPU
-		{01B35836-CD1D-0631-A280-04996E3E2058}.Release|Any CPU.Build.0 = Release|Any CPU
 		{748DF05A-61E9-3214-6751-2FB8D9C75A1C}.Debug|Any CPU.ActiveCfg = Debug|Any CPU
 		{748DF05A-61E9-3214-6751-2FB8D9C75A1C}.Debug|Any CPU.Build.0 = Debug|Any CPU
 		{748DF05A-61E9-3214-6751-2FB8D9C75A1C}.Release|Any CPU.ActiveCfg = Release|Any CPU