Merge branch 'master' into feature/Bitmap8BitEncoding

src/ImageSharp/Common/Helpers/Vector4Utils.cs

@@ -92,7 +92,7 @@ namespace SixLabors.ImageSharp
         }
 
         /// <summary>
-        /// Bulk variant of <see cref="Transform(ref Vector4, ref ColorMatrix)"/>
+        /// Bulk variant of <see cref="Transform(ref Vector4, ref ColorMatrix)"/>.
         /// </summary>
         /// <param name="vectors">The span of vectors</param>
         /// <param name="matrix">The transformation matrix.</param>

src/ImageSharp/PixelFormats/PixelImplementations/Byte4.cs

@@ -149,7 +149,7 @@ namespace SixLabors.ImageSharp.PixelFormats
         public override string ToString()
         {
             var vector = this.ToVector4();
-            return FormattableString.Invariant($"Bgra5551({vector.X:#0.##}, {vector.Y:#0.##}, {vector.Z:#0.##}, {vector.W:#0.##})");
+            return FormattableString.Invariant($"Byte4({vector.X:#0.##}, {vector.Y:#0.##}, {vector.Z:#0.##}, {vector.W:#0.##})");
         }
 
         /// <summary>

src/ImageSharp/Processing/Processors/Convolution/EdgeDetector2DProcessor.cs

@@ -40,7 +40,7 @@ namespace SixLabors.ImageSharp.Processing.Processors.Convolution
         public DenseMatrix<float> KernelY { get; }
 
         /// <inheritdoc/>
-        public bool Grayscale { get; set; }
+        public bool Grayscale { get; }
 
         /// <inheritdoc />
         protected override void OnFrameApply(ImageFrame<TPixel> source, Rectangle sourceRectangle, Configuration configuration)

tests/ImageSharp.Tests/FileTestBase.cs

@@ -1,6 +1,7 @@
 // Copyright (c) Six Labors and contributors.
 // Licensed under the Apache License, Version 2.0.
 
+using System;
 using System.Collections.Generic;
 
 namespace SixLabors.ImageSharp.Tests
@@ -8,6 +9,7 @@ namespace SixLabors.ImageSharp.Tests
     /// <summary>
     /// The test base class for reading and writing to files.
     /// </summary>
