Change RefreshErrorAlg such that it also works with null messages

scala-steward-org · Nov 1, 2021 · 981dd86 · 981dd86
1 parent 1d81cf8
commit 981dd86
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/modules/core/src/main/scala/org/scalasteward/core/repocache/RefreshErrorAlg.scala b/modules/core/src/main/scala/org/scalasteward/core/repocache/RefreshErrorAlg.scala
@@ -46,7 +46,7 @@ final class RefreshErrorAlg[F[_]](
   def persistError[A](repo: Repo)(fa: F[A]): F[A] =
     fa.handleErrorWith { t =>
       dateTimeAlg.currentTimestamp.flatMap { now =>
-        kvStore.put(repo, Entry(now, t.getMessage))
+        kvStore.put(repo, Entry(now, t.toString))
       } >> F.raiseError[A](t)
     }
 

diff --git a/modules/core/src/test/scala/org/scalasteward/core/repocache/RefreshErrorAlgTest.scala b/modules/core/src/test/scala/org/scalasteward/core/repocache/RefreshErrorAlgTest.scala
@@ -20,8 +20,10 @@ class RefreshErrorAlgTest extends CatsEffectSuite {
     val p = refreshErrorAlg.persistError(repo)(MockEff.raiseError(new Throwable())).attempt >>
       refreshErrorAlg.skipIfFailedRecently(repo)(MockEff.unit)
 
+    val expected = "Skipping due to previous error"
     p.runA(MockState.empty).attempt.map { obtained =>
-      assert(obtained.fold(_.getMessage, _.toString).contains("Skipping due to previous error"))
+      val message = obtained.fold(_.getMessage, _.toString).take(expected.length)
+      assertEquals(message, expected)
     }
   }
 }