Merge pull request #1080 from wordpress-mobile/andy/issue-20738

[Bug] Fix crash and add unit test to cover the issue.

Author: notandyvee

aztec/src/main/kotlin/org/wordpress/aztec/spans/MarkSpan.kt

@@ -26,7 +26,7 @@ class MarkSpan : CharacterStyle, IAztecInlineSpan {
     }
 
     private fun safelyParseColor(colorString: String?): Int? {
-        if (colorString == null) {
+        if (colorString.isNullOrBlank()) {
             return null
         }
         return try {

aztec/src/test/kotlin/org/wordpress/aztec/MarkSpanTest.kt

@@ -0,0 +1,39 @@
+package org.wordpress.aztec
+
+import org.junit.Assert
+import org.junit.Test
+import org.wordpress.aztec.spans.MarkSpan
+
+class MarkSpanTest {
+    /**
+     * Test used to confirm two crashes related are fixed.
+     *
+     * https://github.com/wordpress-mobile/WordPress-Android/issues/20738
+     */
+    @Test
+    fun `Calling MarkSpan#safelyParseColor with empty string should not cause a crash`() {
+        var error = false
+        try {
+            MarkSpan(colorString = "")
+        } catch (e: Exception) {
+            error = true
+        }
+        Assert.assertFalse(error)
+    }
+
+    /**
+     * Test used to confirm two crashes related are fixed.
+     *
+     * https://github.com/wordpress-mobile/WordPress-Android/issues/20694
+     */
+    @Test
+    fun `Calling MarkSpan#safelyParseColor with null string should not cause a crash`() {
+        var error = false
+        try {
+            MarkSpan(colorString = null)
+        } catch (e: Exception) {
+            error = true
+        }
+        Assert.assertFalse(error)
+    }
+}