+    [Obsolete("See: https://github.com/SixLabors/ImageSharp/issues/868")]
     public abstract class FileTestBase
     {
         /// <summary>

tests/ImageSharp.Tests/PixelFormats/Argb32Tests.cs

@@ -1,6 +1,7 @@
 // Copyright (c) Six Labors and contributors.
 // Licensed under the Apache License, Version 2.0.
 
+using System;
 using System.Numerics;
 using SixLabors.ImageSharp.PixelFormats;
 using Xunit;
@@ -9,6 +10,67 @@ namespace SixLabors.ImageSharp.Tests.PixelFormats
 {
     public class Argb32Tests
     {
+        /// <summary>
+        /// Tests the equality operators for equality.
+        /// </summary>
+        [Fact]
+        public void AreEqual()
+        {
+            var color1 = new Argb32(0.0f, 0.0f, 0.0f, 0.0f);
+            var color2 = new Argb32(new Vector4(0.0f));
+            var color3 = new Argb32(new Vector4(1.0f, 0.0f, 1.0f, 1.0f));
+            var color4 = new Argb32(1.0f, 0.0f, 1.0f, 1.0f);
+
+            Assert.Equal(color1, color2);
+            Assert.Equal(color3, color4);
+        }
+
+        /// <summary>
+        /// Tests the equality operators for inequality.
+        /// </summary>
+        [Fact]
+        public void AreNotEqual()
+        {
+            var color1 = new Argb32(0.0f, 0.0f, 0.0f, 0.0f);
+            var color2 = new Argb32(new Vector4(1.0f));
+            var color3 = new Argb32(new Vector4(1.0f, 0.0f, 0.0f, 1.0f));
+            var color4 = new Argb32(1.0f, 1.0f, 0.0f, 1.0f);
+
+            Assert.NotEqual(color1, color2);
+            Assert.NotEqual(color3, color4);
+        }
+
+        /// <summary>
+        /// Tests whether the color constructor correctly assign properties.
+        /// </summary>
+        [Fact]
+        public void ConstructorAssignsProperties()
+        {
+            var color1 = new Argb32(1, .1f, .133f, .864f);
+            Assert.Equal(255, color1.R);
+            Assert.Equal((byte)Math.Round(.1f * 255), color1.G);
+            Assert.Equal((byte)Math.Round(.133f * 255), color1.B);
+            Assert.Equal((byte)Math.Round(.864f * 255), color1.A);
+
+            var color2 = new Argb32(1, .1f, .133f);
+            Assert.Equal(255, color2.R);
+            Assert.Equal(Math.Round(.1f * 255), color2.G);
+            Assert.Equal(Math.Round(.133f * 255), color2.B);
+            Assert.Equal(255, color2.A);
+
+            var color4 = new Argb32(new Vector3(1, .1f, .133f));
+            Assert.Equal(255, color4.R);
+            Assert.Equal(Math.Round(.1f * 255), color4.G);
+            Assert.Equal(Math.Round(.133f * 255), color4.B);
+            Assert.Equal(255, color4.A);
+
+            var color5 = new Argb32(new Vector4(1, .1f, .133f, .5f));
+            Assert.Equal(255, color5.R);
+            Assert.Equal(Math.Round(.1f * 255), color5.G);
+            Assert.Equal(Math.Round(.133f * 255), color5.B);
+            Assert.Equal(Math.Round(.5f * 255), color5.A);
+        }
+
         [Fact]
         public void Argb32_PackedValue()
         {

tests/ImageSharp.Tests/PixelFormats/Bgr24Tests.cs

@@ -9,6 +9,24 @@ namespace SixLabors.ImageSharp.Tests.PixelFormats
 {
     public class Bgr24Tests
     {
+        [Fact]
+        public void AreEqual()
+        {
+            var color1 = new Bgr24(byte.MaxValue, 0, byte.MaxValue);
+            var color2 = new Bgr24(byte.MaxValue, 0, byte.MaxValue);
+            
+            Assert.Equal(color1, color2);
+        }
+
+        [Fact]
+        public void AreNotEqual()
+        {
+            var color1 = new Bgr24(byte.MaxValue, 0, 0);
+            var color2 = new Bgr24(byte.MaxValue, 0, byte.MaxValue);
+
+            Assert.NotEqual(color1, color2);
+        }
+
         public static readonly TheoryData<byte, byte, byte> ColorData =
             new TheoryData<byte, byte, byte>() { { 1, 2, 3 }, { 4, 5, 6 }, { 0, 255, 42 } };
 

tests/ImageSharp.Tests/PixelFormats/Bgr565Tests.cs

@@ -9,6 +9,36 @@ namespace SixLabors.ImageSharp.Tests.PixelFormats
 {
     public class Bgr565Tests
     {
+        /// <summary>
+        /// Tests the equality operators for equality.
+        /// </summary>
+        [Fact]
+        public void AreEqual()
+        {
+            var color1 = new Bgr565(0.0f, 0.0f, 0.0f);
+            var color2 = new Bgr565(new Vector3(0.0f));
+            var color3 = new Bgr565(new Vector3(1.0f, 0.0f, 1.0f));
+            var color4 = new Bgr565(1.0f, 0.0f, 1.0f);
+
+            Assert.Equal(color1, color2);
+            Assert.Equal(color3, color4);
+        }
+
+        /// <summary>
+        /// Tests the equality operators for inequality.
+        /// </summary>
+        [Fact]
+        public void AreNotEqual()
+        {
+            var color1 = new Bgr565(0.0f, 0.0f, 0.0f);
+            var color2 = new Bgr565(new Vector3(1.0f));
+            var color3 = new Bgr565(new Vector3(1.0f, 0.0f, 0.0f));
+            var color4 = new Bgr565(1.0f, 1.0f, 0.0f);
+
+            Assert.NotEqual(color1, color2);
+            Assert.NotEqual(color3, color4);
+        }
+
         [Fact]
         public void Bgr565_PackedValue()
         {
@@ -77,6 +107,140 @@ namespace SixLabors.ImageSharp.Tests.PixelFormats
             Assert.Equal(expected, bgr.PackedValue);
         }
 
+        [Fact]
+        public void Bgr565_FromArgb32()
+        {
+            // arrange
+            var bgr1 = default(Bgr565);
+            var bgr2 = default(Bgr565);
+            ushort expected1 = ushort.MaxValue;
+            ushort expected2 = ushort.MaxValue;
+
+            // act
+            bgr1.FromArgb32(new Argb32(1.0f, 1.0f, 1.0f, 1.0f));
+            bgr2.FromArgb32(new Argb32(1.0f, 1.0f, 1.0f, 0.0f));
+
+            // assert
+            Assert.Equal(expected1, bgr1.PackedValue);
+            Assert.Equal(expected2, bgr2.PackedValue);
+        }
+
+        [Fact]
+        public void Bgr565_FromRgba32()
+        {
+            // arrange
+            var bgr1 = default(Bgr565);
+            var bgr2 = default(Bgr565);
+            ushort expected1 = ushort.MaxValue;
+            ushort expected2 = ushort.MaxValue;
+
+            // act
+            bgr1.FromRgba32(new Rgba32(1.0f, 1.0f, 1.0f, 1.0f));
+            bgr2.FromRgba32(new Rgba32(1.0f, 1.0f, 1.0f, 0.0f));
+
+            // assert
+            Assert.Equal(expected1, bgr1.PackedValue);
+            Assert.Equal(expected2, bgr2.PackedValue);
+        }
+
+        [Fact]
+        public void Bgr565_ToRgba32()
+        {
+            // arrange
+            var bgra = new Bgr565(Vector3.One);
+            var expected = new Rgba32(Vector4.One);
+            var actual = default(Rgba32);
+
+            // act
+            bgra.ToRgba32(ref actual);
+
+            Assert.Equal(expected, actual);
+        }
+
+        [Fact]
+        public void Bgra565_FromRgb48()
+        {
+            // arrange
+            var bgr = default(Bgr565);
+            ushort expectedPackedValue = ushort.MaxValue;
+
+            // act
+            bgr.FromRgb48(new Rgb48(ushort.MaxValue, ushort.MaxValue, ushort.MaxValue));
+
+            // assert
+            Assert.Equal(expectedPackedValue, bgr.PackedValue);
+        }
+
+        [Fact]
+        public void Bgra565_FromRgba64()
+        {
+            // arrange
+            var bgr = default(Bgr565);
+            ushort expectedPackedValue = ushort.MaxValue;
+
+            // act
+            bgr.FromRgba64(new Rgba64(ushort.MaxValue, ushort.MaxValue, ushort.MaxValue, ushort.MaxValue));
+
+            // assert
+            Assert.Equal(expectedPackedValue, bgr.PackedValue);
+        }
+
+        [Fact]
+        public void Bgr565_FromBgr24()
+        {
+            // arrange
+            var bgr = default(Bgr565);
+            ushort expected = ushort.MaxValue;
+            
+            // act
+            bgr.FromBgr24(new Bgr24(byte.MaxValue, byte.MaxValue, byte.MaxValue));
+            
+            // assert
+            Assert.Equal(expected, bgr.PackedValue);
+        }
+
+        [Fact]
+        public void Bgr565_FromRgb24()
+        {
+            // arrange
+            var bgr = default(Bgr565);
+            ushort expected = ushort.MaxValue;
+
+            // act
+            bgr.FromRgb24(new Rgb24(byte.MaxValue, byte.MaxValue, byte.MaxValue));
+
+            // assert
+            Assert.Equal(expected, bgr.PackedValue);
+        }
+
+        [Fact]
+        public void Bgr565_FromGrey8()
+        {
+            // arrange
+            var bgr = default(Bgr565);
+            ushort expected = ushort.MaxValue;
+
+            // act
+            bgr.FromGray8(new Gray8(byte.MaxValue));
+
+            // assert
+            Assert.Equal(expected, bgr.PackedValue);
+        }
+
+        [Fact]
+        public void Bgr565_FromGrey16()
+        {
+            // arrange
+            var bgr = default(Bgr565);
+            ushort expected = ushort.MaxValue;
+
+            // act
+            bgr.FromGray16(new Gray16(ushort.MaxValue));
+
+            // assert
+            Assert.Equal(expected, bgr.PackedValue);
+        }
+
         [Fact]
         public void Bgr565_Clamping()
         {

tests/ImageSharp.Tests/PixelFormats/Bgra32Tests.cs

@@ -1,6 +1,7 @@
 // Copyright (c) Six Labors and contributors.
 // Licensed under the Apache License, Version 2.0.
 
+using System;
 using System.Numerics;
 using SixLabors.ImageSharp.PixelFormats;
 using Xunit;
@@ -9,6 +10,30 @@ namespace SixLabors.ImageSharp.Tests.PixelFormats
 {
     public class Bgra32Tests
     {
+        /// <summary>
+        /// Tests the equality operators for equality.
+        /// </summary>
+        [Fact]
+        public void AreEqual()
+        {
+            var color1 = new Bgra32(byte.MaxValue, byte.MaxValue, byte.MaxValue, byte.MaxValue);
+            var color2 = new Bgra32(byte.MaxValue, byte.MaxValue, byte.MaxValue);
+            
+            Assert.Equal(color1, color2);
+        }
+
+        /// <summary>
+        /// Tests the equality operators for inequality.
+        /// </summary>
+        [Fact]
+        public void AreNotEqual()
+        {
+            var color1 = new Bgra32(0, 0, byte.MaxValue, byte.MaxValue);
+            var color2 = new Bgra32(byte.MaxValue, byte.MaxValue, byte.MaxValue);
+
+            Assert.NotEqual(color1, color2);
+        }
+
         public static readonly TheoryData<byte, byte, byte, byte> ColorData =
             new TheoryData<byte, byte, byte, byte>()
                 {

tests/ImageSharp.Tests/PixelFormats/Bgra4444Tests.cs

@@ -9,6 +9,36 @@ namespace SixLabors.ImageSharp.Tests.PixelFormats
 {
     public class Bgra4444Tests
     {
+        /// <summary>
+        /// Tests the equality operators for equality.
+        /// </summary>
+        [Fact]
+        public void AreEqual()
+        {
+            var color1 = new Bgra4444(0.0f, 0.0f, 0.0f, 0.0f);
+            var color2 = new Bgra4444(new Vector4(0.0f));
+            var color3 = new Bgra4444(new Vector4(1.0f, 0.0f, 1.0f, 1.0f));
+            var color4 = new Bgra4444(1.0f, 0.0f, 1.0f, 1.0f);
+
+            Assert.Equal(color1, color2);
+            Assert.Equal(color3, color4);
+        }
+
+        /// <summary>
+        /// Tests the equality operators for inequality.
+        /// </summary>
+        [Fact]
+        public void AreNotEqual()
+        {
+            var color1 = new Bgra4444(0.0f, 0.0f, 0.0f, 0.0f);
+            var color2 = new Bgra4444(new Vector4(1.0f));
+            var color3 = new Bgra4444(new Vector4(1.0f, 0.0f, 0.0f, 1.0f));
+            var color4 = new Bgra4444(1.0f, 1.0f, 0.0f, 1.0f);
+
+            Assert.NotEqual(color1, color2);
+            Assert.NotEqual(color3, color4);
+        }
+
         [Fact]
         public void Bgra4444_PackedValue()
         {
@@ -48,6 +78,20 @@ namespace SixLabors.ImageSharp.Tests.PixelFormats
             Assert.Equal(1, actual.W);
         }
 
+        [Fact]
+        public void Bgra4444_ToRgba32()
+        {
+            // arrange
+            var bgra = new Bgra4444(Vector4.One);
+            var expected = new Rgba32(Vector4.One);
+            var actual = default(Rgba32);
+
+            // act
+            bgra.ToRgba32(ref actual);
+
+            Assert.Equal(expected, actual);
+        }
+
         [Fact]
         public void Bgra4444_FromScaledVector4()
         {
@@ -78,6 +122,122 @@ namespace SixLabors.ImageSharp.Tests.PixelFormats
             Assert.Equal(expected, bgra.PackedValue);
         }
 
+        [Fact]
+        public void Bgra4444_FromArgb32()
+        {
+            // arrange
+            var bgra = default(Bgra4444);
+            ushort expectedPackedValue = ushort.MaxValue;
+
+            // act
+            bgra.FromArgb32(new Argb32(255, 255, 255, 255));
+
+            // assert
+            Assert.Equal(expectedPackedValue, bgra.PackedValue);
+        }
+
+        [Fact]
+        public void Bgra4444_FromRgba32()
+        {
+            // arrange
+            var bgra1 = default(Bgra4444);
+            var bgra2 = default(Bgra4444);
+            ushort expectedPackedValue1 = ushort.MaxValue;
+            ushort expectedPackedValue2 = 0xFF0F;
+
+            // act
+            bgra1.FromRgba32(new Rgba32(255, 255, 255, 255));
+            bgra2.FromRgba32(new Rgba32(255, 0, 255, 255));
+
+            // assert
+            Assert.Equal(expectedPackedValue1, bgra1.PackedValue);
+            Assert.Equal(expectedPackedValue2, bgra2.PackedValue);
+        }
+
+        [Fact]
+        public void Bgra4444_FromRgb48()
+        {
+            // arrange
+            var bgra = default(Bgra4444);
+            ushort expectedPackedValue = ushort.MaxValue;
+
+            // act
+            bgra.FromRgb48(new Rgb48(ushort.MaxValue, ushort.MaxValue, ushort.MaxValue));
+
+            // assert
+            Assert.Equal(expectedPackedValue, bgra.PackedValue);
+        }
+
+        [Fact]
+        public void Bgra4444_FromRgba64()
+        {
+            // arrange
+            var bgra = default(Bgra4444);
+            ushort expectedPackedValue = ushort.MaxValue;
+
+            // act
+            bgra.FromRgba64(new Rgba64(ushort.MaxValue, ushort.MaxValue, ushort.MaxValue, ushort.MaxValue));
+
+            // assert
+            Assert.Equal(expectedPackedValue, bgra.PackedValue);
+        }
+
+        [Fact]
+        public void Bgra4444_FromGrey16()
+        {
+            // arrange
+            var bgra = default(Bgra4444);
+            ushort expectedPackedValue = ushort.MaxValue;
+
+            // act
+            bgra.FromGray16(new Gray16(ushort.MaxValue));
+
+            // assert
+            Assert.Equal(expectedPackedValue, bgra.PackedValue);
+        }
+
+        [Fact]
+        public void Bgra4444_FromGrey8()
+        {
+            // arrange
+            var bgra = default(Bgra4444);
+            ushort expectedPackedValue = ushort.MaxValue;
+
+            // act
+            bgra.FromGray8(new Gray8(byte.MaxValue));
+
+            // assert
+            Assert.Equal(expectedPackedValue, bgra.PackedValue);
+        }
+
+        [Fact]
+        public void Bgra4444_FromBgr24()
+        {
+            // arrange
+            var bgra = default(Bgra4444);
+            ushort expectedPackedValue = ushort.MaxValue;
+
+            // act
+            bgra.FromBgr24(new Bgr24(byte.MaxValue, byte.MaxValue, byte.MaxValue));
+
+            // assert
+            Assert.Equal(expectedPackedValue, bgra.PackedValue);
+        }
+
+        [Fact]
+        public void Bgra4444_FromRgb24()
+        {
+            // arrange
+            var bgra = default(Bgra4444);
+            ushort expectedPackedValue = ushort.MaxValue;
+
+            // act
+            bgra.FromRgb24(new Rgb24(byte.MaxValue, byte.MaxValue, byte.MaxValue));
+
+            // assert
+            Assert.Equal(expectedPackedValue, bgra.PackedValue);
+        }
+
         [Fact]
         public void Bgra4444_Clamping()
         {

tests/ImageSharp.Tests/PixelFormats/Bgra5551Tests.cs

@@ -1,7 +1,6 @@
 // Copyright (c) Six Labors and contributors.
 // Licensed under the Apache License, Version 2.0.
 
-using System;
 using System.Numerics;
 using SixLabors.ImageSharp.PixelFormats;
 using Xunit;
@@ -10,6 +9,36 @@ namespace SixLabors.ImageSharp.Tests.PixelFormats
 {
     public class Bgra5551Tests
     {
+        /// <summary>
+        /// Tests the equality operators for equality.
+        /// </summary>
+        [Fact]
+        public void AreEqual()
+        {
+            var color1 = new Bgra5551(0.0f, 0.0f, 0.0f, 0.0f);
+            var color2 = new Bgra5551(new Vector4(0.0f));
+            var color3 = new Bgra5551(new Vector4(1.0f, 0.0f, 0.0f, 1.0f));
+            var color4 = new Bgra5551(1.0f, 0.0f, 0.0f, 1.0f);
+            
+            Assert.Equal(color1, color2);
+            Assert.Equal(color3, color4);
+        }
+
+        /// <summary>
+        /// Tests the equality operators for inequality.
+        /// </summary>
+        [Fact]
+        public void AreNotEqual()
+        {
+            var color1 = new Bgra5551(0.0f, 0.0f, 0.0f, 0.0f);
+            var color2 = new Bgra5551(new Vector4(1.0f));
+            var color3 = new Bgra5551(new Vector4(1.0f, 0.0f, 0.0f, 1.0f));
+            var color4 = new Bgra5551(1.0f, 1.0f, 0.0f, 1.0f);
+
+            Assert.NotEqual(color1, color2);
+            Assert.NotEqual(color3, color4);
+        }
+
         [Fact]
         public void Bgra5551_PackedValue()
         {
@@ -54,6 +83,20 @@ namespace SixLabors.ImageSharp.Tests.PixelFormats
             Assert.Equal(1, actual.W);
         }
 
+        [Fact]
+        public void Bgra5551_ToRgba32()
+        {
+            // arrange
+            var bgra = new Bgra5551(Vector4.One);
+            var expected = new Rgba32(Vector4.One);
+            var actual = default(Rgba32);
+
+            // act
+            bgra.ToRgba32(ref actual);
+
+            Assert.Equal(expected, actual);
+        }
+
         [Fact]
         public void Bgra5551_FromScaledVector4()
         {
@@ -70,6 +113,22 @@ namespace SixLabors.ImageSharp.Tests.PixelFormats
             Assert.Equal(expected, actual);
         }
 
+        [Fact]
+        public void Bgra5551_FromBgra5551()
+        {
+            // arrange
+            var bgra = default(Bgra5551);
+            var actual = default(Bgra5551);
+            var expected = new Bgra5551(1.0f, 0.0f, 1.0f, 1.0f);
+
+            // act 
+            bgra.FromBgra5551(expected);
+            actual.FromBgra5551(bgra);
+
+            // assert
+            Assert.Equal(expected, actual);
+        }
+
         [Fact]
         public void Bgra5551_FromRgba32()
         {
@@ -120,6 +179,20 @@ namespace SixLabors.ImageSharp.Tests.PixelFormats
             Assert.Equal(expectedPackedValue, bgra.PackedValue);
         }
 
+        [Fact]
+        public void Bgra5551_FromRgb48()
+        {
+            // arrange
+            var bgra = default(Bgra5551);
+            ushort expectedPackedValue = ushort.MaxValue;
+
+            // act
+            bgra.FromRgb48(new Rgb48(ushort.MaxValue, ushort.MaxValue, ushort.MaxValue));
+
+            // assert
+            Assert.Equal(expectedPackedValue, bgra.PackedValue);
+        }
+
         [Fact]
         public void Bgra5551_FromRgba64()
         {

tests/ImageSharp.Tests/PixelFormats/Byte4Tests.cs

@@ -9,6 +9,36 @@ namespace SixLabors.ImageSharp.Tests.PixelFormats
 {
     public class Byte4Tests
     {
+        /// <summary>
+        /// Tests the equality operators for equality.
+        /// </summary>
+        [Fact]
+        public void AreEqual()
+        {
+            var color1 = new Byte4(0.0f, 0.0f, 0.0f, 0.0f);
+            var color2 = new Byte4(new Vector4(0.0f));
+            var color3 = new Byte4(new Vector4(1.0f, 0.0f, 1.0f, 1.0f));
+            var color4 = new Byte4(1.0f, 0.0f, 1.0f, 1.0f);
+
+            Assert.Equal(color1, color2);
+            Assert.Equal(color3, color4);
+        }
+
+        /// <summary>
+        /// Tests the equality operators for inequality.
+        /// </summary>
+        [Fact]
+        public void AreNotEqual()
+        {
+            var color1 = new Byte4(0.0f, 0.0f, 0.0f, 0.0f);
+            var color2 = new Byte4(new Vector4(1.0f));
+            var color3 = new Byte4(new Vector4(1.0f, 0.0f, 0.0f, 1.0f));
+            var color4 = new Byte4(1.0f, 1.0f, 0.0f, 1.0f);
+
+            Assert.NotEqual(color1, color2);
+            Assert.NotEqual(color3, color4);
+        }
+
         [Fact]
         public void Byte4_PackedValue()
         {
@@ -45,6 +75,20 @@ namespace SixLabors.ImageSharp.Tests.PixelFormats
             Assert.Equal(1, actual.W);
         }
 
+        [Fact]
+        public void Byte4_ToRgba32()
+        {
+            // arrange
+            var byte4 = new Byte4(byte.MaxValue, byte.MaxValue, byte.MaxValue, byte.MaxValue);
+            var expected = new Rgba32(Vector4.One);
+            var actual = default(Rgba32);
+
+            // act
+            byte4.ToRgba32(ref actual);
+
+            Assert.Equal(expected, actual);
+        }
+
         [Fact]
         public void Byte4_FromScaledVector4()
         {
@@ -61,18 +105,130 @@ namespace SixLabors.ImageSharp.Tests.PixelFormats
             Assert.Equal(expected, actual);
         }
 
+        [Fact]
+        public void Byte4_FromArgb32()
+        {
+            // arrange
+            var byte4 = default(Byte4);
+            uint expectedPackedValue = uint.MaxValue;
+
+            // act
+            byte4.FromArgb32(new Argb32(255, 255, 255, 255));
+
+            // assert
+            Assert.Equal(expectedPackedValue, byte4.PackedValue);
+        }
+
+        [Fact]
+        public void Byte4_FromBgr24()
+        {
+            // arrange
+            var byte4 = default(Byte4);
+            uint expectedPackedValue = uint.MaxValue;
+
+            // act
+            byte4.FromBgr24(new Bgr24(byte.MaxValue, byte.MaxValue, byte.MaxValue));
+
+            // assert
+            Assert.Equal(expectedPackedValue, byte4.PackedValue);
+        }
+
+        [Fact]
+        public void Byte4_FromGrey8()
+        {
+            // arrange
+            var byte4 = default(Byte4);
+            uint expectedPackedValue = uint.MaxValue;
+
+            // act
+            byte4.FromGray8(new Gray8(byte.MaxValue));
+
+            // assert
+            Assert.Equal(expectedPackedValue, byte4.PackedValue);
+        }
+
+        [Fact]
+        public void Byte4_FromGrey16()
+        {
+            // arrange
+            var byte4 = default(Byte4);
+            uint expectedPackedValue = uint.MaxValue;
+
+            // act
+            byte4.FromGray16(new Gray16(ushort.MaxValue));
+
+            // assert
+            Assert.Equal(expectedPackedValue, byte4.PackedValue);
+        }
+
+        [Fact]
+        public void Byte4_FromRgb24()
+        {
+            // arrange
+            var byte4 = default(Byte4);
+            uint expectedPackedValue = uint.MaxValue;
+
+            // act
+            byte4.FromRgb24(new Rgb24(byte.MaxValue, byte.MaxValue, byte.MaxValue));
+
+            // assert
+            Assert.Equal(expectedPackedValue, byte4.PackedValue);
+        }
+
         [Fact]
         public void Byte4_FromBgra5551()
         {
             // arrange
-            var rgb = default(Byte4);
+            var byte4 = default(Byte4);
             uint expected = 0xFFFFFFFF;
 
             // act
-            rgb.FromBgra5551(new Bgra5551(1.0f, 1.0f, 1.0f, 1.0f));
+            byte4.FromBgra5551(new Bgra5551(1.0f, 1.0f, 1.0f, 1.0f));
+
+            // assert
+            Assert.Equal(expected, byte4.PackedValue);
+        }
+
+        [Fact]
+        public void Byte4_FromRgba32()
+        {
+            // arrange
+            var byte4 = default(Byte4);
+            uint expectedPackedValue1 = uint.MaxValue;
+            
+            // act
+            byte4.FromRgba32(new Rgba32(255, 255, 255, 255));
+            
+            // assert
+            Assert.Equal(expectedPackedValue1, byte4.PackedValue);
+        }
+
+        [Fact]
+        public void Byte4_FromRgb48()
+        {
+            // arrange
+            var byte4 = default(Byte4);
+            uint expectedPackedValue = uint.MaxValue;
+
+            // act
+            byte4.FromRgb48(new Rgb48(ushort.MaxValue, ushort.MaxValue, ushort.MaxValue));
+
+            // assert
+            Assert.Equal(expectedPackedValue, byte4.PackedValue);
+        }
+
+        [Fact]
+        public void Byte4_FromRgba64()
+        {
+            // arrange
+            var byte4 = default(Byte4);
+            uint expectedPackedValue = uint.MaxValue;
+
+            // act
+            byte4.FromRgba64(new Rgba64(ushort.MaxValue, ushort.MaxValue, ushort.MaxValue, ushort.MaxValue));
 
             // assert
-            Assert.Equal(expected, rgb.PackedValue);
+            Assert.Equal(expectedPackedValue, byte4.PackedValue);
         }
 
         [Fact]

tests/ImageSharp.Tests/PixelFormats/Gray16Tests.cs

@@ -9,6 +9,24 @@ namespace SixLabors.ImageSharp.Tests.PixelFormats
 {
     public class Gray16Tests
     {
+        [Fact]
+        public void AreEqual()
+        {
+            var color1 = new Gray16(3000);
+            var color2 = new Gray16(3000);
+
+            Assert.Equal(color1, color2);
+        }
+
+        [Fact]
+        public void AreNotEqual()
+        {
+            var color1 = new Gray16(12345);
+            var color2 = new Gray16(54321);
+
+            Assert.NotEqual(color1, color2);
+        }
+
         [Theory]
         [InlineData(0)]
         [InlineData(65535)]

tests/ImageSharp.Tests/PixelFormats/Gray8Tests.cs

@@ -1,10 +1,8 @@
 // Copyright (c) Six Labors and contributors.
 // Licensed under the Apache License, Version 2.0.
 
-using System.Diagnostics;
 using System.Numerics;
 
-using SixLabors.ImageSharp.Common.Helpers;
 using SixLabors.ImageSharp.PixelFormats;
 using Xunit;
 
@@ -45,6 +43,24 @@ namespace SixLabors.ImageSharp.Tests.PixelFormats
         public void Gray8_PackedValue_EqualsInput(byte input)
             => Assert.Equal(input, new Gray8(input).PackedValue);
 
+        [Fact]
+        public void AreEqual()
+        {
+            var color1 = new Gray8(100);
+            var color2 = new Gray8(100);
+            
+            Assert.Equal(color1, color2);
+        }
+
+        [Fact]
+        public void AreNotEqual()
+        {
+            var color1 = new Gray8(100);
+            var color2 = new Gray8(200);
+
+            Assert.NotEqual(color1, color2);
+        }
+
         [Fact]
         public void Gray8_FromScaledVector4()
         {

tests/ImageSharp.Tests/PixelFormats/NormalizedByte4Tests.cs

@@ -9,6 +9,36 @@ namespace SixLabors.ImageSharp.Tests.PixelFormats
 {
     public class NormalizedByte4Tests
     {
+        /// <summary>
+        /// Tests the equality operators for equality.
+        /// </summary>
+        [Fact]
+        public void AreEqual()
+        {
+            var color1 = new NormalizedByte4(0.0f, 0.0f, 0.0f, 0.0f);
+            var color2 = new NormalizedByte4(new Vector4(0.0f));
+            var color3 = new NormalizedByte4(new Vector4(1.0f, 0.0f, 1.0f, 1.0f));
+            var color4 = new NormalizedByte4(1.0f, 0.0f, 1.0f, 1.0f);
+
+            Assert.Equal(color1, color2);
+            Assert.Equal(color3, color4);
+        }
+
+        /// <summary>
+        /// Tests the equality operators for inequality.
+        /// </summary>
+        [Fact]
+        public void AreNotEqual()
+        {
+            var color1 = new NormalizedByte4(0.0f, 0.0f, 0.0f, 0.0f);
+            var color2 = new NormalizedByte4(new Vector4(1.0f));
+            var color3 = new NormalizedByte4(new Vector4(1.0f, 0.0f, 0.0f, 1.0f));
+            var color4 = new NormalizedByte4(1.0f, 1.0f, 0.0f, 1.0f);
+
+            Assert.NotEqual(color1, color2);
+            Assert.NotEqual(color3, color4);
+        }
+
         [Fact]
         public void NormalizedByte4_PackedValues()
         {
@@ -61,6 +91,90 @@ namespace SixLabors.ImageSharp.Tests.PixelFormats
             Assert.Equal(expected, actual);
         }
 
+        [Fact]
+        public void NormalizedByte4_FromArgb32()
+        {
+            // arrange
+            var byte4 = default(NormalizedByte4);
+            Vector4 expected = Vector4.One;
+
+            // act
+            byte4.FromArgb32(new Argb32(255, 255, 255, 255));
+
+            // assert
+            Assert.Equal(expected, byte4.ToScaledVector4());
+        }
+
+        [Fact]
+        public void NormalizedByte4_FromBgr24()
+        {
+            // arrange
+            var byte4 = default(NormalizedByte4);
+            Vector4 expected = Vector4.One;
+
+            // act
+            byte4.FromBgr24(new Bgr24(byte.MaxValue, byte.MaxValue, byte.MaxValue));
+
+            // assert
+            Assert.Equal(expected, byte4.ToScaledVector4());
+        }
+
+        [Fact]
+        public void NormalizedByte4_FromGrey8()
+        {
+            // arrange
+            var byte4 = default(NormalizedByte4);
+            Vector4 expected = Vector4.One;
+
+            // act
+            byte4.FromGray8(new Gray8(byte.MaxValue));
+
+            // assert
+            Assert.Equal(expected, byte4.ToScaledVector4());
+        }
+
+        [Fact]
+        public void NormalizedByte4_FromGrey16()
+        {
+            // arrange
+            var byte4 = default(NormalizedByte4);
+            Vector4 expected = Vector4.One;
+
+            // act
+            byte4.FromGray16(new Gray16(ushort.MaxValue));
+
+            // assert
+            Assert.Equal(expected, byte4.ToScaledVector4());
+        }
+
+        [Fact]
+        public void NormalizedByte4_FromRgb24()
+        {
+            // arrange
+            var byte4 = default(NormalizedByte4);
+            Vector4 expected = Vector4.One;
+
+            // act
+            byte4.FromRgb24(new Rgb24(byte.MaxValue, byte.MaxValue, byte.MaxValue));
+
+            // assert
+            Assert.Equal(expected, byte4.ToScaledVector4());
+        }
+
+        [Fact]
+        public void NormalizedByte4_FromRgba32()
+        {
+            // arrange
+            var byte4 = default(NormalizedByte4);
+            Vector4 expected = Vector4.One;
+
+            // act
+            byte4.FromRgba32(new Rgba32(255, 255, 255, 255));
+
+            // assert
+            Assert.Equal(expected, byte4.ToScaledVector4());
+        }
+
         [Fact]
         public void NormalizedByte4_FromBgra5551()
         {
@@ -74,5 +188,47 @@ namespace SixLabors.ImageSharp.Tests.PixelFormats
             // assert
             Assert.Equal(expected, normalizedByte4.ToVector4());
         }
+
+        [Fact]
+        public void NormalizedByte4_FromRgb48()
+        {
+            // arrange
+            var byte4 = default(NormalizedByte4);
+            Vector4 expected = Vector4.One;
+
+            // act
+            byte4.FromRgb48(new Rgb48(ushort.MaxValue, ushort.MaxValue, ushort.MaxValue));
+
+            // assert
+            Assert.Equal(expected, byte4.ToScaledVector4());
+        }
+
+        [Fact]
+        public void NormalizedByte4_FromRgba64()
+        {
+            // arrange
+            var byte4 = default(NormalizedByte4);
+            Vector4 expected = Vector4.One;
+
+            // act
+            byte4.FromRgba64(new Rgba64(ushort.MaxValue, ushort.MaxValue, ushort.MaxValue, ushort.MaxValue));
+
+            // assert
+            Assert.Equal(expected, byte4.ToScaledVector4());
+        }
+
+        [Fact]
+        public void NormalizedByte4_ToRgba32()
+        {
+            // arrange
+            var byte4 = new NormalizedByte4(byte.MaxValue, byte.MaxValue, byte.MaxValue, byte.MaxValue);
+            var expected = new Rgba32(Vector4.One);
+            var actual = default(Rgba32);
+
+            // act
+            byte4.ToRgba32(ref actual);
+
+            Assert.Equal(expected, actual);
+        }
     }
 }

tests/ImageSharp.Tests/PixelFormats/NormalizedShort4Tests.cs

@@ -9,6 +9,36 @@ namespace SixLabors.ImageSharp.Tests.PixelFormats
 {
     public class NormalizedShort4Tests
     {
+        /// <summary>
+        /// Tests the equality operators for equality.
+        /// </summary>
+        [Fact]
+        public void AreEqual()
+        {
+            var color1 = new NormalizedShort4(0.0f, 0.0f, 0.0f, 0.0f);
+            var color2 = new NormalizedShort4(new Vector4(0.0f));
+            var color3 = new NormalizedShort4(new Vector4(1.0f, 0.0f, 1.0f, 1.0f));
+            var color4 = new NormalizedShort4(1.0f, 0.0f, 1.0f, 1.0f);
+
+            Assert.Equal(color1, color2);
+            Assert.Equal(color3, color4);
+        }
+
+        /// <summary>
+        /// Tests the equality operators for inequality.
+        /// </summary>
+        [Fact]
+        public void AreNotEqual()
+        {
+            var color1 = new NormalizedShort4(0.0f, 0.0f, 0.0f, 0.0f);
+            var color2 = new NormalizedShort4(new Vector4(1.0f));
+            var color3 = new NormalizedShort4(new Vector4(1.0f, 0.0f, 0.0f, 1.0f));
+            var color4 = new NormalizedShort4(1.0f, 1.0f, 0.0f, 1.0f);
+
+            Assert.NotEqual(color1, color2);
+            Assert.NotEqual(color3, color4);
+        }
+
         [Fact]
         public void NormalizedShort4_PackedValues()
         {
@@ -62,6 +92,132 @@ namespace SixLabors.ImageSharp.Tests.PixelFormats
             Assert.Equal(expected, actual);
         }
 
+        [Fact]
+        public void NormalizedShort4_FromArgb32()
+        {
+            // arrange
+            var byte4 = default(NormalizedShort4);
+            Vector4 expected = Vector4.One;
+
+            // act
+            byte4.FromArgb32(new Argb32(255, 255, 255, 255));
+
+            // assert
+            Assert.Equal(expected, byte4.ToScaledVector4());
+        }
+
+        [Fact]
+        public void NormalizedShort4_FromBgr24()
+        {
+            // arrange
+            var byte4 = default(NormalizedShort4);
+            Vector4 expected = Vector4.One;
+
+            // act
+            byte4.FromBgr24(new Bgr24(byte.MaxValue, byte.MaxValue, byte.MaxValue));
+
+            // assert
+            Assert.Equal(expected, byte4.ToScaledVector4());
+        }
+
+        [Fact]
+        public void NormalizedShort4_FromGrey8()
+        {
+            // arrange
+            var byte4 = default(NormalizedShort4);
+            Vector4 expected = Vector4.One;
+
+            // act
+            byte4.FromGray8(new Gray8(byte.MaxValue));
+
+            // assert
+            Assert.Equal(expected, byte4.ToScaledVector4());
+        }
+
+        [Fact]
+        public void NormalizedShort4_FromGrey16()
+        {
+            // arrange
+            var byte4 = default(NormalizedShort4);
+            Vector4 expected = Vector4.One;
+
+            // act
+            byte4.FromGray16(new Gray16(ushort.MaxValue));
+
+            // assert
+            Assert.Equal(expected, byte4.ToScaledVector4());
+        }
+
+        [Fact]
+        public void NormalizedShort4_FromRgb24()
+        {
+            // arrange
+            var byte4 = default(NormalizedShort4);
+            Vector4 expected = Vector4.One;
+
+            // act
+            byte4.FromRgb24(new Rgb24(byte.MaxValue, byte.MaxValue, byte.MaxValue));
+
+            // assert
+            Assert.Equal(expected, byte4.ToScaledVector4());
+        }
+
+        [Fact]
+        public void NormalizedShort4_FromRgba32()
+        {
+            // arrange
+            var byte4 = default(NormalizedShort4);
+            Vector4 expected = Vector4.One;
+
+            // act
+            byte4.FromRgba32(new Rgba32(255, 255, 255, 255));
+
+            // assert
+            Assert.Equal(expected, byte4.ToScaledVector4());
+        }
+
+        [Fact]
+        public void NormalizedShort4_FromRgb48()
+        {
+            // arrange
+            var byte4 = default(NormalizedShort4);
+            Vector4 expected = Vector4.One;
+
+            // act
+            byte4.FromRgb48(new Rgb48(ushort.MaxValue, ushort.MaxValue, ushort.MaxValue));
+
+            // assert
+            Assert.Equal(expected, byte4.ToScaledVector4());
+        }
+
+        [Fact]
+        public void NormalizedShort4_FromRgba64()
+        {
+            // arrange
+            var byte4 = default(NormalizedShort4);
+            Vector4 expected = Vector4.One;
+
+            // act
+            byte4.FromRgba64(new Rgba64(ushort.MaxValue, ushort.MaxValue, ushort.MaxValue, ushort.MaxValue));
+
+            // assert
+            Assert.Equal(expected, byte4.ToScaledVector4());
+        }
+
+        [Fact]
+        public void NormalizedShort4_ToRgba32()
+        {
+            // arrange
+            var byte4 = new NormalizedShort4(byte.MaxValue, byte.MaxValue, byte.MaxValue, byte.MaxValue);
+            var expected = new Rgba32(Vector4.One);
+            var actual = default(Rgba32);
+
+            // act
+            byte4.ToRgba32(ref actual);
+
+            Assert.Equal(expected, actual);
+        }
+
         [Fact]
         public void NormalizedShort4_FromBgra5551()
         {

tests/ImageSharp.Tests/PixelFormats/Rgba1010102Tests.cs

@@ -9,6 +9,36 @@ namespace SixLabors.ImageSharp.Tests.PixelFormats
 {
     public class Rgba1010102Tests
     {
+        /// <summary>
+        /// Tests the equality operators for equality.
+        /// </summary>
+        [Fact]
+        public void AreEqual()
+        {
+            var color1 = new Rgba1010102(0.0f, 0.0f, 0.0f, 0.0f);
+            var color2 = new Rgba1010102(new Vector4(0.0f));
+            var color3 = new Rgba1010102(new Vector4(1.0f, 0.0f, 1.0f, 1.0f));
+            var color4 = new Rgba1010102(1.0f, 0.0f, 1.0f, 1.0f);
+
+            Assert.Equal(color1, color2);
+            Assert.Equal(color3, color4);
+        }
+
+        /// <summary>
+        /// Tests the equality operators for inequality.
+        /// </summary>
+        [Fact]
+        public void AreNotEqual()
+        {
+            var color1 = new Rgba1010102(0.0f, 0.0f, 0.0f, 0.0f);
+            var color2 = new Rgba1010102(new Vector4(1.0f));
+            var color3 = new Rgba1010102(new Vector4(1.0f, 0.0f, 0.0f, 1.0f));
+            var color4 = new Rgba1010102(1.0f, 1.0f, 0.0f, 1.0f);
+
+            Assert.NotEqual(color1, color2);
+            Assert.NotEqual(color3, color4);
+        }
+
         [Fact]
         public void Rgba1010102_PackedValue()
         {
@@ -78,6 +108,122 @@ namespace SixLabors.ImageSharp.Tests.PixelFormats
             Assert.Equal(expected, rgba.PackedValue);
         }
 
+        [Fact]
+        public void Rgba1010102_FromArgb32()
+        {
+            // arrange
+            var rgba = default(Rgba1010102);
+            uint expectedPackedValue = uint.MaxValue;
+
+            // act
+            rgba.FromArgb32(new Argb32(255, 255, 255, 255));
+
+            // assert
+            Assert.Equal(expectedPackedValue, rgba.PackedValue);
+        }
+
+        [Fact]
+        public void Rgba1010102_FromRgba32()
+        {
+            // arrange
+            var rgba1 = default(Rgba1010102);
+            var rgba2 = default(Rgba1010102);
+            uint expectedPackedValue1 = uint.MaxValue;
+            uint expectedPackedValue2 = 0xFFF003FF;
+
+            // act
+            rgba1.FromRgba32(new Rgba32(255, 255, 255, 255));
+            rgba2.FromRgba32(new Rgba32(255, 0, 255, 255));
+
+            // assert
+            Assert.Equal(expectedPackedValue1, rgba1.PackedValue);
+            Assert.Equal(expectedPackedValue2, rgba2.PackedValue);
+        }
+
+        [Fact]
+        public void Rgba1010102_FromBgr24()
+        {
+            // arrange
+            var rgba = default(Rgba1010102);
+            uint expectedPackedValue = uint.MaxValue;
+
+            // act
+            rgba.FromBgr24(new Bgr24(byte.MaxValue, byte.MaxValue, byte.MaxValue));
+
+            // assert
+            Assert.Equal(expectedPackedValue, rgba.PackedValue);
+        }
+
+        [Fact]
+        public void Rgba1010102_FromGrey8()
+        {
+            // arrange
+            var rgba = default(Rgba1010102);
+            uint expectedPackedValue = uint.MaxValue;
+
+            // act
+            rgba.FromGray8(new Gray8(byte.MaxValue));
+
+            // assert
+            Assert.Equal(expectedPackedValue, rgba.PackedValue);
+        }
+
+        [Fact]
+        public void Rgba1010102_FromGrey16()
+        {
+            // arrange
+            var rgba = default(Rgba1010102);
+            uint expectedPackedValue = uint.MaxValue;
+
+            // act
+            rgba.FromGray16(new Gray16(ushort.MaxValue));
+
+            // assert
+            Assert.Equal(expectedPackedValue, rgba.PackedValue);
+        }
+
+        [Fact]
+        public void Rgba1010102_FromRgb24()
+        {
+            // arrange
+            var rgba = default(Rgba1010102);
+            uint expectedPackedValue = uint.MaxValue;
+
+            // act
+            rgba.FromRgb24(new Rgb24(byte.MaxValue, byte.MaxValue, byte.MaxValue));
+
+            // assert
+            Assert.Equal(expectedPackedValue, rgba.PackedValue);
+        }
+
+        [Fact]
+        public void Rgba1010102_FromRgb48()
+        {
+            // arrange
+            var rgba = default(Rgba1010102);
+            uint expectedPackedValue = uint.MaxValue;
+
+            // act
+            rgba.FromRgb48(new Rgb48(ushort.MaxValue, ushort.MaxValue, ushort.MaxValue));
+
+            // assert
+            Assert.Equal(expectedPackedValue, rgba.PackedValue);
+        }
+
+        [Fact]
+        public void Rgba1010102_FromRgba64()
+        {
+            // arrange
+            var rgba = default(Rgba1010102);
+            uint expectedPackedValue = uint.MaxValue;
+
+            // act
+            rgba.FromRgba64(new Rgba64(ushort.MaxValue, ushort.MaxValue, ushort.MaxValue, ushort.MaxValue));
+
+            // assert
+            Assert.Equal(expectedPackedValue, rgba.PackedValue);
+        }
+
         [Fact]
         public void Rgba1010102_Clamping()
         {

tests/ImageSharp.Tests/PixelFormats/RgbaVectorTests.cs

@@ -146,5 +146,47 @@ namespace SixLabors.ImageSharp.Tests.PixelFormats
             // assert
             Assert.Equal(expected, actual);
         }
+
+        [Fact]
+        public void RgbaVector_FromBgra5551()
+        {
+            // arrange
+            var rgb = default(RgbaVector);
+            Vector4 expected = Vector4.One;
+
+            // act
+            rgb.FromBgra5551(new Bgra5551(1.0f, 1.0f, 1.0f, 1.0f));
+
+            // assert
+            Assert.Equal(expected, rgb.ToScaledVector4());
+        }
+
+        [Fact]
+        public void RgbaVector_FromGrey16()
+        {
+            // arrange
+            var rgba = default(RgbaVector);
+            Vector4 expected = Vector4.One;
+
+            // act
+            rgba.FromGray16(new Gray16(ushort.MaxValue));
+
+            // assert
+            Assert.Equal(expected, rgba.ToScaledVector4());
+        }
+
+        [Fact]
+        public void RgbaVector_FromGrey8()
+        {
+            // arrange
+            var rgba = default(RgbaVector);
+            Vector4 expected = Vector4.One;
+
+            // act
+            rgba.FromGray8(new Gray8(byte.MaxValue));
+
+            // assert
+            Assert.Equal(expected, rgba.ToScaledVector4());
+        }
     }
 }

tests/ImageSharp.Tests/Processing/Processors/Convolution/Basic1ParameterConvolutionTests.cs

@@ -0,0 +1,56 @@
+// // Copyright (c) Six Labors and contributors.
+// // Licensed under the Apache License, Version 2.0.
+
+using SixLabors.ImageSharp.PixelFormats;
+using SixLabors.ImageSharp.Processing;
+using SixLabors.ImageSharp.Tests.TestUtilities.ImageComparison;
+using SixLabors.Primitives;
+
+using Xunit;
+
+namespace SixLabors.ImageSharp.Tests.Processing.Processors.Convolution
+{
+    [GroupOutput("Convolution")]
+    public abstract class Basic1ParameterConvolutionTests
+    {
+        private static readonly ImageComparer ValidatorComparer = ImageComparer.TolerantPercentage(0.05F);
+        
+        public static readonly TheoryData<int> Values = new TheoryData<int> { 3, 5 };
+        
+        public static readonly string[] InputImages =
+            {
+                TestImages.Bmp.Car,
+                TestImages.Png.CalliphoraPartial
+            };
+
+        [Theory]
+        [WithFileCollection(nameof(InputImages), nameof(Values), PixelTypes.Rgba32)]
+        public void OnFullImage<TPixel>(TestImageProvider<TPixel> provider, int value)
+            where TPixel : struct, IPixel<TPixel>
+        {
+            provider.Utility.TestGroupName = this.GetType().Name;
+            provider.RunValidatingProcessorTest(
+                x => this.Apply(x, value),
+                value,
+                ValidatorComparer);
+        }
+
+        [Theory]
+        [WithFileCollection(nameof(InputImages), nameof(Values), PixelTypes.Rgba32)]
+        public void InBox<TPixel>(TestImageProvider<TPixel> provider, int value)
+            where TPixel : struct, IPixel<TPixel>
+        {
+            provider.Utility.TestGroupName = this.GetType().Name;
+            provider.RunRectangleConstrainedValidatingProcessorTest(
+                (x, rect) => this.Apply(x, value, rect),
+                value,
+                ValidatorComparer);
+        }
+
+        protected abstract void Apply<TPixel>(IImageProcessingContext<TPixel> ctx, int value)
+            where TPixel : struct, IPixel<TPixel>;
+        
+        protected abstract void Apply<TPixel>(IImageProcessingContext<TPixel> ctx, int value, Rectangle bounds)
+            where TPixel : struct, IPixel<TPixel>;
+    }
+}

tests/ImageSharp.Tests/Processing/Processors/Convolution/BoxBlurTest.cs

@@ -1,51 +1,17 @@
 // Copyright (c) Six Labors and contributors.
 // Licensed under the Apache License, Version 2.0.
 
-using SixLabors.ImageSharp.PixelFormats;
 using SixLabors.ImageSharp.Processing;
-using SixLabors.ImageSharp.Tests.TestUtilities.ImageComparison;
-
 using SixLabors.Primitives;
-using Xunit;
 
 namespace SixLabors.ImageSharp.Tests.Processing.Processors.Convolution
 {
-    public class BoxBlurTest : FileTestBase
+    [GroupOutput("Convolution")]
+    public class BoxBlurTest : Basic1ParameterConvolutionTests
     {
-        public static readonly TheoryData<int> BoxBlurValues
+        protected override void Apply<TPixel>(IImageProcessingContext<TPixel> ctx, int value) => ctx.BoxBlur(value);
-        = new TheoryData<int>
-        {
-            3,
-            5
-        };
-
-        [Theory]
-        [WithFileCollection(nameof(DefaultFiles), nameof(BoxBlurValues), DefaultPixelType)]
-        public void ImageShouldApplyBoxBlurFilter<TPixel>(TestImageProvider<TPixel> provider, int value)
-            where TPixel : struct, IPixel<TPixel>
-        {
-            using (Image<TPixel> image = provider.GetImage())
-            {
-                image.Mutate(x => x.BoxBlur(value));
-                image.DebugSave(provider, value);
-            }
-        }
-
-        [Theory]
-        [WithFileCollection(nameof(DefaultFiles), nameof(BoxBlurValues), DefaultPixelType)]
-        public void ImageShouldApplyBoxBlurFilterInBox<TPixel>(TestImageProvider<TPixel> provider, int value)
-            where TPixel : struct, IPixel<TPixel>
-        {
-            using (Image<TPixel> source = provider.GetImage())
-            using (Image<TPixel> image = source.Clone())
-            {
-                var bounds = new Rectangle(10, 10, image.Width / 2, image.Height / 2);
-
-                image.Mutate(x => x.BoxBlur(value, bounds));
-                image.DebugSave(provider, value);
 
-                ImageComparer.Tolerant().VerifySimilarityIgnoreRegion(source, image, bounds);
+        protected override void Apply<TPixel>(IImageProcessingContext<TPixel> ctx, int value, Rectangle bounds) =>
-            }
+            ctx.BoxBlur(value, bounds);
-        }
     }
 }

tests/ImageSharp.Tests/Processing/Processors/Convolution/DetectEdgesTest.cs

@@ -10,13 +10,16 @@ using Xunit;
 
 namespace SixLabors.ImageSharp.Tests.Processing.Processors.Convolution
 {
-    public class DetectEdgesTest : FileTestBase
+    [GroupOutput("Convolution")]
+    public class DetectEdgesTest
     {
         // I think our comparison is not accurate enough (nor can be) for RgbaVector.
         // The image pixels are identical according to BeyondCompare.
         private static readonly ImageComparer ValidatorComparer = ImageComparer.TolerantPercentage(0.0456F);
 
-        public static readonly string[] CommonTestImages = { TestImages.Png.Bike };
+        public static readonly string[] TestImages = { Tests.TestImages.Png.Bike };
+        
+        public const PixelTypes CommonNonDefaultPixelTypes = PixelTypes.Rgba32 | PixelTypes.Bgra32 | PixelTypes.RgbaVector;
 
         public static readonly TheoryData<EdgeDetectionOperators> DetectEdgesFilters = new TheoryData<EdgeDetectionOperators>
         {
@@ -33,7 +36,7 @@ namespace SixLabors.ImageSharp.Tests.Processing.Processors.Convolution
         };
 
         [Theory]
-        [WithFileCollection(nameof(CommonTestImages), DefaultPixelType)]
+        [WithFileCollection(nameof(TestImages), PixelTypes.Rgba32)]
         public void DetectEdges_WorksOnWrappedMemoryImage<TPixel>(TestImageProvider<TPixel> provider)
             where TPixel : struct, IPixel<TPixel>
         {
@@ -49,8 +52,8 @@ namespace SixLabors.ImageSharp.Tests.Processing.Processors.Convolution
         }
 
         [Theory]
-        [WithTestPatternImages(nameof(DetectEdgesFilters), 100, 100, DefaultPixelType)]
+        [WithTestPatternImages(nameof(DetectEdgesFilters), 100, 100, PixelTypes.Rgba32)]
-        [WithFileCollection(nameof(CommonTestImages), nameof(DetectEdgesFilters), DefaultPixelType)]
+        [WithFileCollection(nameof(TestImages), nameof(DetectEdgesFilters), PixelTypes.Rgba32)]
         public void DetectEdges_WorksWithAllFilters<TPixel>(TestImageProvider<TPixel> provider, EdgeDetectionOperators detector)
             where TPixel : struct, IPixel<TPixel>
         {
@@ -63,7 +66,7 @@ namespace SixLabors.ImageSharp.Tests.Processing.Processors.Convolution
         }
 
         [Theory]
-        [WithFileCollection(nameof(CommonTestImages), CommonNonDefaultPixelTypes)]
+        [WithFileCollection(nameof(TestImages), CommonNonDefaultPixelTypes)]
         public void DetectEdges_IsNotBoundToSinglePixelType<TPixel>(TestImageProvider<TPixel> provider)
             where TPixel : struct, IPixel<TPixel>
         {
@@ -76,7 +79,7 @@ namespace SixLabors.ImageSharp.Tests.Processing.Processors.Convolution
         }
 
         [Theory]
-        [WithFile(TestImages.Gif.Giphy, DefaultPixelType)]
+        [WithFile(Tests.TestImages.Gif.Giphy, PixelTypes.Rgba32)]
         public void DetectEdges_IsAppliedToAllFrames<TPixel>(TestImageProvider<TPixel> provider)
             where TPixel : struct, IPixel<TPixel>
         {
@@ -88,7 +91,7 @@ namespace SixLabors.ImageSharp.Tests.Processing.Processors.Convolution
         }
 
         [Theory]
-        [WithFileCollection(nameof(CommonTestImages), DefaultPixelType)]
+        [WithFileCollection(nameof(TestImages), PixelTypes.Rgba32)]
         public void DetectEdges_InBox<TPixel>(TestImageProvider<TPixel> provider)
             where TPixel : struct, IPixel<TPixel>
         {

tests/ImageSharp.Tests/Processing/Processors/Convolution/GaussianBlurTest.cs

@@ -10,37 +10,12 @@ using Xunit;
 
 namespace SixLabors.ImageSharp.Tests.Processing.Processors.Convolution
 {
-    public class GaussianBlurTest : FileTestBase
+    [GroupOutput("Convolution")]
+    public class GaussianBlurTest : Basic1ParameterConvolutionTests
     {
-        public static readonly TheoryData<int> GaussianBlurValues = new TheoryData<int> { 3, 5 };
+        protected override void Apply<TPixel>(IImageProcessingContext<TPixel> ctx, int value) => ctx.GaussianBlur(value);
 
-        [Theory]
+        protected override void Apply<TPixel>(IImageProcessingContext<TPixel> ctx, int value, Rectangle bounds) =>
-        [WithFileCollection(nameof(DefaultFiles), nameof(GaussianBlurValues), DefaultPixelType)]
+            ctx.GaussianBlur(value, bounds);
-        public void ImageShouldApplyGaussianBlurFilter<TPixel>(TestImageProvider<TPixel> provider, int value)
-            where TPixel : struct, IPixel<TPixel>
-        {
-            using (Image<TPixel> image = provider.GetImage())
-            {
-                image.Mutate(x => x.GaussianBlur(value));
-                image.DebugSave(provider, value);
-            }
-        }
-
-        [Theory]
-        [WithFileCollection(nameof(DefaultFiles), nameof(GaussianBlurValues), DefaultPixelType)]
-        public void ImageShouldApplyGaussianBlurFilterInBox<TPixel>(TestImageProvider<TPixel> provider, int value)
-            where TPixel : struct, IPixel<TPixel>
-        {
-            using (Image<TPixel> source = provider.GetImage())
-            using (Image<TPixel> image = source.Clone())
-            {
-                var bounds = new Rectangle(10, 10, image.Width / 2, image.Height / 2);
-
-                image.Mutate(x => x.GaussianBlur(value, bounds));
-                image.DebugSave(provider, value);
-
-                ImageComparer.Tolerant().VerifySimilarityIgnoreRegion(source, image, bounds);
-            }
-        }
     }
 }

tests/ImageSharp.Tests/Processing/Processors/Convolution/GaussianSharpenTest.cs

@@ -9,42 +9,12 @@ using Xunit;
 
 namespace SixLabors.ImageSharp.Tests.Processing.Processors.Convolution
 {
-    public class GaussianSharpenTest : FileTestBase
+    [GroupOutput("Convolution")]
+    public class GaussianSharpenTest : Basic1ParameterConvolutionTests
     {
-        public static readonly TheoryData<int> GaussianSharpenValues
+        protected override void Apply<TPixel>(IImageProcessingContext<TPixel> ctx, int value) => ctx.GaussianSharpen(value);
-        = new TheoryData<int>
-        {
-            3,
-            5
-        };
 
-        [Theory]
+        protected override void Apply<TPixel>(IImageProcessingContext<TPixel> ctx, int value, Rectangle bounds) =>
-        [WithFileCollection(nameof(DefaultFiles), nameof(GaussianSharpenValues), DefaultPixelType)]
+            ctx.GaussianSharpen(value, bounds);
-        public void ImageShouldApplyGaussianSharpenFilter<TPixel>(TestImageProvider<TPixel> provider, int value)
-            where TPixel : struct, IPixel<TPixel>
-        {
-            using (Image<TPixel> image = provider.GetImage())
-            {
-                image.Mutate(x => x.GaussianSharpen(value));
-                image.DebugSave(provider, value);
-            }
-        }
-
-        [Theory]
-        [WithFileCollection(nameof(DefaultFiles), nameof(GaussianSharpenValues), DefaultPixelType)]
-        public void ImageShouldApplyGaussianSharpenFilterInBox<TPixel>(TestImageProvider<TPixel> provider, int value)
-            where TPixel : struct, IPixel<TPixel>
-        {
-            using (Image<TPixel> source = provider.GetImage())
-            using (var image = source.Clone())
-            {
-                var bounds = new Rectangle(10, 10, image.Width / 2, image.Height / 2);
-
-                image.Mutate(x => x.GaussianSharpen(value, bounds));
-                image.DebugSave(provider, value);
-
-                ImageComparer.Tolerant().VerifySimilarityIgnoreRegion(source, image, bounds);
-            }
-        }
     }
 }

tests/ImageSharp.Tests/Processing/Processors/Dithering/DitherTests.cs

@@ -2,129 +2,153 @@
 // Licensed under the Apache License, Version 2.0.
 
 using SixLabors.ImageSharp.PixelFormats;
-using SixLabors.ImageSharp.Tests.TestUtilities.ImageComparison;
+using SixLabors.ImageSharp.Primitives;
 using SixLabors.ImageSharp.Processing;
 using SixLabors.ImageSharp.Processing.Dithering;
 using SixLabors.ImageSharp.Processing.Processors.Dithering;
+using SixLabors.ImageSharp.Tests.TestUtilities.ImageComparison;
 using SixLabors.Primitives;
+
 using Xunit;
 
 namespace SixLabors.ImageSharp.Tests.Processing.Processors.Binarization
 {
-    public class DitherTests : FileTestBase
+    public class DitherTests
     {
-        public static readonly string[] CommonTestImages =
+        public const PixelTypes CommonNonDefaultPixelTypes =
-            {
+            PixelTypes.Rgba32 | PixelTypes.Bgra32 | PixelTypes.Rgb24 | PixelTypes.RgbaVector;
-                TestImages.Png.CalliphoraPartial, TestImages.Png.Bike
+        
-            };
+        public static readonly string[] CommonTestImages = { TestImages.Png.CalliphoraPartial, TestImages.Png.Bike };
-
+
-        public static readonly TheoryData<string, IOrderedDither> OrderedDitherers = new TheoryData<string, IOrderedDither>
+        public static readonly TheoryData<IErrorDiffuser> ErrorDiffusers = new TheoryData<IErrorDiffuser>
-        {
+                                                                               {
-            { "Bayer8x8", KnownDitherers.BayerDither8x8 },
+                                                                                   KnownDiffusers.Atkinson,
-            { "Bayer4x4", KnownDitherers.BayerDither4x4 },
+                                                                                   KnownDiffusers.Burks,
-            { "Ordered3x3", KnownDitherers.OrderedDither3x3 },
+                                                                                   KnownDiffusers.FloydSteinberg,
-            { "Bayer2x2", KnownDitherers.BayerDither2x2 }
+                                                                                   KnownDiffusers.JarvisJudiceNinke,
-        };
+                                                                                   KnownDiffusers.Sierra2,
-
+                                                                                   KnownDiffusers.Sierra3,
-        public static readonly TheoryData<string, IErrorDiffuser> ErrorDiffusers = new TheoryData<string, IErrorDiffuser>
+                                                                                   KnownDiffusers.SierraLite,
-        {
+                                                                                   KnownDiffusers.StevensonArce,
-            { "Atkinson", KnownDiffusers.Atkinson },
+                                                                                   KnownDiffusers.Stucki,
-            { "Burks", KnownDiffusers.Burks },
+                                                                               };
-            { "FloydSteinberg", KnownDiffusers.FloydSteinberg },
+
-            { "JarvisJudiceNinke", KnownDiffusers.JarvisJudiceNinke },
+        public static readonly TheoryData<IOrderedDither> OrderedDitherers = new TheoryData<IOrderedDither>
-            { "Sierra2", KnownDiffusers.Sierra2 },
+                                                                                 {
-            { "Sierra3", KnownDiffusers.Sierra3 },
+                                                                                     KnownDitherers.BayerDither8x8,
-            { "SierraLite", KnownDiffusers.SierraLite },
+                                                                                     KnownDitherers.BayerDither4x4,
-            { "StevensonArce", KnownDiffusers.StevensonArce },
+                                                                                     KnownDitherers.OrderedDither3x3,
-            { "Stucki", KnownDiffusers.Stucki },
+                                                                                     KnownDitherers.BayerDither2x2
-        };
+                                                                                 };
-
+        private static readonly ImageComparer ValidatorComparer = ImageComparer.TolerantPercentage(0.05f);
+        
         private static IOrderedDither DefaultDitherer => KnownDitherers.BayerDither4x4;
 
         private static IErrorDiffuser DefaultErrorDiffuser => KnownDiffusers.Atkinson;
 
+        /// <summary>
+        /// The output is visually correct old 32bit runtime,
+        /// but it is very different because of floating point inaccuracies.
+        /// </summary>
+        private static readonly bool SkipAllDitherTests =
+            !TestEnvironment.Is64BitProcess && string.IsNullOrEmpty(TestEnvironment.NetCoreVersion);
+
         [Theory]
-        [WithFileCollection(nameof(CommonTestImages), nameof(OrderedDitherers), DefaultPixelType)]
+        [WithFile(TestImages.Png.CalliphoraPartial, PixelTypes.Rgba32)]
-        [WithTestPatternImages(nameof(OrderedDitherers), 100, 100, DefaultPixelType)]
+        public void ApplyDiffusionFilterInBox<TPixel>(TestImageProvider<TPixel> provider)
-        public void DitherFilter_WorksWithAllDitherers<TPixel>(TestImageProvider<TPixel> provider, string name, IOrderedDither ditherer)
             where TPixel : struct, IPixel<TPixel>
         {
-            using (Image<TPixel> image = provider.GetImage())
+            if (SkipAllDitherTests)
             {
-                image.Mutate(x => x.Dither(ditherer));
+                return;
-                image.DebugSave(provider, name);
             }
+            
+            provider.RunRectangleConstrainedValidatingProcessorTest(
+                (x, rect) => x.Diffuse(DefaultErrorDiffuser, .5F, rect),
+                comparer: ValidatorComparer);
         }
 
         [Theory]
-        [WithFileCollection(nameof(CommonTestImages), nameof(ErrorDiffusers), DefaultPixelType)]
+        [WithFile(TestImages.Png.CalliphoraPartial, PixelTypes.Rgba32)]
-        [WithTestPatternImages(nameof(ErrorDiffusers), 100, 100, DefaultPixelType)]
+        public void ApplyDitherFilterInBox<TPixel>(TestImageProvider<TPixel> provider)
-        public void DiffusionFilter_WorksWithAllErrorDiffusers<TPixel>(TestImageProvider<TPixel> provider, string name, IErrorDiffuser diffuser)
             where TPixel : struct, IPixel<TPixel>
         {
-            using (Image<TPixel> image = provider.GetImage())
+            if (SkipAllDitherTests)
             {
-                image.Mutate(x => x.Diffuse(diffuser, .5F));
+                return;
-                image.DebugSave(provider, name);
             }
+            
+            provider.RunRectangleConstrainedValidatingProcessorTest(
+                (x, rect) => x.Dither(DefaultDitherer, rect),
+                comparer: ValidatorComparer);
         }
 
         [Theory]
         [WithFile(TestImages.Png.Filter0, CommonNonDefaultPixelTypes)]
-        public void DitherFilter_ShouldNotDependOnSinglePixelType<TPixel>(TestImageProvider<TPixel> provider)
+        public void DiffusionFilter_ShouldNotDependOnSinglePixelType<TPixel>(TestImageProvider<TPixel> provider)
             where TPixel : struct, IPixel<TPixel>
         {
-            using (Image<TPixel> image = provider.GetImage())
+            if (SkipAllDitherTests)
             {
-                image.Mutate(x => x.Dither(DefaultDitherer));
+                return;
-                image.DebugSave(provider);
             }
+            
+            // Increased tolerance because of compatibility issues on .NET 4.6.2:
+            var comparer = ImageComparer.TolerantPercentage(1f);
+            provider.RunValidatingProcessorTest(x => x.Diffuse(DefaultErrorDiffuser, 0.5f), comparer: comparer);
         }
 
         [Theory]
-        [WithFile(TestImages.Png.Filter0, CommonNonDefaultPixelTypes)]
+        [WithFileCollection(nameof(CommonTestImages), nameof(ErrorDiffusers), PixelTypes.Rgba32)]
-        public void DiffusionFilter_ShouldNotDependOnSinglePixelType<TPixel>(TestImageProvider<TPixel> provider)
+        public void DiffusionFilter_WorksWithAllErrorDiffusers<TPixel>(
+            TestImageProvider<TPixel> provider,
+            IErrorDiffuser diffuser)
             where TPixel : struct, IPixel<TPixel>
         {
-            using (Image<TPixel> image = provider.GetImage())
+            if (SkipAllDitherTests)
             {
-                image.Mutate(x => x.Diffuse(DefaultErrorDiffuser, 0.5f));
+                return;
-                image.DebugSave(provider);
             }
+            
+            provider.RunValidatingProcessorTest(
+                x => x.Diffuse(diffuser, 0.5f),
+                testOutputDetails: diffuser.GetType().Name,
+                comparer: ValidatorComparer,
+                appendPixelTypeToFileName: false);
         }
 
         [Theory]
-        [WithFile(TestImages.Png.CalliphoraPartial, DefaultPixelType)]
+        [WithFile(TestImages.Png.Filter0, CommonNonDefaultPixelTypes)]
-        public void ApplyDitherFilterInBox<TPixel>(TestImageProvider<TPixel> provider)
+        public void DitherFilter_ShouldNotDependOnSinglePixelType<TPixel>(TestImageProvider<TPixel> provider)
             where TPixel : struct, IPixel<TPixel>
         {
-            using (Image<TPixel> source = provider.GetImage())
+            if (SkipAllDitherTests)
-            using (Image<TPixel> image = source.Clone())
             {
-                var bounds = new Rectangle(10, 10, image.Width / 2, image.Height / 2);
+                return;
-
-                image.Mutate(x => x.Dither(DefaultDitherer, bounds));
-                image.DebugSave(provider);
-
-                ImageComparer.Tolerant().VerifySimilarityIgnoreRegion(source, image, bounds);
             }
+            
+            provider.RunValidatingProcessorTest(
+                x => x.Dither(DefaultDitherer),
+                comparer: ValidatorComparer);
         }
 
         [Theory]
-        [WithFile(TestImages.Png.CalliphoraPartial, DefaultPixelType)]
+        [WithFileCollection(nameof(CommonTestImages), nameof(OrderedDitherers), PixelTypes.Rgba32)]
-        public void ApplyDiffusionFilterInBox<TPixel>(TestImageProvider<TPixel> provider)
+        public void DitherFilter_WorksWithAllDitherers<TPixel>(
+            TestImageProvider<TPixel> provider,
+            IOrderedDither ditherer)
             where TPixel : struct, IPixel<TPixel>
         {
-            using (Image<TPixel> source = provider.GetImage())
+            if (SkipAllDitherTests)
-            using (Image<TPixel> image = source.Clone())
             {
-                var bounds = new Rectangle(10, 10, image.Width / 2, image.Height / 2);
+                return;
-
-                image.Mutate(x => x.Diffuse(DefaultErrorDiffuser, .5F, bounds));
-                image.DebugSave(provider);
-
-                ImageComparer.Tolerant().VerifySimilarityIgnoreRegion(source, image, bounds);
             }
+            
+            provider.RunValidatingProcessorTest(
+                x => x.Dither(ditherer),
+                testOutputDetails: ditherer.GetType().Name,
+                comparer: ValidatorComparer,
+                appendPixelTypeToFileName: false);
         }
     }
 }

tests/ImageSharp.Tests/Processing/Processors/Effects/BackgroundColorTest.cs

@@ -10,35 +10,30 @@ using Xunit;
 
 namespace SixLabors.ImageSharp.Tests.Processing.Processors.Effects
 {
-    public class BackgroundColorTest : FileTestBase
+    [GroupOutput("Effects")]
+    public class BackgroundColorTest
     {
+        public static readonly string[] InputImages =
+            {
+                TestImages.Png.Splash,
+                TestImages.Png.Ducky
+            };
+        
         [Theory]
-        [WithFileCollection(nameof(DefaultFiles), DefaultPixelType)]
+        [WithFileCollection(nameof(InputImages), PixelTypes.Rgba32)]
-        public void ImageShouldApplyBackgroundColorFilter<TPixel>(TestImageProvider<TPixel> provider)
+        public void FullImage<TPixel>(TestImageProvider<TPixel> provider)
             where TPixel : struct, IPixel<TPixel>
         {
-            using (Image<TPixel> image = provider.GetImage())
+            provider.RunValidatingProcessorTest(x =>  x.BackgroundColor(NamedColors<TPixel>.HotPink));
-            {
-                image.Mutate(x => x.BackgroundColor(NamedColors<TPixel>.HotPink));
-                image.DebugSave(provider);
-            }
         }
 
         [Theory]
-        [WithFileCollection(nameof(DefaultFiles), DefaultPixelType)]
+        [WithFileCollection(nameof(InputImages), PixelTypes.Rgba32)]
-        public void ImageShouldApplyBackgroundColorFilterInBox<TPixel>(TestImageProvider<TPixel> provider)
+        public void InBox<TPixel>(TestImageProvider<TPixel> provider)
             where TPixel : struct, IPixel<TPixel>
         {
-            using (Image<TPixel> source = provider.GetImage())
+            provider.RunRectangleConstrainedValidatingProcessorTest(
-            using (var image = source.Clone())
+                (x, rect) => x.BackgroundColor(NamedColors<TPixel>.HotPink, rect));
-            {
-                var bounds = new Rectangle(10, 10, image.Width / 2, image.Height / 2);
-
-                image.Mutate(x => x.BackgroundColor(NamedColors<TPixel>.HotPink, bounds));
-                image.DebugSave(provider);
-
-                ImageComparer.Tolerant().VerifySimilarityIgnoreRegion(source, image, bounds);
-            }
         }
     }
 }

tests/ImageSharp.Tests/Processing/Processors/Effects/OilPaintTest.cs

@@ -10,40 +10,40 @@ using Xunit;
 
 namespace SixLabors.ImageSharp.Tests.Processing.Processors.Effects
 {
-    public class OilPaintTest : FileTestBase
+    [GroupOutput("Effects")]
+    public class OilPaintTest
     {
         public static readonly TheoryData<int, int> OilPaintValues = new TheoryData<int, int>
                                                                          {
-                                                                             { 15, 10 }, { 6, 5 }
+                                                                             { 15, 10 }, 
+                                                                             { 6, 5 }
                                                                          };
+        public static readonly string[] InputImages =
+            {
+                TestImages.Png.CalliphoraPartial,
+                TestImages.Bmp.Car
+            };
 
         [Theory]
-        [WithFileCollection(nameof(DefaultFiles), nameof(OilPaintValues), DefaultPixelType)]
+        [WithFileCollection(nameof(InputImages), nameof(OilPaintValues), PixelTypes.Rgba32)]
-        public void ApplyOilPaintFilter<TPixel>(TestImageProvider<TPixel> provider, int levels, int brushSize)
+        public void FullImage<TPixel>(TestImageProvider<TPixel> provider, int levels, int brushSize)
             where TPixel : struct, IPixel<TPixel>
         {
-            using (Image<TPixel> image = provider.GetImage())
+            provider.RunValidatingProcessorTest(
-            {
+                x => x.OilPaint(levels, brushSize),
-                image.Mutate(x => x.OilPaint(levels, brushSize));
+                $"{levels}-{brushSize}",
-                image.DebugSave(provider, string.Join("-", levels, brushSize));
+                appendPixelTypeToFileName: false);
-            }
         }
 
         [Theory]
-        [WithFileCollection(nameof(DefaultFiles), nameof(OilPaintValues), DefaultPixelType)]
+        [WithFileCollection(nameof(InputImages), nameof(OilPaintValues), PixelTypes.Rgba32)]
-        public void ApplyOilPaintFilterInBox<TPixel>(TestImageProvider<TPixel> provider, int levels, int brushSize)
+        [WithTestPatternImages(nameof(OilPaintValues), 100, 100, PixelTypes.Rgba32)]
+        public void InBox<TPixel>(TestImageProvider<TPixel> provider, int levels, int brushSize)
             where TPixel : struct, IPixel<TPixel>
         {
-            using (Image<TPixel> source = provider.GetImage())
+            provider.RunRectangleConstrainedValidatingProcessorTest(
-            using (Image<TPixel> image = source.Clone())
+                (x, rect) => x.OilPaint(levels, brushSize, rect),
-            {
+                $"{levels}-{brushSize}");
-                var bounds = new Rectangle(10, 10, image.Width / 2, image.Height / 2);
-
-                image.Mutate(x => x.OilPaint(levels, brushSize, bounds));
-                image.DebugSave(provider, string.Join("-", levels, brushSize));
-
-                ImageComparer.Tolerant().VerifySimilarityIgnoreRegion(source, image, bounds);
-            }
         }
     }
 }

tests/ImageSharp.Tests/Processing/Processors/Effects/PixelateTest.cs

@@ -4,82 +4,32 @@
 using SixLabors.ImageSharp.PixelFormats;
 using SixLabors.ImageSharp.Processing;
 using SixLabors.ImageSharp.Tests.TestUtilities.ImageComparison;
-
 using SixLabors.Primitives;
+
 using Xunit;
 
 namespace SixLabors.ImageSharp.Tests.Processing.Processors.Effects
 {
-    public class PixelateTest : FileTestBase
+    [GroupOutput("Effects")]
+    public class PixelateTest
     {
-        public static readonly TheoryData<int> PixelateValues
+        public static readonly TheoryData<int> PixelateValues = new TheoryData<int> { 4, 8 };
-        = new TheoryData<int>
-        {
-            4 ,
-            8
-        };
 
         [Theory]
-        [WithTestPatternImages(nameof(PixelateValues), 320, 240, PixelTypes.Rgba32)]
+        [WithFile(TestImages.Png.Ducky, nameof(PixelateValues), PixelTypes.Rgba32)]
-        public void ImageShouldApplyPixelateFilter<TPixel>(TestImageProvider<TPixel> provider, int value)
+        public void FullImage<TPixel>(TestImageProvider<TPixel> provider, int value)
             where TPixel : struct, IPixel<TPixel>
         {
-            using (Image<TPixel> image = provider.GetImage())
+            provider.RunValidatingProcessorTest(x => x.Pixelate(value), value, appendPixelTypeToFileName: false);
-            {
-                image.Mutate(x => x.Pixelate(value));
-                image.DebugSave(provider, value);
-
-                // Test the neigbouring pixels
-                for (int y = 0; y < image.Height; y += value)
-                {
-                    for (int x = 0; x < image.Width; x += value)
-                    {
-                        TPixel source = image[x, y];
-                        for (int pixY = y; pixY < y + value && pixY < image.Height; pixY++)
-                        {
-                            for (int pixX = x; pixX < x + value && pixX < image.Width; pixX++)
-                            {
-                                Assert.Equal(source, image[pixX, pixY]);
-                            }
-                        }
-                    }
-                }
-            }
         }
 
         [Theory]
         [WithTestPatternImages(nameof(PixelateValues), 320, 240, PixelTypes.Rgba32)]
-        public void ImageShouldApplyPixelateFilterInBox<TPixel>(TestImageProvider<TPixel> provider, int value)
+        [WithFile(TestImages.Png.CalliphoraPartial, nameof(PixelateValues), PixelTypes.Rgba32)]
+        public void InBox<TPixel>(TestImageProvider<TPixel> provider, int value)
             where TPixel : struct, IPixel<TPixel>
         {
-            using (Image<TPixel> source = provider.GetImage())
+            provider.RunRectangleConstrainedValidatingProcessorTest((x, rect) => x.Pixelate(value, rect), value);
-            using (var image = source.Clone())
-            {
-                var bounds = new Rectangle(image.Width / 4, image.Height / 4, image.Width / 2, image.Height / 2);
-
-                image.Mutate(x => x.Pixelate(value, bounds));
-                image.DebugSave(provider, value);
-
-                for (int y = 0; y < image.Height; y++)
-                {
-                    for (int x = 0; x < image.Width; x++)
-                    {
-                        int tx = x;
-                        int ty = y;
-                        TPixel sourceColor = source[tx, ty];
-                        if (bounds.Contains(tx, ty))
-                        {
-                            int sourceX = tx - ((tx - bounds.Left) % value) + (value / 2);
-                            int sourceY = ty - ((ty - bounds.Top) % value) + (value / 2);
-
-                            sourceColor = image[sourceX, sourceY];
-                        }
-                        Assert.Equal(sourceColor, image[tx, ty]);
-                    }
-                }
-
-                ImageComparer.Tolerant().VerifySimilarityIgnoreRegion(source, image, bounds);
-            }
         }
     }
 }

tests/ImageSharp.Tests/Processing/Processors/Overlays/GlowTest.cs

@@ -10,59 +10,14 @@ using Xunit;
 
 namespace SixLabors.ImageSharp.Tests.Processing.Processors.Overlays
 {
-    public class GlowTest : FileTestBase
+    [GroupOutput("Overlays")]
+    public class GlowTest : OverlayTestBase
     {
-        [Theory]
+        protected override void Apply<T>(IImageProcessingContext<T> ctx, T color) => ctx.Glow(color);
-        [WithFileCollection(nameof(DefaultFiles), DefaultPixelType)]
-        public void ImageShouldApplyGlowFilter<TPixel>(TestImageProvider<TPixel> provider)
-            where TPixel : struct, IPixel<TPixel>
-        {
-            using (Image<TPixel> image = provider.GetImage())
-            {
-                image.Mutate(x => x.Glow());
-                image.DebugSave(provider);
-            }
-        }
 
-        [Theory]
+        protected override void Apply<T>(IImageProcessingContext<T> ctx, float radiusX, float radiusY) =>
-        [WithFileCollection(nameof(DefaultFiles), DefaultPixelType)]
+            ctx.Glow(radiusX);
-        public void ImageShouldApplyGlowFilterColor<TPixel>(TestImageProvider<TPixel> provider)
-            where TPixel : struct, IPixel<TPixel>
-        {
-            using (Image<TPixel> image = provider.GetImage())
-            {
-                image.Mutate(x => x.Glow(NamedColors<TPixel>.Orange));
-                image.DebugSave(provider);
-            }
-        }
 
-        [Theory]
+        protected override void Apply<T>(IImageProcessingContext<T> ctx, Rectangle rect) => ctx.Glow(rect);
-        [WithFileCollection(nameof(DefaultFiles), DefaultPixelType)]
-        public void ImageShouldApplyGlowFilterRadius<TPixel>(TestImageProvider<TPixel> provider)
-            where TPixel : struct, IPixel<TPixel>
-        {
-            using (Image<TPixel> image = provider.GetImage())
-            {
-                image.Mutate(x => x.Glow(image.Width / 4F));
-                image.DebugSave(provider);
-            }
-        }
-
-        [Theory]
-        [WithFileCollection(nameof(DefaultFiles), DefaultPixelType)]
-        public void ImageShouldApplyGlowFilterInBox<TPixel>(TestImageProvider<TPixel> provider)
-            where TPixel : struct, IPixel<TPixel>
-        {
-            using (Image<TPixel> source = provider.GetImage())
-            using (var image = source.Clone())
-            {
-                var bounds = new Rectangle(10, 10, image.Width / 2, image.Height / 2);
-
-                image.Mutate(x => x.Glow(bounds));
-                image.DebugSave(provider);
-
-                ImageComparer.Tolerant().VerifySimilarityIgnoreRegion(source, image, bounds);
-            }
-        }
     }
 }

tests/ImageSharp.Tests/Processing/Processors/Overlays/OverlayTestBase.cs

@@ -0,0 +1,70 @@
+// Copyright (c) Six Labors and contributors.
+// Licensed under the Apache License, Version 2.0.
+
+using System.Reflection;
+
+using SixLabors.ImageSharp.PixelFormats;
+using SixLabors.ImageSharp.Processing;
+using SixLabors.ImageSharp.Tests.TestUtilities.ImageComparison;
+using SixLabors.Primitives;
+
+using Xunit;
+
+namespace SixLabors.ImageSharp.Tests.Processing.Processors.Overlays
+{
+    [GroupOutput("Overlays")]
+    public abstract class OverlayTestBase
+    {
+        public static string[] ColorNames = { "Blue", "White" };
+
+        public static string[] InputImages = { TestImages.Png.Ducky, TestImages.Png.Splash };
+
+        private static readonly ImageComparer ValidatorComparer = ImageComparer.TolerantPercentage(0.05f);
+        
+        [Theory]
+        [WithFileCollection(nameof(InputImages), nameof(ColorNames), PixelTypes.Rgba32)]
+        public void FullImage_ApplyColor<TPixel>(TestImageProvider<TPixel> provider, string colorName)
+            where TPixel : struct, IPixel<TPixel>
+        {
+            provider.Utility.TestGroupName = this.GetType().Name;
+            var f = (FieldInfo)typeof(NamedColors<TPixel>).GetMember(colorName)[0];
+            TPixel color = (TPixel)f.GetValue(null);
+
+            provider.RunValidatingProcessorTest(x => this.Apply(x, color), colorName, ValidatorComparer, appendPixelTypeToFileName: false);
+        }
+
+        [Theory]
+        [WithFileCollection(nameof(InputImages), PixelTypes.Rgba32)]
+        public void FullImage_ApplyRadius<TPixel>(TestImageProvider<TPixel> provider)
+            where TPixel : struct, IPixel<TPixel>
+        {
+            provider.Utility.TestGroupName = this.GetType().Name;
+            provider.RunValidatingProcessorTest(
+                x =>
+                    {
+                        Size size = x.GetCurrentSize();
+                        this.Apply(x, size.Width / 4f, size.Height / 4f);
+                    },
+                comparer: ValidatorComparer,
+                appendPixelTypeToFileName: false);
+        }
+
+        [Theory]
+        [WithFileCollection(nameof(InputImages), PixelTypes.Rgba32)]
+        public void InBox<TPixel>(TestImageProvider<TPixel> provider)
+            where TPixel : struct, IPixel<TPixel>
+        {
+            provider.Utility.TestGroupName = this.GetType().Name;
+            provider.RunRectangleConstrainedValidatingProcessorTest((x, rect) => this.Apply(x, rect));
+        }
+
+        protected abstract void Apply<T>(IImageProcessingContext<T> ctx, T color)
+            where T : struct, IPixel<T>;
+        
+        protected abstract void Apply<T>(IImageProcessingContext<T> ctx, float radiusX, float radiusY)
+            where T : struct, IPixel<T>;
+        
+        protected abstract void Apply<T>(IImageProcessingContext<T> ctx, Rectangle rect)
+            where T : struct, IPixel<T>;
+    }
+}

tests/ImageSharp.Tests/Processing/Processors/Overlays/VignetteTest.cs

@@ -1,68 +1,19 @@
 // Copyright (c) Six Labors and contributors.
 // Licensed under the Apache License, Version 2.0.
 
-using SixLabors.ImageSharp.PixelFormats;
 using SixLabors.ImageSharp.Processing;
-using SixLabors.ImageSharp.Tests.TestUtilities.ImageComparison;
-
 using SixLabors.Primitives;
-using Xunit;
 
 namespace SixLabors.ImageSharp.Tests.Processing.Processors.Overlays
 {
-    public class VignetteTest : FileTestBase
+    [GroupOutput("Overlays")]
+    public class VignetteTest : OverlayTestBase
     {
-        [Theory]
+        protected override void Apply<T>(IImageProcessingContext<T> ctx, T color) => ctx.Vignette(color);
-        [WithFileCollection(nameof(DefaultFiles), DefaultPixelType)]
-        public void ImageShouldApplyVignetteFilter<TPixel>(TestImageProvider<TPixel> provider)
-            where TPixel : struct, IPixel<TPixel>
-        {
-            using (Image<TPixel> image = provider.GetImage())
-            {
-                image.Mutate(x => x.Vignette());
-                image.DebugSave(provider);
-            }
-        }
-
-        [Theory]
-        [WithFileCollection(nameof(DefaultFiles), DefaultPixelType)]
-        public void ImageShouldApplyVignetteFilterColor<TPixel>(TestImageProvider<TPixel> provider)
-            where TPixel : struct, IPixel<TPixel>
-        {
-            using (Image<TPixel> image = provider.GetImage())
-            {
-                image.Mutate(x => x.Vignette(NamedColors<TPixel>.Orange));
-                image.DebugSave(provider);
-            }
-        }
-
-        [Theory]
-        [WithFileCollection(nameof(DefaultFiles), DefaultPixelType)]
-        public void ImageShouldApplyVignetteFilterRadius<TPixel>(TestImageProvider<TPixel> provider)
-            where TPixel : struct, IPixel<TPixel>
-        {
-            using (Image<TPixel> image = provider.GetImage())
-            {
-                image.Mutate(x => x.Vignette(image.Width / 4F, image.Height / 4F));
-                image.DebugSave(provider);
-            }
-        }
-
-        [Theory]
-        [WithFileCollection(nameof(DefaultFiles), DefaultPixelType)]
-        public void ImageShouldApplyVignetteFilterInBox<TPixel>(TestImageProvider<TPixel> provider)
-            where TPixel : struct, IPixel<TPixel>
-        {
-            using (Image<TPixel> source = provider.GetImage())
-            using (var image = source.Clone())
-            {
-                var bounds = new Rectangle(10, 10, image.Width / 2, image.Height / 2);
 
-                image.Mutate(x => x.Vignette(bounds));
+        protected override void Apply<T>(IImageProcessingContext<T> ctx, float radiusX, float radiusY) =>
-                image.DebugSave(provider);
+            ctx.Vignette(radiusX, radiusY);
 
-                ImageComparer.Tolerant().VerifySimilarityIgnoreRegion(source, image, bounds);
+        protected override void Apply<T>(IImageProcessingContext<T> ctx, Rectangle rect) => ctx.Vignette(rect);
-            }
-        }
     }
 }

tests/ImageSharp.Tests/Processing/Processors/Transforms/AutoOrientTests.cs

@@ -6,27 +6,16 @@ using SixLabors.ImageSharp.Metadata.Profiles.Exif;
 using SixLabors.ImageSharp.PixelFormats;
 using SixLabors.ImageSharp.Processing;
 using SixLabors.ImageSharp.Formats.Jpeg.Components.Decoder;
+using SixLabors.ImageSharp.Tests.TestUtilities.ImageComparison;
+
 using Xunit;
 
 namespace SixLabors.ImageSharp.Tests.Processing.Processors.Transforms
 {
-    public class AutoOrientTests : FileTestBase
+    [GroupOutput("Transforms")]
+    public class AutoOrientTests
     {
-        public static readonly string[] FlipFiles = { TestImages.Bmp.F };
+        public const string FlipTestFile = TestImages.Bmp.F;
-
-        public static readonly TheoryData<RotateMode, FlipMode, ushort> OrientationValues
-            = new TheoryData<RotateMode, FlipMode, ushort>
-        {
-            { RotateMode.None,      FlipMode.None,       0 },
-            { RotateMode.None,      FlipMode.None,       1 },
-            { RotateMode.None,      FlipMode.Horizontal, 2 },
-            { RotateMode.Rotate180, FlipMode.None,       3 },
-            { RotateMode.Rotate180, FlipMode.Horizontal, 4 },
-            { RotateMode.Rotate90,  FlipMode.Horizontal, 5 },
-            { RotateMode.Rotate270, FlipMode.None,       6 },
-            { RotateMode.Rotate90,  FlipMode.Vertical,   7 },
-            { RotateMode.Rotate90,  FlipMode.None,       8 },
-        };
 
         public static readonly TheoryData<ExifDataType, byte[]> InvalidOrientationValues
             = new TheoryData<ExifDataType, byte[]>
@@ -38,27 +27,38 @@ namespace SixLabors.ImageSharp.Tests.Processing.Processors.Transforms
             { ExifDataType.SignedLong, BitConverter.GetBytes((int) 5) }
         };
 
+        public static readonly TheoryData<ushort> ExifOrientationValues = new TheoryData<ushort>()
+                                                                              {
+                                                                                  0,
+                                                                                  1,
+                                                                                  2,
+                                                                                  3,
+                                                                                  4,
+                                                                                  5,
+                                                                                  6,
+                                                                                  7,
+                                                                                  8
+                                                                              };
+
         [Theory]
-        [WithFileCollection(nameof(FlipFiles), nameof(OrientationValues), DefaultPixelType)]
+        [WithFile(FlipTestFile, nameof(ExifOrientationValues), PixelTypes.Rgba32)]
-        public void ImageShouldAutoRotate<TPixel>(TestImageProvider<TPixel> provider, RotateMode rotateType, FlipMode flipType, ushort orientation)
+        public void AutoOrient_WorksForAllExifOrientations<TPixel>(TestImageProvider<TPixel> provider, ushort orientation)
             where TPixel : struct, IPixel<TPixel>
         {
             using (Image<TPixel> image = provider.GetImage())
             {
                 image.Metadata.ExifProfile = new ExifProfile();
                 image.Metadata.ExifProfile.SetValue(ExifTag.Orientation, orientation);
-
+                
-                image.Mutate(x => x.RotateFlip(rotateType, flipType));
-                image.DebugSave(provider, string.Join("_", rotateType, flipType, orientation, "1_before"));
-
                 image.Mutate(x => x.AutoOrient());
-                image.DebugSave(provider, string.Join("_", rotateType, flipType, orientation, "2_after"));
+                image.DebugSave(provider, orientation, appendPixelTypeToFileName: false);
+                image.CompareToReferenceOutput(provider, orientation, appendPixelTypeToFileName: false);
             }
         }
 
         [Theory]
-        [WithFileCollection(nameof(FlipFiles), nameof(InvalidOrientationValues), DefaultPixelType)]
+        [WithFile(FlipTestFile, nameof(InvalidOrientationValues), PixelTypes.Rgba32)]
-        public void ImageShouldAutoRotateInvalidValues<TPixel>(TestImageProvider<TPixel> provider, ExifDataType dataType, byte[] orientation)
+        public void AutoOrient_WorksWithCorruptExifData<TPixel>(TestImageProvider<TPixel> provider, ExifDataType dataType, byte[] orientation)
             where TPixel : struct, IPixel<TPixel>
         {
             var profile = new ExifProfile();
@@ -72,11 +72,19 @@ namespace SixLabors.ImageSharp.Tests.Processing.Processors.Transforms
             bytes[18] = (byte)dataType;
             // Change the number of components
             bytes[20] = 1;
+            
+            byte[] orientationCodeData = new byte[8];
+            Array.Copy(orientation, orientationCodeData, orientation.Length);
+
+            ulong orientationCode = BitConverter.ToUInt64(orientationCodeData, 0);
 
             using (Image<TPixel> image = provider.GetImage())
+            using (Image<TPixel> reference = image.Clone())
             {
                 image.Metadata.ExifProfile = new ExifProfile(bytes);
                 image.Mutate(x => x.AutoOrient());
+                image.DebugSave(provider, $"{dataType}-{orientationCode}", appendPixelTypeToFileName: false);
+                ImageComparer.Exact.VerifySimilarity(image, reference);
             }
         }
     }

tests/ImageSharp.Tests/Processing/Processors/Transforms/CropTest.cs

@@ -13,7 +13,7 @@ using Xunit;
 namespace SixLabors.ImageSharp.Tests.Processing.Processors.Transforms
 {
     [GroupOutput("Transforms")]
-    public class CropTest : FileTestBase
+    public class CropTest
     {
         [Theory]
         [WithTestPatternImages(70, 30, PixelTypes.Rgba32, 0, 0, 70, 30)]

tests/ImageSharp.Tests/Processing/Processors/Transforms/EntropyCropTest.cs

@@ -7,25 +7,24 @@ using Xunit;
 
 namespace SixLabors.ImageSharp.Tests.Processing.Processors.Transforms
 {
-    public class EntropyCropTest : FileTestBase
+    [GroupOutput("Transforms")]
+    public class EntropyCropTest
     {
-        public static readonly TheoryData<float> EntropyCropValues
+        public static readonly TheoryData<float> EntropyCropValues = new TheoryData<float> { .25F, .75F };
-        = new TheoryData<float>
+
-        {
+        public static readonly string[] InputImages =
-            .25F,
+            {
-            .75F
+                TestImages.Png.Ducky,
-        };
+                TestImages.Jpeg.Baseline.Jpeg400,
+                TestImages.Jpeg.Baseline.MultiScanBaselineCMYK
+            };
 
         [Theory]
-        [WithFileCollection(nameof(DefaultFiles), nameof(EntropyCropValues), DefaultPixelType)]
+        [WithFileCollection(nameof(InputImages), nameof(EntropyCropValues), PixelTypes.Rgba32)]
-        public void ImageShouldEntropyCrop<TPixel>(TestImageProvider<TPixel> provider, float value)
+        public void EntropyCrop<TPixel>(TestImageProvider<TPixel> provider, float value)
             where TPixel : struct, IPixel<TPixel>
         {
-            using (Image<TPixel> image = provider.GetImage())
+            provider.RunValidatingProcessorTest(x => x.EntropyCrop(value), value, appendPixelTypeToFileName: false);
-            {
-                image.Mutate(x => x.EntropyCrop(value));
-                image.DebugSave(provider, value);
-            }
         }
     }
 }

tests/ImageSharp.Tests/Processing/Processors/Transforms/SkewTest.cs

@@ -1,72 +0,0 @@
-// Copyright (c) Six Labors and contributors.
-// Licensed under the Apache License, Version 2.0.
-
-using System;
-using System.Collections.Generic;
-using System.Reflection;
-
-using SixLabors.ImageSharp.Processing;
-using SixLabors.ImageSharp.Processing.Processors.Transforms;
-using SixLabors.ImageSharp.PixelFormats;
-using Xunit;
-
-namespace SixLabors.ImageSharp.Tests.Processing.Processors.Transforms
-{
-    public class SkewTest : FileTestBase
-    {
-        public static readonly TheoryData<float, float> SkewValues
-            = new TheoryData<float, float>
-        {
-            { 20, 10 },
-            { -20, -10 }
-        };
-
-        public static readonly List<string> ResamplerNames
-            = new List<string>
-        {
-            nameof(KnownResamplers.Bicubic),
-            nameof(KnownResamplers.Box),
-            nameof(KnownResamplers.CatmullRom),
-            nameof(KnownResamplers.Hermite),
-            nameof(KnownResamplers.Lanczos2),
-            nameof(KnownResamplers.Lanczos3),
-            nameof(KnownResamplers.Lanczos5),
-            nameof(KnownResamplers.Lanczos8),
-            nameof(KnownResamplers.MitchellNetravali),
-            nameof(KnownResamplers.NearestNeighbor),
-            nameof(KnownResamplers.Robidoux),
-            nameof(KnownResamplers.RobidouxSharp),
-            nameof(KnownResamplers.Spline),
-            nameof(KnownResamplers.Triangle),
-            nameof(KnownResamplers.Welch),
-        };
-
-        [Theory]
-        [WithTestPatternImages(nameof(SkewValues), 100, 50, DefaultPixelType)]
-        public void ImageShouldSkew<TPixel>(TestImageProvider<TPixel> provider, float x, float y)
-            where TPixel : struct, IPixel<TPixel>
-        {
-            using (Image<TPixel> image = provider.GetImage())
-            {
-                image.Mutate(i => i.Skew(x, y));
-                image.DebugSave(provider, string.Join("_", x, y));
-            }
-        }
-
-        [Theory]
-        [WithTestPatternImages(nameof(SkewValues), 100, 50, DefaultPixelType)]
-        public void ImageShouldSkewWithSampler<TPixel>(TestImageProvider<TPixel> provider, float x, float y)
-            where TPixel : struct, IPixel<TPixel>
-        {
-            foreach (string resamplerName in ResamplerNames)
-            {
-                IResampler sampler = TestUtils.GetResampler(resamplerName);
-                using (Image<TPixel> image = provider.GetImage())
-                {
-                    image.Mutate(i => i.Skew(x, y, sampler));
-                    image.DebugSave(provider, string.Join("_", x, y, resamplerName));
-                }
-            }
-        }
-    }
-}

tests/ImageSharp.Tests/Processing/Processors/Transforms/SkewTests.cs

@@ -0,0 +1,67 @@
+// Copyright (c) Six Labors and contributors.
+// Licensed under the Apache License, Version 2.0.
+
+using SixLabors.ImageSharp.PixelFormats;
+using SixLabors.ImageSharp.Processing;
+using SixLabors.ImageSharp.Processing.Processors.Transforms;
+using SixLabors.ImageSharp.Tests.TestUtilities.ImageComparison;
+
+using Xunit;
+
+namespace SixLabors.ImageSharp.Tests.Processing.Processors.Transforms
+{
+    [GroupOutput("Transforms")]
+    public class SkewTests
+    {
+        private const PixelTypes CommonPixelTypes = PixelTypes.Bgra32 | PixelTypes.Rgb24;
+
+        public static readonly string[] ResamplerNames = new[]
+                                                             {
+                                                                 nameof(KnownResamplers.Bicubic),
+                                                                 nameof(KnownResamplers.Box),
+                                                                 nameof(KnownResamplers.CatmullRom),
+                                                                 nameof(KnownResamplers.Hermite),
+                                                                 nameof(KnownResamplers.Lanczos2),
+                                                                 nameof(KnownResamplers.Lanczos3),
+                                                                 nameof(KnownResamplers.Lanczos5),
+                                                                 nameof(KnownResamplers.Lanczos8),
+                                                                 nameof(KnownResamplers.MitchellNetravali),
+                                                                 nameof(KnownResamplers.NearestNeighbor),
+                                                                 nameof(KnownResamplers.Robidoux),
+                                                                 nameof(KnownResamplers.RobidouxSharp),
+                                                                 nameof(KnownResamplers.Spline),
+                                                                 nameof(KnownResamplers.Triangle),
+                                                                 nameof(KnownResamplers.Welch),
+                                                             };
+
+        public static readonly TheoryData<float, float> SkewValues = new TheoryData<float, float>
+                                                                         {
+                                                                             { 20, 10 },
+                                                                             { -20, -10 }
+                                                                         };
+
+        private static readonly ImageComparer ValidatorComparer = ImageComparer.TolerantPercentage(0.01f);
+
+        [Theory]
+        [WithTestPatternImages(nameof(SkewValues), 100, 50, CommonPixelTypes)]
+        public void Skew_IsNotBoundToSinglePixelType<TPixel>(TestImageProvider<TPixel> provider, float x, float y)
+            where TPixel : struct, IPixel<TPixel>
+        {
+            provider.RunValidatingProcessorTest(ctx => ctx.Skew(x, y), $"{x}_{y}", ValidatorComparer);
+        }
+
+        [Theory]
+        [WithFile(TestImages.Png.Ducky, nameof(ResamplerNames), PixelTypes.Rgba32)]
+        public void Skew_WorksWithAllResamplers<TPixel>(TestImageProvider<TPixel> provider, string resamplerName)
+            where TPixel : struct, IPixel<TPixel>
+        {
+            IResampler sampler = TestUtils.GetResampler(resamplerName);
+
+            provider.RunValidatingProcessorTest(
+                x => x.Skew(21, 32, sampler),
+                resamplerName,
+                comparer: ValidatorComparer,
+                appendPixelTypeToFileName: false);
+        }
+    }
+}

tests/Images/External

@@ -1 +1 @@
-Subproject commit 1ca515499663e8b0b7c924a49b8d212f7447bdb0
+Subproject commit c057090b4402120a83a8efe251aa5b691db9c0dc