PR review round two.

Author: notandyvee

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

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

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

@@ -8,7 +8,6 @@ import org.junit.runner.RunWith
 import org.robolectric.RobolectricTestRunner
 import org.wordpress.aztec.spans.MarkSpan
 
-@RunWith(RobolectricTestRunner::class)
 class MarkSpanTest {
     /**
      * Test used to confirm two crashes related are fixed.
@@ -18,16 +17,12 @@ class MarkSpanTest {
     @Test
     fun `Calling MarkSpan#safelyParseColor with empty string should not cause a crash`() {
         var error = false
-        var result: Int? = null
         try {
-            val span = mockk<MarkSpan>()
-            every { span.safelyParseColor("") } coAnswers { callOriginal() }
-            result = span.safelyParseColor("")
+            MarkSpan(colorString = "")
         } catch (e: Exception) {
             error = true
         }
         Assert.assertFalse(error)
-        Assert.assertEquals(null, result)
     }
 
     /**
@@ -38,15 +33,11 @@ class MarkSpanTest {
     @Test
     fun `Calling MarkSpan#safelyParseColor with null string should not cause a crash`() {
         var error = false
-        var result: Int? = null
         try {
-            val span = mockk<MarkSpan>()
-            every { span.safelyParseColor(null) } coAnswers { callOriginal() }
-            result = span.safelyParseColor(null)
+            MarkSpan(colorString = null)
         } catch (e: Exception) {
             error = true
         }
         Assert.assertFalse(error)
-        Assert.assertEquals(null, result)
     }
-}
+}