Changeset 16978 in josm for trunk/test/unit/org

Timestamp:

2020-08-30T09:46:47+02:00 (4 years ago)

Author:

simon04

Message:

fix #19706, fix #19725 - Do not upscale images when using ImageResizeMode.BOUNDED

File:

• trunk/test/unit/org/openstreetmap/josm/tools/ImageResizeModeTest.java (modified) (5 diffs)

trunk/test/unit/org/openstreetmap/josm/tools/ImageResizeModeTest.java

@@ -14 +14 @@
 import edu.umd.cs.findbugs.annotations.SuppressFBWarnings;
 
+/**
+ * Unit tests of {@link ImageResizeMode} class.
+ */
 public class ImageResizeModeTest {
 
@@ -23 +26 @@
     public JOSMTestRules test = new JOSMTestRules().preferences();
 
+    final Dimension image = new Dimension(64, 48);
+    final Dimension smallImage = new Dimension(14, 10);
+
+    /**
+     * Test method for {@link ImageResizeMode#computeDimension} using {@link ImageResizeMode#AUTO}
+     */
     @Test
-    public void testComputeDimension() {
-        final Dimension image = new Dimension(64, 48);
+    public void testComputeDimensionAuto() {
         assertThrows(IllegalArgumentException.class, () -> ImageResizeMode.AUTO.computeDimension(new Dimension(0, 0), image));
         assertEquals(new Dimension(64, 48), ImageResizeMode.AUTO.computeDimension(ImageResource.DEFAULT_DIMENSION, image));
@@ -33 +41 @@
         assertEquals(new Dimension(21, 16), ImageResizeMode.AUTO.computeDimension(new Dimension(-1, 16), image));
         assertEquals(new Dimension(24, 32), ImageResizeMode.AUTO.computeDimension(new Dimension(24, 32), image));
+    }
+
+    /**
+     * Test method for {@link ImageResizeMode#computeDimension} using {@link ImageResizeMode#BOUNDED}
+     */
+    @Test
+    public void testComputeDimensionBounded() {
         assertEquals(new Dimension(64, 48), ImageResizeMode.BOUNDED.computeDimension(ImageResource.DEFAULT_DIMENSION, image));
         assertEquals(new Dimension(64, 48), ImageResizeMode.BOUNDED.computeDimension(new Dimension(-1, -1), image));
@@ -39 +54 @@
         assertEquals(new Dimension(21, 16), ImageResizeMode.BOUNDED.computeDimension(new Dimension(-1, 16), image));
         assertEquals(new Dimension(24, 18), ImageResizeMode.BOUNDED.computeDimension(new Dimension(24, 32), image));
+        assertEquals(new Dimension(14, 10), ImageResizeMode.BOUNDED.computeDimension(new Dimension(64, 48), smallImage));
+        assertEquals(new Dimension(14, 10), ImageResizeMode.BOUNDED.computeDimension(new Dimension(16, 16), smallImage));
+        assertEquals(new Dimension(11, 8), ImageResizeMode.BOUNDED.computeDimension(new Dimension(16, 8), smallImage));
+        assertEquals(new Dimension(12, 8), ImageResizeMode.BOUNDED.computeDimension(new Dimension(12, 12), smallImage));
+        assertEquals(new Dimension(11, 8), ImageResizeMode.BOUNDED.computeDimension(new Dimension(12, 8), smallImage));
+        assertEquals(new Dimension(8, 5), ImageResizeMode.BOUNDED.computeDimension(new Dimension(8, 16), smallImage));
+        assertEquals(new Dimension(8, 5), ImageResizeMode.BOUNDED.computeDimension(new Dimension(8, 8), smallImage));
+        assertEquals(new Dimension(8, 5), ImageResizeMode.BOUNDED.computeDimension(new Dimension(8, 8), smallImage));
+    }
+
+    /**
+     * Test method for {@link ImageResizeMode#computeDimension} using {@link ImageResizeMode#PADDED}
+     */
+    @Test
+    public void testComputeDimensionPadded() {
         assertThrows(IllegalArgumentException.class, () -> ImageResizeMode.PADDED.computeDimension(new Dimension(0, 0), image));
         assertThrows(IllegalArgumentException.class, () -> ImageResizeMode.PADDED.computeDimension(ImageResource.DEFAULT_DIMENSION, image));
@@ -48 +78 @@
     }
 
+    /**
+     * Test method for {@link ImageResizeMode#cacheKey}
+     */
     @Test
     public void testCacheKey() {