code review feedback

Author: asolntsev

src/main/java/net/datafaker/providers/base/Image.java

@@ -2,8 +2,8 @@
 
 
 import javax.imageio.ImageIO;
-import java.awt.*;
 import java.awt.Color;
+import java.awt.Graphics2D;
 import java.awt.image.BufferedImage;
 import java.io.ByteArrayOutputStream;
 import java.io.IOException;

src/test/java/net/datafaker/assertions/ImageAssert.java

@@ -26,6 +26,10 @@ public class ImageAssert extends AbstractAssert<ImageAssert, ImageData> {
         super(actual, ImageAssert.class);
     }
 
+    public ImageAssert is(ImageType type) {
+        return is(type, 256, 256);
+    }
+
     public ImageAssert is(ImageType type, int width, int height) {
         assertImageDataUrl(type);
         return switch (type) {
@@ -65,7 +69,7 @@ private ImageAssert verifySvg(ImageData image, int width, int height) {
         byte[] bytes = Base64.getDecoder().decode(content);
         try {
             String xml = new String(bytes);
-            assertThat(xml).matches("<svg xmlns=\".+\" width=\"256\" height=\"256\">.*");
+            assertThat(xml).matches("<svg xmlns=\".+\" width=\"256\" height=\"256\">.*</svg>");
 
             DocumentBuilder builder = DocumentBuilderFactory.newInstance().newDocumentBuilder();
             Document doc = builder.parse(new ByteArrayInputStream(bytes));

src/test/java/net/datafaker/providers/base/ImageTest.java

@@ -21,37 +21,37 @@ class ImageTest {
 
     @Test
     void bmp() {
-        assertThatImage(faker.image().base64BMP()).is(BMP, 256, 256);
+        assertThatImage(faker.image().base64BMP()).is(BMP);
     }
 
     @Test
     void gif() {
-        assertThatImage(faker.image().base64GIF()).is(GIF, 256, 256);
+        assertThatImage(faker.image().base64GIF()).is(GIF);
     }
 
     @Test
     void png() {
-        assertThatImage(faker.image().base64PNG()).is(PNG, 256, 256);
+        assertThatImage(faker.image().base64PNG()).is(PNG);
     }
 
     @Test
     void jpg() {
-        assertThatImage(faker.image().base64JPG()).is(JPEG, 256, 256);
+        assertThatImage(faker.image().base64JPG()).is(JPEG);
     }
 
     @Test
     void jpeg() {
-        assertThatImage(faker.image().base64JPEG()).is(JPEG, 256, 256);
+        assertThatImage(faker.image().base64JPEG()).is(JPEG);
     }
 
     @Test
     void svg() {
-        assertThatImage(faker.image().base64SVG()).is(SVG, 256, 256);
+        assertThatImage(faker.image().base64SVG()).is(SVG);
     }
 
     @Test
     void tiff() {
-        assertThatImage(faker.image().base64TIFF()).is(TIFF, 256, 256);
+        assertThatImage(faker.image().base64TIFF()).is(TIFF);
     }
 
     @ParameterizedTest
@@ -65,15 +65,15 @@ void base64(ImageType imageType) {
     @Test
     void defaultBuilder_generatesPngImage() {
         String image = faker.image().base64(Image.ImageBuilder.builder().build());
-        assertThatImage(image).is(PNG, 256, 256);
+        assertThatImage(image).is(PNG);
     }
 
     @Test
     void customBase64builder() {
         String gif = faker.image().base64(Image.ImageBuilder.builder()
             .type(ImageType.GIF)
             .build());
-        assertThatImage(gif).is(GIF, 256, 256);
+        assertThatImage(gif).is(GIF);
     }
 
     @Test