From bee66f234ccc0e23eccfc205f89322624734107e Mon Sep 17 00:00:00 2001 From: Anton Firszov Date: Sat, 31 Mar 2018 21:49:26 +0200 Subject: [PATCH] higher tolerance for Resize and Filters --- .../Formats/Gif/GifDecoderTests.cs | 4 +-- .../Formats/Jpg/JpegDecoderTests.cs | 10 +++--- .../Processors/Transforms/ResizeTests.cs | 31 ++++++++++--------- .../TestUtilities/TestImageExtensions.cs | 11 ++++--- .../TestUtilities/TestUtils.cs | 6 +++- 5 files changed, 35 insertions(+), 27 deletions(-) diff --git a/tests/ImageSharp.Tests/Formats/Gif/GifDecoderTests.cs b/tests/ImageSharp.Tests/Formats/Gif/GifDecoderTests.cs index 9cdb9f8b1..82d281d85 100644 --- a/tests/ImageSharp.Tests/Formats/Gif/GifDecoderTests.cs +++ b/tests/ImageSharp.Tests/Formats/Gif/GifDecoderTests.cs @@ -67,7 +67,7 @@ namespace SixLabors.ImageSharp.Tests using (Image image = provider.GetImage()) { image.DebugSave(provider); - image.CompareFirstFrameToReferenceOutput(provider, ImageComparer.Exact); + image.CompareFirstFrameToReferenceOutput(ImageComparer.Exact, provider); } } @@ -85,7 +85,7 @@ namespace SixLabors.ImageSharp.Tests { Assert.Equal(expectedFrameCount, image.Frames.Count); image.DebugSave(provider); - image.CompareFirstFrameToReferenceOutput(provider, ImageComparer.Exact); + image.CompareFirstFrameToReferenceOutput(ImageComparer.Exact, provider); } } diff --git a/tests/ImageSharp.Tests/Formats/Jpg/JpegDecoderTests.cs b/tests/ImageSharp.Tests/Formats/Jpg/JpegDecoderTests.cs index 95ee40e80..bba1fe342 100644 --- a/tests/ImageSharp.Tests/Formats/Jpg/JpegDecoderTests.cs +++ b/tests/ImageSharp.Tests/Formats/Jpg/JpegDecoderTests.cs @@ -134,7 +134,7 @@ namespace SixLabors.ImageSharp.Tests.Formats.Jpg image.DebugSave(provider); provider.Utility.TestName = DecodeBaselineJpegOutputName; - image.CompareToReferenceOutput(provider, ImageComparer.Tolerant(BaselineTolerance_PdfJs), appendPixelTypeToFileName: false); + image.CompareToReferenceOutput(ImageComparer.Tolerant(BaselineTolerance_PdfJs), provider, appendPixelTypeToFileName: false); } } @@ -148,8 +148,8 @@ namespace SixLabors.ImageSharp.Tests.Formats.Jpg image.DebugSave(provider); provider.Utility.TestName = DecodeBaselineJpegOutputName; image.CompareToReferenceOutput( - provider, this.GetImageComparerForOrigDecoder(provider), + provider, appendPixelTypeToFileName: false); } } @@ -165,8 +165,8 @@ namespace SixLabors.ImageSharp.Tests.Formats.Jpg provider.Utility.TestName = DecodeBaselineJpegOutputName; image.CompareToReferenceOutput( - provider, ImageComparer.Tolerant(BaselineTolerance_PdfJs), + provider, appendPixelTypeToFileName: false); } } @@ -193,8 +193,8 @@ namespace SixLabors.ImageSharp.Tests.Formats.Jpg provider.Utility.TestName = DecodeProgressiveJpegOutputName; image.CompareToReferenceOutput( - provider, this.GetImageComparerForOrigDecoder(provider), + provider, appendPixelTypeToFileName: false); } } @@ -210,8 +210,8 @@ namespace SixLabors.ImageSharp.Tests.Formats.Jpg provider.Utility.TestName = DecodeProgressiveJpegOutputName; image.CompareToReferenceOutput( - provider, ImageComparer.Tolerant(ProgressiveTolerance_PdfJs), + provider, appendPixelTypeToFileName: false); } } diff --git a/tests/ImageSharp.Tests/Processing/Processors/Transforms/ResizeTests.cs b/tests/ImageSharp.Tests/Processing/Processors/Transforms/ResizeTests.cs index 80fbcaf84..567986dd5 100644 --- a/tests/ImageSharp.Tests/Processing/Processors/Transforms/ResizeTests.cs +++ b/tests/ImageSharp.Tests/Processing/Processors/Transforms/ResizeTests.cs @@ -12,11 +12,14 @@ using Xunit; namespace SixLabors.ImageSharp.Tests.Processing.Processors.Transforms { using SixLabors.ImageSharp.Processing.Transforms; + using SixLabors.ImageSharp.Tests.TestUtilities.ImageComparison; public class ResizeTests : FileTestBase { public static readonly string[] CommonTestImages = { TestImages.Png.CalliphoraPartial }; + private static readonly ImageComparer ResizeValidatorComparer = ImageComparer.Tolerant(0.01f / 100); + public static readonly TheoryData AllReSamplers = new TheoryData { @@ -50,7 +53,7 @@ namespace SixLabors.ImageSharp.Tests.Processing.Processors.Transforms string details = $"{name}-{ratio.ToString(System.Globalization.CultureInfo.InvariantCulture)}"; image.DebugSave(provider, details); - image.CompareToReferenceOutput(provider, details); + image.CompareToReferenceOutput(ResizeValidatorComparer, provider, details); } } @@ -64,7 +67,7 @@ namespace SixLabors.ImageSharp.Tests.Processing.Processors.Transforms image.Mutate(x => x.Resize(image.Size() / 2, true)); image.DebugSave(provider); - image.CompareToReferenceOutput(provider); + image.CompareToReferenceOutput(ResizeValidatorComparer, provider); } } @@ -78,7 +81,7 @@ namespace SixLabors.ImageSharp.Tests.Processing.Processors.Transforms image.Mutate(x => x.Resize(image.Width / 2, image.Height / 2, true)); image.DebugSave(provider); - image.CompareToReferenceOutput(provider); + image.CompareToReferenceOutput(ResizeValidatorComparer, provider); } } @@ -91,7 +94,7 @@ namespace SixLabors.ImageSharp.Tests.Processing.Processors.Transforms { image.Mutate(x => x.Resize(image.Width / 2, image.Height / 2)); image.DebugSave(provider); - image.CompareToReferenceOutput(provider); + image.CompareToReferenceOutput(ResizeValidatorComparer, provider); } } @@ -122,7 +125,7 @@ namespace SixLabors.ImageSharp.Tests.Processing.Processors.Transforms image.Mutate(x => x.Resize(image.Width, image.Height, KnownResamplers.Bicubic, sourceRectangle, destRectangle, false)); image.DebugSave(provider); - image.CompareToReferenceOutput(provider); + image.CompareToReferenceOutput(ResizeValidatorComparer, provider); } } @@ -136,7 +139,7 @@ namespace SixLabors.ImageSharp.Tests.Processing.Processors.Transforms image.Mutate(x => x.Resize(image.Width / 3, 0, false)); image.DebugSave(provider); - image.CompareToReferenceOutput(provider); + image.CompareToReferenceOutput(ResizeValidatorComparer, provider); } } @@ -150,7 +153,7 @@ namespace SixLabors.ImageSharp.Tests.Processing.Processors.Transforms image.Mutate(x => x.Resize(0, image.Height / 3, false)); image.DebugSave(provider); - image.CompareToReferenceOutput(provider); + image.CompareToReferenceOutput(ResizeValidatorComparer, provider); } } @@ -169,7 +172,7 @@ namespace SixLabors.ImageSharp.Tests.Processing.Processors.Transforms image.Mutate(x => x.Resize(options)); image.DebugSave(provider); - image.CompareToReferenceOutput(provider); + image.CompareToReferenceOutput(ResizeValidatorComparer, provider); } } @@ -188,7 +191,7 @@ namespace SixLabors.ImageSharp.Tests.Processing.Processors.Transforms image.Mutate(x => x.Resize(options)); image.DebugSave(provider); - image.CompareToReferenceOutput(provider); + image.CompareToReferenceOutput(ResizeValidatorComparer, provider); } } @@ -208,7 +211,7 @@ namespace SixLabors.ImageSharp.Tests.Processing.Processors.Transforms image.Mutate(x => x.Resize(options)); image.DebugSave(provider); - image.CompareToReferenceOutput(provider); + image.CompareToReferenceOutput(ResizeValidatorComparer, provider); } } @@ -228,7 +231,7 @@ namespace SixLabors.ImageSharp.Tests.Processing.Processors.Transforms image.Mutate(x => x.Resize(options)); image.DebugSave(provider); - image.CompareToReferenceOutput(provider); + image.CompareToReferenceOutput(ResizeValidatorComparer, provider); } } @@ -248,7 +251,7 @@ namespace SixLabors.ImageSharp.Tests.Processing.Processors.Transforms image.Mutate(x => x.Resize(options)); image.DebugSave(provider); - image.CompareToReferenceOutput(provider); + image.CompareToReferenceOutput(ResizeValidatorComparer, provider); } } @@ -268,7 +271,7 @@ namespace SixLabors.ImageSharp.Tests.Processing.Processors.Transforms image.Mutate(x => x.Resize(options)); image.DebugSave(provider); - image.CompareToReferenceOutput(provider); + image.CompareToReferenceOutput(ResizeValidatorComparer, provider); } } @@ -288,7 +291,7 @@ namespace SixLabors.ImageSharp.Tests.Processing.Processors.Transforms image.Mutate(x => x.Resize(options)); image.DebugSave(provider); - image.CompareToReferenceOutput(provider); + image.CompareToReferenceOutput(ResizeValidatorComparer, provider); } } diff --git a/tests/ImageSharp.Tests/TestUtilities/TestImageExtensions.cs b/tests/ImageSharp.Tests/TestUtilities/TestImageExtensions.cs index 3df71a08a..dbae4f85d 100644 --- a/tests/ImageSharp.Tests/TestUtilities/TestImageExtensions.cs +++ b/tests/ImageSharp.Tests/TestUtilities/TestImageExtensions.cs @@ -120,6 +120,7 @@ namespace SixLabors.ImageSharp.Tests /// The extension /// A boolean indicating whether we should debug save + compare against a grayscale image, smaller in size. /// A boolean indicating whether to append the pixel type to the output file name. + /// A custom for the verification /// public static Image CompareToReferenceOutput( this Image image, @@ -132,22 +133,22 @@ namespace SixLabors.ImageSharp.Tests { return CompareToReferenceOutput( image, - provider, ImageComparer.Tolerant(), + provider, testOutputDetails, extension, grayscale, appendPixelTypeToFileName); } - + /// /// Compares the image against the expected Reference output, throws an exception if the images are not similar enough. /// The output file should be named identically to the output produced by . /// /// The pixel format /// The image - /// The image provider /// The to use + /// The image provider /// Details to be concatenated to the test output file, describing the parameters of the test. /// The extension /// A boolean indicating whether we should debug save + compare against a grayscale image, smaller in size. @@ -155,8 +156,8 @@ namespace SixLabors.ImageSharp.Tests /// public static Image CompareToReferenceOutput( this Image image, - ITestImageProvider provider, ImageComparer comparer, + ITestImageProvider provider, object testOutputDetails = null, string extension = "png", bool grayscale = false, @@ -177,8 +178,8 @@ namespace SixLabors.ImageSharp.Tests public static Image CompareFirstFrameToReferenceOutput( this Image image, - ITestImageProvider provider, ImageComparer comparer, + ITestImageProvider provider, object testOutputDetails = null, string extension = "png", bool grayscale = false, diff --git a/tests/ImageSharp.Tests/TestUtilities/TestUtils.cs b/tests/ImageSharp.Tests/TestUtilities/TestUtils.cs index d31094085..21174d9e7 100644 --- a/tests/ImageSharp.Tests/TestUtilities/TestUtils.cs +++ b/tests/ImageSharp.Tests/TestUtilities/TestUtils.cs @@ -165,6 +165,10 @@ namespace SixLabors.ImageSharp.Tests ImageComparer comparer = null) where TPixel : struct, IPixel { + if (comparer == null) + { + comparer = ImageComparer.Tolerant(0.5f / 100); + } using (Image image = provider.GetImage()) { image.Mutate(process); @@ -173,7 +177,7 @@ namespace SixLabors.ImageSharp.Tests // TODO: Investigate the cause of pixel inaccuracies under Linux if (TestEnvironment.IsWindows) { - image.CompareToReferenceOutput(provider, testOutputDetails); + image.CompareToReferenceOutput(comparer, provider, testOutputDetails); } } }