Skip to content

Commit aed866e

Browse files
Remove bad test server implementation
1 parent 661b721 commit aed866e

File tree

5 files changed

+37
-562
lines changed

5 files changed

+37
-562
lines changed

tests/ImageSharp.Web.Tests/Commands/QueryCollectionUriParserTests.cs

Lines changed: 11 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
// Copyright (c) Six Labors.
1+
// Copyright (c) Six Labors.
22
// Licensed under the Apache License, Version 2.0.
33

44
using System;
@@ -20,10 +20,18 @@ public void QueryCollectionParserExtractsCommands()
2020
{ "height", "200" }
2121
};
2222

23-
HttpContext context = TestHelpers.CreateHttpContext();
23+
HttpContext context = CreateHttpContext();
2424
IDictionary<string, string> actual = new QueryCollectionRequestParser().ParseRequestCommands(context);
2525

2626
Assert.Equal(expected, actual);
2727
}
28+
29+
private static HttpContext CreateHttpContext()
30+
{
31+
var httpContext = new DefaultHttpContext();
32+
httpContext.Request.Path = "/testwebsite.com/image-12345.jpeg";
33+
httpContext.Request.QueryString = new QueryString("?width=400&height=200");
34+
return httpContext;
35+
}
2836
}
29-
}
37+
}
Lines changed: 26 additions & 47 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,6 @@
11
// Copyright (c) Six Labors.
22
// Licensed under the Apache License, Version 2.0.
33

4-
using Microsoft.AspNetCore.TestHost;
54
using Microsoft.Extensions.DependencyInjection;
65
using SixLabors.ImageSharp.Web.DependencyInjection;
76
using Xunit;
@@ -13,85 +12,65 @@ public class ServiceRegistrationExtensionsTests
1312
[Fact]
1413
public void CanAddRemoveImageProviders()
1514
{
16-
void RemoveServices(IServiceCollection services)
17-
{
18-
IImageSharpBuilder builder = services.AddImageSharp()
19-
.AddProvider<MockImageProvider>();
15+
var services = new ServiceCollection();
2016

21-
Assert.Contains(services, x => x.ImplementationType == typeof(MockImageProvider));
17+
IImageSharpBuilder builder = services.AddImageSharp()
18+
.AddProvider<MockImageProvider>();
2219

23-
builder.RemoveProvider<MockImageProvider>();
20+
Assert.Contains(services, x => x.ImplementationType == typeof(MockImageProvider));
2421

25-
Assert.DoesNotContain(services, x => x.ImplementationType == typeof(MockImageProvider));
26-
}
22+
builder.RemoveProvider<MockImageProvider>();
2723

28-
using (TestServer server = ImageSharpTestServer.CreateTestServer(ImageSharpTestServer.DefaultConfig, RemoveServices))
29-
{
30-
}
24+
Assert.DoesNotContain(services, x => x.ImplementationType == typeof(MockImageProvider));
3125
}
3226

3327
[Fact]
3428
public void CanAddRemoveFactoryImageProviders()
3529
{
36-
void RemoveServices(IServiceCollection services)
37-
{
38-
IImageSharpBuilder builder = services.AddImageSharp()
39-
.AddProvider(_ => new MockImageProvider());
30+
var services = new ServiceCollection();
4031

41-
Assert.DoesNotContain(services, x => x.ImplementationType == typeof(MockImageProvider));
32+
IImageSharpBuilder builder = services.AddImageSharp()
33+
.AddProvider(_ => new MockImageProvider());
4234

43-
Assert.Contains(services, x => x.ImplementationFactory?.Method.ReturnType == typeof(MockImageProvider));
35+
Assert.DoesNotContain(services, x => x.ImplementationType == typeof(MockImageProvider));
4436

45-
builder.RemoveProvider<MockImageProvider>();
37+
Assert.Contains(services, x => x.ImplementationFactory?.Method.ReturnType == typeof(MockImageProvider));
4638

47-
Assert.DoesNotContain(services, x => x.ImplementationFactory?.Method.ReturnType == typeof(MockImageProvider));
48-
}
39+
builder.RemoveProvider<MockImageProvider>();
4940

50-
using (TestServer server = ImageSharpTestServer.CreateTestServer(ImageSharpTestServer.DefaultConfig, RemoveServices))
51-
{
52-
}
41+
Assert.DoesNotContain(services, x => x.ImplementationFactory?.Method.ReturnType == typeof(MockImageProvider));
5342
}
5443

5544
[Fact]
5645
public void CanAddRemoveImageProcessors()
5746
{
58-
void RemoveServices(IServiceCollection services)
59-
{
60-
IImageSharpBuilder builder = services.AddImageSharp()
61-
.AddProcessor<MockWebProcessor>();
47+
var services = new ServiceCollection();
6248

63-
Assert.Contains(services, x => x.ImplementationType == typeof(MockWebProcessor));
49+
IImageSharpBuilder builder = services.AddImageSharp()
50+
.AddProcessor<MockWebProcessor>();
6451

65-
builder.RemoveProcessor<MockWebProcessor>();
52+
Assert.Contains(services, x => x.ImplementationType == typeof(MockWebProcessor));
6653

67-
Assert.DoesNotContain(services, x => x.ImplementationType == typeof(MockWebProcessor));
68-
}
54+
builder.RemoveProcessor<MockWebProcessor>();
6955

70-
using (TestServer server = ImageSharpTestServer.CreateTestServer(ImageSharpTestServer.DefaultConfig, RemoveServices))
71-
{
72-
}
56+
Assert.DoesNotContain(services, x => x.ImplementationType == typeof(MockWebProcessor));
7357
}
7458

7559
[Fact]
7660
public void CanAddRemoveFactoryImageProcessors()
7761
{
78-
void RemoveServices(IServiceCollection services)
79-
{
80-
IImageSharpBuilder builder = services.AddImageSharp()
81-
.AddProcessor(_ => new MockWebProcessor());
62+
var services = new ServiceCollection();
8263

83-
Assert.DoesNotContain(services, x => x.ImplementationType == typeof(MockWebProcessor));
64+
IImageSharpBuilder builder = services.AddImageSharp()
65+
.AddProcessor(_ => new MockWebProcessor());
8466

85-
Assert.Contains(services, x => x.ImplementationFactory?.Method.ReturnType == typeof(MockWebProcessor));
67+
Assert.DoesNotContain(services, x => x.ImplementationType == typeof(MockWebProcessor));
8668

87-
builder.RemoveProcessor<MockWebProcessor>();
69+
Assert.Contains(services, x => x.ImplementationFactory?.Method.ReturnType == typeof(MockWebProcessor));
8870

89-
Assert.DoesNotContain(services, x => x.ImplementationFactory?.Method.ReturnType == typeof(MockWebProcessor));
90-
}
71+
builder.RemoveProcessor<MockWebProcessor>();
9172

92-
using (TestServer server = ImageSharpTestServer.CreateTestServer(ImageSharpTestServer.DefaultConfig, RemoveServices))
93-
{
94-
}
73+
Assert.DoesNotContain(services, x => x.ImplementationFactory?.Method.ReturnType == typeof(MockWebProcessor));
9574
}
9675
}
9776
}

tests/ImageSharp.Web.Tests/ImageSharpTestServer.cs

Lines changed: 0 additions & 234 deletions
This file was deleted.

0 commit comments

Comments
 (0)