From 83221617166219001e4193034985ff8f1e6f38bd Mon Sep 17 00:00:00 2001 From: James Jackson-South Date: Fri, 22 Dec 2017 01:08:27 +1100 Subject: [PATCH] Use test pattern files for tests --- .../Processing/ColorMatrix/Grayscale.cs | 2 +- .../{ColorMatrix => Filters}/BlackWhiteTest.cs | 8 ++++---- .../{ColorMatrix => Filters}/BrightnessTest.cs | 6 +++--- .../ColorBlindnessTest.cs | 8 ++++---- .../{ColorMatrix => Filters}/ContrastTest.cs | 6 +++--- .../{ColorMatrix => Filters}/FilterTest.cs | 15 +++++++++------ .../{ColorMatrix => Filters}/GrayscaleTest.cs | 7 ++++--- .../{ColorMatrix => Filters}/HueTest.cs | 8 ++++---- .../{ColorMatrix => Filters}/InvertTest.cs | 6 +++--- .../{ColorMatrix => Filters}/KodachromeTest.cs | 8 ++++---- .../{ColorMatrix => Filters}/LomographTest.cs | 8 ++++---- .../{ColorMatrix => Filters}/OpacityTest.cs | 6 +++--- .../{ColorMatrix => Filters}/PolaroidTest.cs | 8 ++++---- .../{ColorMatrix => Filters}/SaturateTest.cs | 8 ++++---- .../{ColorMatrix => Filters}/SepiaTest.cs | 8 ++++---- 15 files changed, 58 insertions(+), 54 deletions(-) rename tests/ImageSharp.Tests/Processing/Processors/{ColorMatrix => Filters}/BlackWhiteTest.cs (80%) rename tests/ImageSharp.Tests/Processing/Processors/{ColorMatrix => Filters}/BrightnessTest.cs (83%) rename tests/ImageSharp.Tests/Processing/Processors/{ColorMatrix => Filters}/ColorBlindnessTest.cs (84%) rename tests/ImageSharp.Tests/Processing/Processors/{ColorMatrix => Filters}/ContrastTest.cs (83%) rename tests/ImageSharp.Tests/Processing/Processors/{ColorMatrix => Filters}/FilterTest.cs (66%) rename tests/ImageSharp.Tests/Processing/Processors/{ColorMatrix => Filters}/GrayscaleTest.cs (87%) rename tests/ImageSharp.Tests/Processing/Processors/{ColorMatrix => Filters}/HueTest.cs (80%) rename tests/ImageSharp.Tests/Processing/Processors/{ColorMatrix => Filters}/InvertTest.cs (84%) rename tests/ImageSharp.Tests/Processing/Processors/{ColorMatrix => Filters}/KodachromeTest.cs (80%) rename tests/ImageSharp.Tests/Processing/Processors/{ColorMatrix => Filters}/LomographTest.cs (80%) rename tests/ImageSharp.Tests/Processing/Processors/{ColorMatrix => Filters}/OpacityTest.cs (84%) rename tests/ImageSharp.Tests/Processing/Processors/{ColorMatrix => Filters}/PolaroidTest.cs (80%) rename tests/ImageSharp.Tests/Processing/Processors/{ColorMatrix => Filters}/SaturateTest.cs (82%) rename tests/ImageSharp.Tests/Processing/Processors/{ColorMatrix => Filters}/SepiaTest.cs (80%) diff --git a/src/ImageSharp/Processing/ColorMatrix/Grayscale.cs b/src/ImageSharp/Processing/ColorMatrix/Grayscale.cs index 4fa80a183f..ee43d5b016 100644 --- a/src/ImageSharp/Processing/ColorMatrix/Grayscale.cs +++ b/src/ImageSharp/Processing/ColorMatrix/Grayscale.cs @@ -103,7 +103,7 @@ namespace SixLabors.ImageSharp /// The . public static IImageProcessingContext Grayscale(this IImageProcessingContext source, GrayscaleMode mode, Rectangle rectangle) where TPixel : struct, IPixel - => Grayscale(source, GrayscaleMode.Bt709, 1F, rectangle); + => Grayscale(source, mode, 1F, rectangle); /// /// Applies Grayscale toning to the image using the given amount. diff --git a/tests/ImageSharp.Tests/Processing/Processors/ColorMatrix/BlackWhiteTest.cs b/tests/ImageSharp.Tests/Processing/Processors/Filters/BlackWhiteTest.cs similarity index 80% rename from tests/ImageSharp.Tests/Processing/Processors/ColorMatrix/BlackWhiteTest.cs rename to tests/ImageSharp.Tests/Processing/Processors/Filters/BlackWhiteTest.cs index c0481809af..601f30a79e 100644 --- a/tests/ImageSharp.Tests/Processing/Processors/ColorMatrix/BlackWhiteTest.cs +++ b/tests/ImageSharp.Tests/Processing/Processors/Filters/BlackWhiteTest.cs @@ -12,7 +12,7 @@ namespace SixLabors.ImageSharp.Tests.Processing.Processors.Filters public class BlackWhiteTest : FileTestBase { [Theory] - [WithFileCollection(nameof(DefaultFiles), DefaultPixelType)] + [WithTestPatternImages(100, 100, DefaultPixelType)] public void ImageShouldApplyBlackWhiteFilter(TestImageProvider provider) where TPixel : struct, IPixel { @@ -24,14 +24,14 @@ namespace SixLabors.ImageSharp.Tests.Processing.Processors.Filters } [Theory] - [WithFileCollection(nameof(DefaultFiles), DefaultPixelType)] + [WithTestPatternImages(100, 100, DefaultPixelType)] public void ImageShouldApplyBlackWhiteFilterInBox(TestImageProvider provider) where TPixel : struct, IPixel { using (Image source = provider.GetImage()) - using (var image = source.Clone()) + using (Image image = source.Clone()) { - var bounds = new Rectangle(10, 10, image.Width / 2, image.Height / 2); + var bounds = new Rectangle(image.Width / 4, image.Width / 4, image.Width / 2, image.Height / 2); image.Mutate(x => x.BlackWhite(bounds)); image.DebugSave(provider); diff --git a/tests/ImageSharp.Tests/Processing/Processors/ColorMatrix/BrightnessTest.cs b/tests/ImageSharp.Tests/Processing/Processors/Filters/BrightnessTest.cs similarity index 83% rename from tests/ImageSharp.Tests/Processing/Processors/ColorMatrix/BrightnessTest.cs rename to tests/ImageSharp.Tests/Processing/Processors/Filters/BrightnessTest.cs index 44804d0b26..b0a830b9d9 100644 --- a/tests/ImageSharp.Tests/Processing/Processors/ColorMatrix/BrightnessTest.cs +++ b/tests/ImageSharp.Tests/Processing/Processors/Filters/BrightnessTest.cs @@ -19,7 +19,7 @@ namespace SixLabors.ImageSharp.Tests.Processing.Processors.Effects }; [Theory] - [WithFileCollection(nameof(DefaultFiles), nameof(BrightnessValues), DefaultPixelType)] + [WithTestPatternImages(nameof(BrightnessValues), 100, 100, DefaultPixelType)] public void ImageShouldApplyBrightnessFilter(TestImageProvider provider, float value) where TPixel : struct, IPixel { @@ -31,14 +31,14 @@ namespace SixLabors.ImageSharp.Tests.Processing.Processors.Effects } [Theory] - [WithFileCollection(nameof(DefaultFiles), nameof(BrightnessValues), DefaultPixelType)] + [WithTestPatternImages(nameof(BrightnessValues), 100, 100, DefaultPixelType)] public void ImageShouldApplyBrightnessFilterInBox(TestImageProvider provider, float value) where TPixel : struct, IPixel { using (Image source = provider.GetImage()) using (Image image = source.Clone()) { - var bounds = new Rectangle(10, 10, image.Width / 2, image.Height / 2); + var bounds = new Rectangle(image.Width / 4, image.Width / 4, image.Width / 2, image.Height / 2); image.Mutate(x => x.Brightness(value, bounds)); image.DebugSave(provider, value); diff --git a/tests/ImageSharp.Tests/Processing/Processors/ColorMatrix/ColorBlindnessTest.cs b/tests/ImageSharp.Tests/Processing/Processors/Filters/ColorBlindnessTest.cs similarity index 84% rename from tests/ImageSharp.Tests/Processing/Processors/ColorMatrix/ColorBlindnessTest.cs rename to tests/ImageSharp.Tests/Processing/Processors/Filters/ColorBlindnessTest.cs index cbc4a2810e..2342fe932d 100644 --- a/tests/ImageSharp.Tests/Processing/Processors/ColorMatrix/ColorBlindnessTest.cs +++ b/tests/ImageSharp.Tests/Processing/Processors/Filters/ColorBlindnessTest.cs @@ -26,7 +26,7 @@ namespace SixLabors.ImageSharp.Tests.Processing.Processors.Filters }; [Theory] - [WithFileCollection(nameof(DefaultFiles), nameof(ColorBlindnessFilters), DefaultPixelType)] + [WithTestPatternImages(nameof(ColorBlindnessFilters), 100, 100, DefaultPixelType)] public void ImageShouldApplyColorBlindnessFilter(TestImageProvider provider, ColorBlindness colorBlindness) where TPixel : struct, IPixel { @@ -38,14 +38,14 @@ namespace SixLabors.ImageSharp.Tests.Processing.Processors.Filters } [Theory] - [WithFileCollection(nameof(DefaultFiles), nameof(ColorBlindnessFilters), DefaultPixelType)] + [WithTestPatternImages(nameof(ColorBlindnessFilters), 100, 100, DefaultPixelType)] public void ImageShouldApplyColorBlindnessFilterInBox(TestImageProvider provider, ColorBlindness colorBlindness) where TPixel : struct, IPixel { using (Image source = provider.GetImage()) - using (var image = source.Clone()) + using (Image image = source.Clone()) { - var bounds = new Rectangle(10, 10, image.Width / 2, image.Height / 2); + var bounds = new Rectangle(image.Width / 4, image.Width / 4, image.Width / 2, image.Height / 2); image.Mutate(x => x.ColorBlindness(colorBlindness, bounds)); image.DebugSave(provider, colorBlindness.ToString()); diff --git a/tests/ImageSharp.Tests/Processing/Processors/ColorMatrix/ContrastTest.cs b/tests/ImageSharp.Tests/Processing/Processors/Filters/ContrastTest.cs similarity index 83% rename from tests/ImageSharp.Tests/Processing/Processors/ColorMatrix/ContrastTest.cs rename to tests/ImageSharp.Tests/Processing/Processors/Filters/ContrastTest.cs index 9117ff1b9f..67b86788aa 100644 --- a/tests/ImageSharp.Tests/Processing/Processors/ColorMatrix/ContrastTest.cs +++ b/tests/ImageSharp.Tests/Processing/Processors/Filters/ContrastTest.cs @@ -19,7 +19,7 @@ namespace SixLabors.ImageSharp.Tests.Processing.Processors.Effects }; [Theory] - [WithFileCollection(nameof(DefaultFiles), nameof(ContrastValues), DefaultPixelType)] + [WithTestPatternImages(nameof(ContrastValues), 100, 100, DefaultPixelType)] public void ImageShouldApplyContrastFilter(TestImageProvider provider, float value) where TPixel : struct, IPixel { @@ -31,14 +31,14 @@ namespace SixLabors.ImageSharp.Tests.Processing.Processors.Effects } [Theory] - [WithFileCollection(nameof(DefaultFiles), nameof(ContrastValues), DefaultPixelType)] + [WithTestPatternImages(nameof(ContrastValues), 100, 100, DefaultPixelType)] public void ImageShouldApplyContrastFilterInBox(TestImageProvider provider, float value) where TPixel : struct, IPixel { using (Image source = provider.GetImage()) using (Image image = source.Clone()) { - var bounds = new Rectangle(10, 10, image.Width / 2, image.Height / 2); + var bounds = new Rectangle(image.Width / 4, image.Width / 4, image.Width / 2, image.Height / 2); image.Mutate(x => x.Contrast(value, bounds)); image.DebugSave(provider, value); diff --git a/tests/ImageSharp.Tests/Processing/Processors/ColorMatrix/FilterTest.cs b/tests/ImageSharp.Tests/Processing/Processors/Filters/FilterTest.cs similarity index 66% rename from tests/ImageSharp.Tests/Processing/Processors/ColorMatrix/FilterTest.cs rename to tests/ImageSharp.Tests/Processing/Processors/Filters/FilterTest.cs index 80447ebf8b..9053308680 100644 --- a/tests/ImageSharp.Tests/Processing/Processors/ColorMatrix/FilterTest.cs +++ b/tests/ImageSharp.Tests/Processing/Processors/Filters/FilterTest.cs @@ -14,31 +14,34 @@ namespace SixLabors.ImageSharp.Tests.Processing.Processors.Filters public class FilterTest : FileTestBase { [Theory] - [WithFileCollection(nameof(DefaultFiles), DefaultPixelType)] + [WithTestPatternImages(100, 100, DefaultPixelType)] public void ImageShouldApplyFilter(TestImageProvider provider) where TPixel : struct, IPixel { using (Image image = provider.GetImage()) { Matrix4x4 brightness = MatrixFilters.CreateBrightnessFilter(0.9F); - Matrix4x4 contrast = MatrixFilters.CreateContrastFilter(1.2F); + Matrix4x4 hue = MatrixFilters.CreateHueFilter(180F); Matrix4x4 saturation = MatrixFilters.CreateSaturateFilter(1.5F); - image.Mutate(x => x.Filter(brightness * contrast * saturation)); + image.Mutate(x => x.Filter(brightness * hue * saturation)); image.DebugSave(provider); } } [Theory] - [WithFileCollection(nameof(DefaultFiles), DefaultPixelType)] + [WithTestPatternImages(100, 100, DefaultPixelType)] public void ImageShouldApplyFilterInBox(TestImageProvider provider) where TPixel : struct, IPixel { using (Image source = provider.GetImage()) using (Image image = source.Clone()) { - var bounds = new Rectangle(10, 10, image.Width / 2, image.Height / 2); + var bounds = new Rectangle(image.Width / 4, image.Width / 4, image.Width / 2, image.Height / 2); - image.Mutate(x => x.Filter(MatrixFilters.CreateBrightnessFilter(1.2F) * MatrixFilters.CreateContrastFilter(1.2F), bounds)); + Matrix4x4 brightness = MatrixFilters.CreateBrightnessFilter(0.9F); + Matrix4x4 hue = MatrixFilters.CreateHueFilter(180F); + Matrix4x4 saturation = MatrixFilters.CreateSaturateFilter(1.5F); + image.Mutate(x => x.Filter(brightness * hue * saturation, bounds)); image.DebugSave(provider); ImageComparer.Tolerant().VerifySimilarityIgnoreRegion(source, image, bounds); diff --git a/tests/ImageSharp.Tests/Processing/Processors/ColorMatrix/GrayscaleTest.cs b/tests/ImageSharp.Tests/Processing/Processors/Filters/GrayscaleTest.cs similarity index 87% rename from tests/ImageSharp.Tests/Processing/Processors/ColorMatrix/GrayscaleTest.cs rename to tests/ImageSharp.Tests/Processing/Processors/Filters/GrayscaleTest.cs index c870659a6b..23d514f352 100644 --- a/tests/ImageSharp.Tests/Processing/Processors/ColorMatrix/GrayscaleTest.cs +++ b/tests/ImageSharp.Tests/Processing/Processors/Filters/GrayscaleTest.cs @@ -24,8 +24,8 @@ namespace SixLabors.ImageSharp.Tests.Processing.Processors.Filters /// Use test patterns over loaded images to save decode time. /// [Theory] - [WithTestPatternImages(nameof(GrayscaleModeTypes), 50, 50, DefaultPixelType)] - public void ImageShouldApplyGrayscaleFilterAll(TestImageProvider provider, GrayscaleMode value) + [WithTestPatternImages(nameof(GrayscaleModeTypes), 100, 100, DefaultPixelType)] + public void ImageShouldApplyGrayscaleFilter(TestImageProvider provider, GrayscaleMode value) where TPixel : struct, IPixel { using (Image image = provider.GetImage()) @@ -45,7 +45,7 @@ namespace SixLabors.ImageSharp.Tests.Processing.Processors.Filters } [Theory] - [WithTestPatternImages(nameof(GrayscaleModeTypes), 50, 50, DefaultPixelType)] + [WithTestPatternImages(nameof(GrayscaleModeTypes), 100, 100, DefaultPixelType)] public void ImageShouldApplyGrayscaleFilterInBox(TestImageProvider provider, GrayscaleMode value) where TPixel : struct, IPixel { @@ -53,6 +53,7 @@ namespace SixLabors.ImageSharp.Tests.Processing.Processors.Filters using (Image image = source.Clone()) { var bounds = new Rectangle(image.Width / 4, image.Height / 4, image.Width / 2, image.Height / 2); + image.Mutate(x => x.Grayscale(value, bounds)); image.DebugSave(provider, value.ToString()); diff --git a/tests/ImageSharp.Tests/Processing/Processors/ColorMatrix/HueTest.cs b/tests/ImageSharp.Tests/Processing/Processors/Filters/HueTest.cs similarity index 80% rename from tests/ImageSharp.Tests/Processing/Processors/ColorMatrix/HueTest.cs rename to tests/ImageSharp.Tests/Processing/Processors/Filters/HueTest.cs index 743d46efaa..5a34595a62 100644 --- a/tests/ImageSharp.Tests/Processing/Processors/ColorMatrix/HueTest.cs +++ b/tests/ImageSharp.Tests/Processing/Processors/Filters/HueTest.cs @@ -19,7 +19,7 @@ namespace SixLabors.ImageSharp.Tests.Processing.Processors.Filters }; [Theory] - [WithFileCollection(nameof(DefaultFiles), nameof(HueValues), DefaultPixelType)] + [WithTestPatternImages(nameof(HueValues), 100, 100, DefaultPixelType)] public void ImageShouldApplyHueFilter(TestImageProvider provider, int value) where TPixel : struct, IPixel { @@ -31,14 +31,14 @@ namespace SixLabors.ImageSharp.Tests.Processing.Processors.Filters } [Theory] - [WithFileCollection(nameof(DefaultFiles), nameof(HueValues), DefaultPixelType)] + [WithTestPatternImages(nameof(HueValues), 100, 100, DefaultPixelType)] public void ImageShouldApplyHueFilterInBox(TestImageProvider provider, int value) where TPixel : struct, IPixel { using (Image source = provider.GetImage()) - using (var image = source.Clone()) + using (Image image = source.Clone()) { - var bounds = new Rectangle(10, 10, image.Width / 2, image.Height / 2); + var bounds = new Rectangle(image.Width / 4, image.Height / 4, image.Width / 2, image.Height / 2); image.Mutate(x => x.Hue(value, bounds)); image.DebugSave(provider, value); diff --git a/tests/ImageSharp.Tests/Processing/Processors/ColorMatrix/InvertTest.cs b/tests/ImageSharp.Tests/Processing/Processors/Filters/InvertTest.cs similarity index 84% rename from tests/ImageSharp.Tests/Processing/Processors/ColorMatrix/InvertTest.cs rename to tests/ImageSharp.Tests/Processing/Processors/Filters/InvertTest.cs index 9536b36f16..2199e691fa 100644 --- a/tests/ImageSharp.Tests/Processing/Processors/ColorMatrix/InvertTest.cs +++ b/tests/ImageSharp.Tests/Processing/Processors/Filters/InvertTest.cs @@ -12,7 +12,7 @@ namespace SixLabors.ImageSharp.Tests.Processing.Processors.Effects public class InvertTest : FileTestBase { [Theory] - [WithFileCollection(nameof(DefaultFiles), DefaultPixelType)] + [WithTestPatternImages(100, 100, DefaultPixelType)] public void ImageShouldApplyInvertFilter(TestImageProvider provider) where TPixel : struct, IPixel { @@ -24,14 +24,14 @@ namespace SixLabors.ImageSharp.Tests.Processing.Processors.Effects } [Theory] - [WithFileCollection(nameof(DefaultFiles), DefaultPixelType)] + [WithTestPatternImages(100, 100, DefaultPixelType)] public void ImageShouldApplyInvertFilterInBox(TestImageProvider provider) where TPixel : struct, IPixel { using (Image source = provider.GetImage()) using (Image image = source.Clone()) { - var bounds = new Rectangle(10, 10, image.Width / 2, image.Height / 2); + var bounds = new Rectangle(image.Width / 4, image.Height / 4, image.Width / 2, image.Height / 2); image.Mutate(x => x.Invert(bounds)); image.DebugSave(provider); diff --git a/tests/ImageSharp.Tests/Processing/Processors/ColorMatrix/KodachromeTest.cs b/tests/ImageSharp.Tests/Processing/Processors/Filters/KodachromeTest.cs similarity index 80% rename from tests/ImageSharp.Tests/Processing/Processors/ColorMatrix/KodachromeTest.cs rename to tests/ImageSharp.Tests/Processing/Processors/Filters/KodachromeTest.cs index 587ff0c013..6d95baaef0 100644 --- a/tests/ImageSharp.Tests/Processing/Processors/ColorMatrix/KodachromeTest.cs +++ b/tests/ImageSharp.Tests/Processing/Processors/Filters/KodachromeTest.cs @@ -12,7 +12,7 @@ namespace SixLabors.ImageSharp.Tests.Processing.Processors.Filters public class KodachromeTest : FileTestBase { [Theory] - [WithFileCollection(nameof(DefaultFiles), DefaultPixelType)] + [WithTestPatternImages(100, 100, DefaultPixelType)] public void ImageShouldApplyKodachromeFilter(TestImageProvider provider) where TPixel : struct, IPixel { @@ -24,14 +24,14 @@ namespace SixLabors.ImageSharp.Tests.Processing.Processors.Filters } [Theory] - [WithFileCollection(nameof(DefaultFiles), DefaultPixelType)] + [WithTestPatternImages(100, 100, DefaultPixelType)] public void ImageShouldApplyKodachromeFilterInBox(TestImageProvider provider) where TPixel : struct, IPixel { using (Image source = provider.GetImage()) - using (var image = source.Clone()) + using (Image image = source.Clone()) { - var bounds = new Rectangle(10, 10, image.Width / 2, image.Height / 2); + var bounds = new Rectangle(image.Width / 4, image.Height / 4, image.Width / 2, image.Height / 2); image.Mutate(x => x.Kodachrome(bounds)); image.DebugSave(provider); diff --git a/tests/ImageSharp.Tests/Processing/Processors/ColorMatrix/LomographTest.cs b/tests/ImageSharp.Tests/Processing/Processors/Filters/LomographTest.cs similarity index 80% rename from tests/ImageSharp.Tests/Processing/Processors/ColorMatrix/LomographTest.cs rename to tests/ImageSharp.Tests/Processing/Processors/Filters/LomographTest.cs index 7087ac7b98..2f9cd4319b 100644 --- a/tests/ImageSharp.Tests/Processing/Processors/ColorMatrix/LomographTest.cs +++ b/tests/ImageSharp.Tests/Processing/Processors/Filters/LomographTest.cs @@ -12,7 +12,7 @@ namespace SixLabors.ImageSharp.Tests.Processing.Processors.Filters public class LomographTest : FileTestBase { [Theory] - [WithFileCollection(nameof(DefaultFiles), DefaultPixelType)] + [WithTestPatternImages(100, 100, DefaultPixelType)] public void ImageShouldApplyLomographFilter(TestImageProvider provider) where TPixel : struct, IPixel { @@ -24,14 +24,14 @@ namespace SixLabors.ImageSharp.Tests.Processing.Processors.Filters } [Theory] - [WithFileCollection(nameof(DefaultFiles), DefaultPixelType)] + [WithTestPatternImages(100, 100, DefaultPixelType)] public void ImageShouldApplyLomographFilterInBox(TestImageProvider provider) where TPixel : struct, IPixel { using (Image source = provider.GetImage()) - using (var image = source.Clone()) + using (Image image = source.Clone()) { - var bounds = new Rectangle(10, 10, image.Width / 2, image.Height / 2); + var bounds = new Rectangle(image.Width / 4, image.Height / 4, image.Width / 2, image.Height / 2); image.Mutate(x => x.Lomograph(bounds)); image.DebugSave(provider); diff --git a/tests/ImageSharp.Tests/Processing/Processors/ColorMatrix/OpacityTest.cs b/tests/ImageSharp.Tests/Processing/Processors/Filters/OpacityTest.cs similarity index 84% rename from tests/ImageSharp.Tests/Processing/Processors/ColorMatrix/OpacityTest.cs rename to tests/ImageSharp.Tests/Processing/Processors/Filters/OpacityTest.cs index 2815233f24..12bf93299a 100644 --- a/tests/ImageSharp.Tests/Processing/Processors/ColorMatrix/OpacityTest.cs +++ b/tests/ImageSharp.Tests/Processing/Processors/Filters/OpacityTest.cs @@ -19,7 +19,7 @@ namespace SixLabors.ImageSharp.Tests.Processing.Processors.Effects }; [Theory] - [WithFileCollection(nameof(DefaultFiles), nameof(AlphaValues), DefaultPixelType)] + [WithTestPatternImages(nameof(AlphaValues), 100, 100, DefaultPixelType)] public void ImageShouldApplyAlphaFilter(TestImageProvider provider, float value) where TPixel : struct, IPixel { @@ -31,14 +31,14 @@ namespace SixLabors.ImageSharp.Tests.Processing.Processors.Effects } [Theory] - [WithFileCollection(nameof(DefaultFiles), nameof(AlphaValues), DefaultPixelType)] + [WithTestPatternImages(nameof(AlphaValues), 100, 100, DefaultPixelType)] public void ImageShouldApplyAlphaFilterInBox(TestImageProvider provider, float value) where TPixel : struct, IPixel { using (Image source = provider.GetImage()) using (Image image = source.Clone()) { - var bounds = new Rectangle(10, 10, image.Width / 2, image.Height / 2); + var bounds = new Rectangle(image.Width / 4, image.Height / 4, image.Width / 2, image.Height / 2); image.Mutate(x => x.Opacity(value, bounds)); image.DebugSave(provider, value); diff --git a/tests/ImageSharp.Tests/Processing/Processors/ColorMatrix/PolaroidTest.cs b/tests/ImageSharp.Tests/Processing/Processors/Filters/PolaroidTest.cs similarity index 80% rename from tests/ImageSharp.Tests/Processing/Processors/ColorMatrix/PolaroidTest.cs rename to tests/ImageSharp.Tests/Processing/Processors/Filters/PolaroidTest.cs index 57d6cdd1df..44e69c09ee 100644 --- a/tests/ImageSharp.Tests/Processing/Processors/ColorMatrix/PolaroidTest.cs +++ b/tests/ImageSharp.Tests/Processing/Processors/Filters/PolaroidTest.cs @@ -12,7 +12,7 @@ namespace SixLabors.ImageSharp.Tests.Processing.Processors.Filters public class PolaroidTest : FileTestBase { [Theory] - [WithFileCollection(nameof(DefaultFiles), DefaultPixelType)] + [WithTestPatternImages(100, 100, DefaultPixelType)] public void ImageShouldApplyPolaroidFilter(TestImageProvider provider) where TPixel : struct, IPixel { @@ -24,14 +24,14 @@ namespace SixLabors.ImageSharp.Tests.Processing.Processors.Filters } [Theory] - [WithFileCollection(nameof(DefaultFiles), DefaultPixelType)] + [WithTestPatternImages(100, 100, DefaultPixelType)] public void ImageShouldApplyPolaroidFilterInBox(TestImageProvider provider) where TPixel : struct, IPixel { using (Image source = provider.GetImage()) - using (var image = source.Clone()) + using (Image image = source.Clone()) { - var bounds = new Rectangle(10, 10, image.Width / 2, image.Height / 2); + var bounds = new Rectangle(image.Width / 4, image.Height / 4, image.Width / 2, image.Height / 2); image.Mutate(x => x.Polaroid(bounds)); image.DebugSave(provider); diff --git a/tests/ImageSharp.Tests/Processing/Processors/ColorMatrix/SaturateTest.cs b/tests/ImageSharp.Tests/Processing/Processors/Filters/SaturateTest.cs similarity index 82% rename from tests/ImageSharp.Tests/Processing/Processors/ColorMatrix/SaturateTest.cs rename to tests/ImageSharp.Tests/Processing/Processors/Filters/SaturateTest.cs index b9e2c3f0f7..8553cad4c2 100644 --- a/tests/ImageSharp.Tests/Processing/Processors/ColorMatrix/SaturateTest.cs +++ b/tests/ImageSharp.Tests/Processing/Processors/Filters/SaturateTest.cs @@ -14,12 +14,12 @@ namespace SixLabors.ImageSharp.Tests.Processing.Processors.Filters public static readonly TheoryData SaturationValues = new TheoryData { - .5f, + .5F, 1.5F, }; [Theory] - [WithFileCollection(nameof(DefaultFiles), nameof(SaturationValues), DefaultPixelType)] + [WithTestPatternImages(nameof(SaturationValues), 100, 100, DefaultPixelType)] public void ImageShouldApplySaturationFilter(TestImageProvider provider, float value) where TPixel : struct, IPixel { @@ -31,14 +31,14 @@ namespace SixLabors.ImageSharp.Tests.Processing.Processors.Filters } [Theory] - [WithFileCollection(nameof(DefaultFiles), nameof(SaturationValues), DefaultPixelType)] + [WithTestPatternImages(nameof(SaturationValues), 100, 100, DefaultPixelType)] public void ImageShouldApplySaturationFilterInBox(TestImageProvider provider, float value) where TPixel : struct, IPixel { using (Image source = provider.GetImage()) using (Image image = source.Clone()) { - var bounds = new Rectangle(10, 10, image.Width / 2, image.Height / 2); + var bounds = new Rectangle(image.Width / 4, image.Height / 4, image.Width / 2, image.Height / 2); image.Mutate(x => x.Saturate(value, bounds)); image.DebugSave(provider, value); diff --git a/tests/ImageSharp.Tests/Processing/Processors/ColorMatrix/SepiaTest.cs b/tests/ImageSharp.Tests/Processing/Processors/Filters/SepiaTest.cs similarity index 80% rename from tests/ImageSharp.Tests/Processing/Processors/ColorMatrix/SepiaTest.cs rename to tests/ImageSharp.Tests/Processing/Processors/Filters/SepiaTest.cs index 71a35a773d..499567ae88 100644 --- a/tests/ImageSharp.Tests/Processing/Processors/ColorMatrix/SepiaTest.cs +++ b/tests/ImageSharp.Tests/Processing/Processors/Filters/SepiaTest.cs @@ -12,7 +12,7 @@ namespace SixLabors.ImageSharp.Tests.Processing.Processors.Filters public class SepiaTest : FileTestBase { [Theory] - [WithFileCollection(nameof(DefaultFiles), DefaultPixelType)] + [WithTestPatternImages(100, 100, DefaultPixelType)] public void ImageShouldApplySepiaFilter(TestImageProvider provider) where TPixel : struct, IPixel { @@ -24,14 +24,14 @@ namespace SixLabors.ImageSharp.Tests.Processing.Processors.Filters } [Theory] - [WithFileCollection(nameof(DefaultFiles), DefaultPixelType)] + [WithTestPatternImages(100, 100, DefaultPixelType)] public void ImageShouldApplySepiaFilterInBox(TestImageProvider provider) where TPixel : struct, IPixel { using (Image source = provider.GetImage()) - using (var image = source.Clone()) + using (Image image = source.Clone()) { - var bounds = new Rectangle(10, 10, image.Width / 2, image.Height / 2); + var bounds = new Rectangle(image.Width / 4, image.Height / 4, image.Width / 2, image.Height / 2); image.Mutate(x => x.Sepia(bounds)); image.DebugSave(provider);