From 31a250d39b0c3515d038f18100fb8af3a117341e Mon Sep 17 00:00:00 2001
From: Dimitry Ivanov <mail@dimitryivanov.ru>
Date: Mon, 3 Dec 2018 17:19:31 +0300
Subject: [PATCH] Fix typo in AsyncDrawable waitingForDimensions

---
 .../noties/markwon/spans/AsyncDrawable.java   |   8 +-
 .../markwon/spans/AsyncDrawableTest.java      | 107 ++++++++++++++++++
 2 files changed, 111 insertions(+), 4 deletions(-)
 create mode 100644 markwon/src/test/java/ru/noties/markwon/spans/AsyncDrawableTest.java

diff --git a/markwon/src/main/java/ru/noties/markwon/spans/AsyncDrawable.java b/markwon/src/main/java/ru/noties/markwon/spans/AsyncDrawable.java
index 5591884c..dbd0fe0b 100644
--- a/markwon/src/main/java/ru/noties/markwon/spans/AsyncDrawable.java
+++ b/markwon/src/main/java/ru/noties/markwon/spans/AsyncDrawable.java
@@ -34,7 +34,7 @@ public class AsyncDrawable extends Drawable {
     private float textSize;
 
     // @since 2.0.1 for use-cases when image is loaded faster than span is drawn and knows canvas width
-    private boolean waitingForDemensions;
+    private boolean waitingForDimensions;
 
     /**
      * @since 1.0.1
@@ -108,11 +108,11 @@ public class AsyncDrawable extends Drawable {
 
         if (canvasWidth == 0) {
             // we still have no bounds - wait for them
-            waitingForDemensions = true;
+            waitingForDimensions = true;
             return;
         }
 
-        waitingForDemensions = false;
+        waitingForDimensions = false;
 
         final Rect bounds = resolveBounds();
         result.setBounds(bounds);
@@ -129,7 +129,7 @@ public class AsyncDrawable extends Drawable {
         this.canvasWidth = width;
         this.textSize = textSize;
 
-        if (waitingForDemensions) {
+        if (waitingForDimensions) {
             initBounds();
         }
     }
diff --git a/markwon/src/test/java/ru/noties/markwon/spans/AsyncDrawableTest.java b/markwon/src/test/java/ru/noties/markwon/spans/AsyncDrawableTest.java
new file mode 100644
index 00000000..873cc404
--- /dev/null
+++ b/markwon/src/test/java/ru/noties/markwon/spans/AsyncDrawableTest.java
@@ -0,0 +1,107 @@
+package ru.noties.markwon.spans;
+
+import android.graphics.Canvas;
+import android.graphics.ColorFilter;
+import android.graphics.Rect;
+import android.graphics.drawable.Drawable;
+import android.support.annotation.NonNull;
+import android.support.annotation.Nullable;
+
+import org.junit.Before;
+import org.junit.Test;
+import org.junit.runner.RunWith;
+import org.robolectric.RobolectricTestRunner;
+import org.robolectric.annotation.Config;
+
+import ru.noties.markwon.renderer.ImageSize;
+import ru.noties.markwon.renderer.ImageSizeResolver;
+import ru.noties.markwon.renderer.ImageSizeResolverDef;
+
+import static org.junit.Assert.assertEquals;
+import static org.junit.Assert.assertFalse;
+import static org.junit.Assert.assertNotNull;
+import static org.junit.Assert.assertNull;
+import static org.junit.Assert.assertTrue;
+import static org.mockito.Mockito.mock;
+
+@RunWith(RobolectricTestRunner.class)
+@Config(manifest = Config.NONE)
+public class AsyncDrawableTest {
+
+    private ImageSizeResolver imageSizeResolver;
+
+    @Before
+    public void before() {
+        imageSizeResolver = new ImageSizeResolverDef();
+    }
+
+    @Test
+    public void no_dimensions_await() {
+        // when drawable have no known dimensions yet, it will await for them
+
+        final AsyncDrawable drawable = new AsyncDrawable("",
+                mock(AsyncDrawable.Loader.class),
+                imageSizeResolver,
+                new ImageSize(new ImageSize.Dimension(100.F, "%"), null));
+
+        final Drawable result = new AbstractDrawable();
+        result.setBounds(0, 0, 0, 0);
+
+        assertFalse(drawable.hasResult());
+        drawable.setResult(result);
+        assertTrue(drawable.hasResult());
+
+        assertTrue(result.getBounds().isEmpty());
+
+        drawable.initWithKnownDimensions(100, 1);
+        assertEquals(
+                new Rect(0, 0, 100, 0),
+                result.getBounds()
+        );
+    }
+
+    @Test
+    public void previous_result_detached() {
+        // when result is present it will be detached (setCallback(null))
+
+        final AsyncDrawable drawable = new AsyncDrawable("",
+                mock(AsyncDrawable.Loader.class),
+                imageSizeResolver,
+                null);
+
+        drawable.setCallback2(mock(Drawable.Callback.class));
+        drawable.initWithKnownDimensions(100, 1);
+
+        final Drawable result1 = new AbstractDrawable();
+        final Drawable result2 = new AbstractDrawable();
+
+        drawable.setResult(result1);
+        assertNotNull(result1.getCallback());
+        drawable.setResult(result2);
+        assertNull(result1.getCallback());
+        assertNotNull(result2.getCallback());
+    }
+
+    private static class AbstractDrawable extends Drawable {
+
+        @Override
+        public void draw(@NonNull Canvas canvas) {
+
+        }
+
+        @Override
+        public void setAlpha(int alpha) {
+
+        }
+
+        @Override
+        public void setColorFilter(@Nullable ColorFilter colorFilter) {
+
+        }
+
+        @Override
+        public int getOpacity() {
+            return 0;
+        }
+    }
+}
\ No newline at end of file