Merge pull request #1688 from SixLabors/bp/serialtests

brianpopow · web-flow · commit 3f96b1e8406c · 2021-07-05T12:30:20.000+02:00
Run some high memory tests serial
diff --git a/src/ImageSharp/Formats/Jpeg/Components/Encoder/YCbCrForwardConverter{TPixel}.cs b/src/ImageSharp/Formats/Jpeg/Components/Encoder/YCbCrForwardConverter{TPixel}.cs
@@ -13,8 +13,8 @@ internal static class YCbCrForwardConverter<TPixel>
     {
         public static void LoadAndStretchEdges(RowOctet<TPixel> source, Span<TPixel> dest, Point start, Size sampleSize, Size totalSize)
         {
-            DebugGuard.MustBeBetweenOrEqualTo(start.X, 1, totalSize.Width - 1, nameof(start.X));
-            DebugGuard.MustBeBetweenOrEqualTo(start.Y, 1, totalSize.Height - 1, nameof(start.Y));
+            DebugGuard.MustBeBetweenOrEqualTo(start.X, 0, totalSize.Width - 1, nameof(start.X));
+            DebugGuard.MustBeBetweenOrEqualTo(start.Y, 0, totalSize.Height - 1, nameof(start.Y));
 
             int width = Math.Min(sampleSize.Width, totalSize.Width - start.X);
             int height = Math.Min(sampleSize.Height, totalSize.Height - start.Y);
diff --git a/tests/ImageSharp.Tests/Formats/Bmp/BmpDecoderTests.cs b/tests/ImageSharp.Tests/Formats/Bmp/BmpDecoderTests.cs
@@ -20,6 +20,7 @@
 // ReSharper disable InconsistentNaming
 namespace SixLabors.ImageSharp.Tests.Formats.Bmp
 {
+    [Collection("RunSerial")]
     [Trait("Format", "Bmp")]
     public class BmpDecoderTests
     {
diff --git a/tests/ImageSharp.Tests/Formats/Bmp/BmpEncoderTests.cs b/tests/ImageSharp.Tests/Formats/Bmp/BmpEncoderTests.cs
@@ -19,6 +19,7 @@
 // ReSharper disable InconsistentNaming
 namespace SixLabors.ImageSharp.Tests.Formats.Bmp
 {
+    [Collection("RunSerial")]
     [Trait("Format", "Bmp")]
     public class BmpEncoderTests
     {
diff --git a/tests/ImageSharp.Tests/Formats/Gif/GifDecoderTests.cs b/tests/ImageSharp.Tests/Formats/Gif/GifDecoderTests.cs
@@ -17,6 +17,7 @@
 // ReSharper disable InconsistentNaming
 namespace SixLabors.ImageSharp.Tests.Formats.Gif
 {
+    [Collection("RunSerial")]
     [Trait("Format", "Gif")]
     public class GifDecoderTests
     {
diff --git a/tests/ImageSharp.Tests/Formats/Gif/GifEncoderTests.cs b/tests/ImageSharp.Tests/Formats/Gif/GifEncoderTests.cs
@@ -14,6 +14,7 @@
 // ReSharper disable InconsistentNaming
 namespace SixLabors.ImageSharp.Tests.Formats.Gif
 {
+    [Collection("RunSerial")]
     [Trait("Format", "Gif")]
     public class GifEncoderTests
     {
diff --git a/tests/ImageSharp.Tests/Formats/Jpg/JpegDecoderTests.cs b/tests/ImageSharp.Tests/Formats/Jpg/JpegDecoderTests.cs
@@ -22,6 +22,7 @@
 namespace SixLabors.ImageSharp.Tests.Formats.Jpg
 {
     // TODO: Scatter test cases into multiple test classes
+    [Collection("RunSerial")]
     [Trait("Format", "Jpg")]
     public partial class JpegDecoderTests
     {
diff --git a/tests/ImageSharp.Tests/Formats/Jpg/JpegEncoderTests.cs b/tests/ImageSharp.Tests/Formats/Jpg/JpegEncoderTests.cs
@@ -20,6 +20,7 @@
 
 namespace SixLabors.ImageSharp.Tests.Formats.Jpg
 {
+    [Collection("RunSerial")]
     [Trait("Format", "Jpg")]
     public class JpegEncoderTests
     {
diff --git a/tests/ImageSharp.Tests/Formats/Png/PngDecoderTests.cs b/tests/ImageSharp.Tests/Formats/Png/PngDecoderTests.cs
@@ -16,6 +16,7 @@
 // ReSharper disable InconsistentNaming
 namespace SixLabors.ImageSharp.Tests.Formats.Png
 {
+    [Collection("RunSerial")]
     [Trait("Format", "Png")]
     public partial class PngDecoderTests
     {
diff --git a/tests/ImageSharp.Tests/Formats/Png/PngEncoderTests.cs b/tests/ImageSharp.Tests/Formats/Png/PngEncoderTests.cs
@@ -15,6 +15,7 @@
 
 namespace SixLabors.ImageSharp.Tests.Formats.Png
 {
+    [Collection("RunSerial")]
     [Trait("Format", "Png")]
     public partial class PngEncoderTests
     {
diff --git a/tests/ImageSharp.Tests/Formats/Tga/TgaDecoderTests.cs b/tests/ImageSharp.Tests/Formats/Tga/TgaDecoderTests.cs
@@ -16,6 +16,7 @@
 // ReSharper disable InconsistentNaming
 namespace SixLabors.ImageSharp.Tests.Formats.Tga
 {
+    [Collection("RunSerial")]
     [Trait("Format", "Tga")]
     public class TgaDecoderTests
     {
diff --git a/tests/ImageSharp.Tests/Formats/Tga/TgaEncoderTests.cs b/tests/ImageSharp.Tests/Formats/Tga/TgaEncoderTests.cs
@@ -13,6 +13,7 @@
 // ReSharper disable InconsistentNaming
 namespace SixLabors.ImageSharp.Tests.Formats.Tga
 {
+    [Collection("RunSerial")]
     [Trait("Format", "Tga")]
     public class TgaEncoderTests
     {
diff --git a/tests/ImageSharp.Tests/Formats/Tiff/TiffDecoderTests.cs b/tests/ImageSharp.Tests/Formats/Tiff/TiffDecoderTests.cs
@@ -17,6 +17,7 @@
 
 namespace SixLabors.ImageSharp.Tests.Formats.Tiff
 {
+    [Collection("RunSerial")]
     [Trait("Format", "Tiff")]
     public class TiffDecoderTests
     {
diff --git a/tests/ImageSharp.Tests/Formats/Tiff/TiffEncoderTests.cs b/tests/ImageSharp.Tests/Formats/Tiff/TiffEncoderTests.cs
@@ -17,6 +17,7 @@
 
 namespace SixLabors.ImageSharp.Tests.Formats.Tiff
 {
+    [Collection("RunSerial")]
     [Trait("Format", "Tiff")]
     public class TiffEncoderTests
     {

Original file line number	Diff line number	Diff line change
`@@ -13,8 +13,8 @@ internal static class YCbCrForwardConverter<TPixel>`
`13`	`13`	`{`
`14`	`14`	`public static void LoadAndStretchEdges(RowOctet<TPixel> source, Span<TPixel> dest, Point start, Size sampleSize, Size totalSize)`
`15`	`15`	`{`
`16`		`- DebugGuard.MustBeBetweenOrEqualTo(start.X, 1, totalSize.Width - 1, nameof(start.X));`
`17`		`- DebugGuard.MustBeBetweenOrEqualTo(start.Y, 1, totalSize.Height - 1, nameof(start.Y));`
	`16`	`+ DebugGuard.MustBeBetweenOrEqualTo(start.X, 0, totalSize.Width - 1, nameof(start.X));`
	`17`	`+ DebugGuard.MustBeBetweenOrEqualTo(start.Y, 0, totalSize.Height - 1, nameof(start.Y));`
`18`	`18`
`19`	`19`	`int width = Math.Min(sampleSize.Width, totalSize.Width - start.X);`
`20`	`20`	`int height = Math.Min(sampleSize.Height, totalSize.Height - start.Y);`
Original file line number	Diff line number	Diff line change
`@@ -20,6 +20,7 @@`
`20`	`20`	`// ReSharper disable InconsistentNaming`
`21`	`21`	`namespace SixLabors.ImageSharp.Tests.Formats.Bmp`
`22`	`22`	`{`
	`23`	`+ [Collection("RunSerial")]`
`23`	`24`	`[Trait("Format", "Bmp")]`
`24`	`25`	`public class BmpDecoderTests`
`25`	`26`	`{`
Original file line number	Diff line number	Diff line change
`@@ -19,6 +19,7 @@`
`19`	`19`	`// ReSharper disable InconsistentNaming`
`20`	`20`	`namespace SixLabors.ImageSharp.Tests.Formats.Bmp`
`21`	`21`	`{`
	`22`	`+ [Collection("RunSerial")]`
`22`	`23`	`[Trait("Format", "Bmp")]`
`23`	`24`	`public class BmpEncoderTests`
`24`	`25`	`{`
Original file line number	Diff line number	Diff line change
`@@ -17,6 +17,7 @@`
`17`	`17`	`// ReSharper disable InconsistentNaming`
`18`	`18`	`namespace SixLabors.ImageSharp.Tests.Formats.Gif`
`19`	`19`	`{`
	`20`	`+ [Collection("RunSerial")]`
`20`	`21`	`[Trait("Format", "Gif")]`
`21`	`22`	`public class GifDecoderTests`
`22`	`23`	`{`
Original file line number	Diff line number	Diff line change
`@@ -14,6 +14,7 @@`
`14`	`14`	`// ReSharper disable InconsistentNaming`
`15`	`15`	`namespace SixLabors.ImageSharp.Tests.Formats.Gif`
`16`	`16`	`{`
	`17`	`+ [Collection("RunSerial")]`
`17`	`18`	`[Trait("Format", "Gif")]`
`18`	`19`	`public class GifEncoderTests`
`19`	`20`	`{`
Original file line number	Diff line number	Diff line change
`@@ -22,6 +22,7 @@`
`22`	`22`	`namespace SixLabors.ImageSharp.Tests.Formats.Jpg`
`23`	`23`	`{`
`24`	`24`	`// TODO: Scatter test cases into multiple test classes`
	`25`	`+ [Collection("RunSerial")]`
`25`	`26`	`[Trait("Format", "Jpg")]`
`26`	`27`	`public partial class JpegDecoderTests`
`27`	`28`	`{`
Original file line number	Diff line number	Diff line change
`@@ -16,6 +16,7 @@`
`16`	`16`	`// ReSharper disable InconsistentNaming`
`17`	`17`	`namespace SixLabors.ImageSharp.Tests.Formats.Png`
`18`	`18`	`{`
	`19`	`+ [Collection("RunSerial")]`
`19`	`20`	`[Trait("Format", "Png")]`
`20`	`21`	`public partial class PngDecoderTests`
`21`	`22`	`{`
Original file line number	Diff line number	Diff line change
`@@ -15,6 +15,7 @@`
`15`	`15`
`16`	`16`	`namespace SixLabors.ImageSharp.Tests.Formats.Png`
`17`	`17`	`{`
	`18`	`+ [Collection("RunSerial")]`
`18`	`19`	`[Trait("Format", "Png")]`
`19`	`20`	`public partial class PngEncoderTests`
`20`	`21`	`{`