This is an automated email from the ASF dual-hosted git repository.

holden pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/spark.git


The following commit(s) were added to refs/heads/master by this push:
     new ac4e7947e4ef [SPARK-54325][TESTS] Use custom matcher in ExecutorSuite 
testThrowable
ac4e7947e4ef is described below

commit ac4e7947e4efa4ca2484d9393b14322f4e636df5
Author: Vlad Rozov <[email protected]>
AuthorDate: Mon Feb 23 12:47:09 2026 -0800

    [SPARK-54325][TESTS] Use custom matcher in ExecutorSuite testThrowable
    
    ### What changes were proposed in this pull request?
    Use custom `BeEqualToIsFatal` matcher instead of `assert` in 
`ExecutorSuite.testThrowable`
    
    ### Why are the changes needed?
    1. simplify `assert` expression and make check more readable
    2. improve error message and print exception stack trace
    3. help with troubleshooting test failure 
(https://github.com/vrozov/spark/actions/runs/19272277095/job/55103520062)
    ```
    Executor.isFatalError(java.lang.OutOfMemoryError) is true != false: 
java.lang.OutOfMemoryError
            at 
org.apache.spark.executor.ExecutorSuite.$anonfun$new$39(ExecutorSuite.scala:537)
            at 
org.apache.spark.executor.ExecutorSuite.testThrowable$1(ExecutorSuite.scala:524)
            at 
org.apache.spark.executor.ExecutorSuite.$anonfun$new$38(ExecutorSuite.scala:537)
            at scala.collection.immutable.Range.foreach$mVc$sp(Range.scala:256)
            at 
org.apache.spark.executor.ExecutorSuite.$anonfun$new$33(ExecutorSuite.scala:536)
            at 
scala.runtime.java8.JFunction0$mcV$sp.apply(JFunction0$mcV$sp.scala:18)
            at 
org.scalatest.enablers.Timed$$anon$1.timeoutAfter(Timed.scala:127)
            at 
org.scalatest.concurrent.TimeLimits$.failAfterImpl(TimeLimits.scala:282)
            at 
org.scalatest.concurrent.TimeLimits.failAfter(TimeLimits.scala:231)
            at 
org.scalatest.concurrent.TimeLimits.failAfter$(TimeLimits.scala:230)
            at org.apache.spark.SparkFunSuite.failAfter(SparkFunSuite.scala:68)
            at 
org.apache.spark.SparkFunSuite.$anonfun$test$2(SparkFunSuite.scala:154)
            at org.scalatest.OutcomeOf.outcomeOf(OutcomeOf.scala:85)
            at org.scalatest.OutcomeOf.outcomeOf$(OutcomeOf.scala:83)
            at org.scalatest.OutcomeOf$.outcomeOf(OutcomeOf.scala:104)
            at org.scalatest.Transformer.apply(Transformer.scala:22)
            at org.scalatest.Transformer.apply(Transformer.scala:20)
            at 
org.scalatest.funsuite.AnyFunSuiteLike$$anon$1.apply(AnyFunSuiteLike.scala:226)
            at 
org.apache.spark.SparkFunSuite.withFixture(SparkFunSuite.scala:226)
            at 
org.scalatest.funsuite.AnyFunSuiteLike.invokeWithFixture$1(AnyFunSuiteLike.scala:224)
            at 
org.scalatest.funsuite.AnyFunSuiteLike.$anonfun$runTest$1(AnyFunSuiteLike.scala:236)
            at org.scalatest.SuperEngine.runTestImpl(Engine.scala:306)
            at 
org.scalatest.funsuite.AnyFunSuiteLike.runTest(AnyFunSuiteLike.scala:236)
            at 
org.scalatest.funsuite.AnyFunSuiteLike.runTest$(AnyFunSuiteLike.scala:218)
            at 
org.apache.spark.SparkFunSuite.org$scalatest$BeforeAndAfterEach$$super$runTest(SparkFunSuite.scala:68)
            at 
org.scalatest.BeforeAndAfterEach.runTest(BeforeAndAfterEach.scala:234)
            at 
org.scalatest.BeforeAndAfterEach.runTest$(BeforeAndAfterEach.scala:227)
            at org.apache.spark.SparkFunSuite.runTest(SparkFunSuite.scala:68)
            at 
org.scalatest.funsuite.AnyFunSuiteLike.$anonfun$runTests$1(AnyFunSuiteLike.scala:269)
            at 
org.scalatest.SuperEngine.$anonfun$runTestsInBranch$1(Engine.scala:413)
            at scala.collection.immutable.List.foreach(List.scala:323)
            at org.scalatest.SuperEngine.traverseSubNodes$1(Engine.scala:401)
            at org.scalatest.SuperEngine.runTestsInBranch(Engine.scala:396)
            at org.scalatest.SuperEngine.runTestsImpl(Engine.scala:475)
            at 
org.scalatest.funsuite.AnyFunSuiteLike.runTests(AnyFunSuiteLike.scala:269)
            at 
org.scalatest.funsuite.AnyFunSuiteLike.runTests$(AnyFunSuiteLike.scala:268)
            at 
org.scalatest.funsuite.AnyFunSuite.runTests(AnyFunSuite.scala:1564)
            at org.scalatest.Suite.run(Suite.scala:1114)
            at org.scalatest.Suite.run$(Suite.scala:1096)
            at 
org.scalatest.funsuite.AnyFunSuite.org$scalatest$funsuite$AnyFunSuiteLike$$super$run(AnyFunSuite.scala:1564)
            at 
org.scalatest.funsuite.AnyFunSuiteLike.$anonfun$run$1(AnyFunSuiteLike.scala:273)
            at org.scalatest.SuperEngine.runImpl(Engine.scala:535)
            at 
org.scalatest.funsuite.AnyFunSuiteLike.run(AnyFunSuiteLike.scala:273)
            at 
org.scalatest.funsuite.AnyFunSuiteLike.run$(AnyFunSuiteLike.scala:272)
            at 
org.apache.spark.SparkFunSuite.org$scalatest$BeforeAndAfterAll$$super$run(SparkFunSuite.scala:68)
            at 
org.scalatest.BeforeAndAfterAll.liftedTree1$1(BeforeAndAfterAll.scala:213)
            at org.scalatest.BeforeAndAfterAll.run(BeforeAndAfterAll.scala:210)
            at org.scalatest.BeforeAndAfterAll.run$(BeforeAndAfterAll.scala:208)
            at org.apache.spark.SparkFunSuite.run(SparkFunSuite.scala:68)
            at org.scalatest.tools.SuiteRunner.run(SuiteRunner.scala:47)
            at 
org.scalatest.tools.Runner$.$anonfun$doRunRunRunDaDoRunRun$13(Runner.scala:1321)
            at 
org.scalatest.tools.Runner$.$anonfun$doRunRunRunDaDoRunRun$13$adapted(Runner.scala:1315)
            at scala.collection.immutable.List.foreach(List.scala:323)
            at 
org.scalatest.tools.Runner$.doRunRunRunDaDoRunRun(Runner.scala:1315)
            at 
org.scalatest.tools.Runner$.$anonfun$runOptionallyWithPassFailReporter$24(Runner.scala:992)
            at 
org.scalatest.tools.Runner$.$anonfun$runOptionallyWithPassFailReporter$24$adapted(Runner.scala:970)
            at 
org.scalatest.tools.Runner$.withClassLoaderAndDispatchReporter(Runner.scala:1481)
            at 
org.scalatest.tools.Runner$.runOptionallyWithPassFailReporter(Runner.scala:970)
            at org.scalatest.tools.Runner$.run(Runner.scala:798)
            at org.scalatest.tools.Runner.run(Runner.scala)
            at 
org.jetbrains.plugins.scala.testingSupport.scalaTest.ScalaTestRunner.runScalaTest2or3(ScalaTestRunner.java:43)
            at 
org.jetbrains.plugins.scala.testingSupport.scalaTest.ScalaTestRunner.main(ScalaTestRunner.java:26)
    
    ScalaTestFailureLocation: org.apache.spark.executor.ExecutorSuite at 
(ExecutorSuite.scala:524)
    org.scalatest.exceptions.TestFailedException: 
Executor.isFatalError(java.lang.OutOfMemoryError) is true != false: 
java.lang.OutOfMemoryError
            at 
org.apache.spark.executor.ExecutorSuite.$anonfun$new$39(ExecutorSuite.scala:537)
            at 
org.apache.spark.executor.ExecutorSuite.testThrowable$1(ExecutorSuite.scala:524)
            at 
org.apache.spark.executor.ExecutorSuite.$anonfun$new$38(ExecutorSuite.scala:537)
            at scala.collection.immutable.Range.foreach$mVc$sp(Range.scala:256)
            at 
org.apache.spark.executor.ExecutorSuite.$anonfun$new$33(ExecutorSuite.scala:536)
            at 
scala.runtime.java8.JFunction0$mcV$sp.apply(JFunction0$mcV$sp.scala:18)
            at 
org.scalatest.enablers.Timed$$anon$1.timeoutAfter(Timed.scala:127)
            at 
org.scalatest.concurrent.TimeLimits$.failAfterImpl(TimeLimits.scala:282)
            at 
org.scalatest.concurrent.TimeLimits.failAfter(TimeLimits.scala:231)
            at 
org.scalatest.concurrent.TimeLimits.failAfter$(TimeLimits.scala:230)
            at org.apache.spark.SparkFunSuite.failAfter(SparkFunSuite.scala:68)
            at 
org.apache.spark.SparkFunSuite.$anonfun$test$2(SparkFunSuite.scala:154)
            at org.scalatest.OutcomeOf.outcomeOf(OutcomeOf.scala:85)
            at org.scalatest.OutcomeOf.outcomeOf$(OutcomeOf.scala:83)
            at org.scalatest.OutcomeOf$.outcomeOf(OutcomeOf.scala:104)
            at org.scalatest.Transformer.apply(Transformer.scala:22)
            at org.scalatest.Transformer.apply(Transformer.scala:20)
            at 
org.scalatest.funsuite.AnyFunSuiteLike$$anon$1.apply(AnyFunSuiteLike.scala:226)
            at 
org.apache.spark.SparkFunSuite.withFixture(SparkFunSuite.scala:226)
            at 
org.scalatest.funsuite.AnyFunSuiteLike.invokeWithFixture$1(AnyFunSuiteLike.scala:224)
            at 
org.scalatest.funsuite.AnyFunSuiteLike.$anonfun$runTest$1(AnyFunSuiteLike.scala:236)
            at org.scalatest.SuperEngine.runTestImpl(Engine.scala:306)
            at 
org.scalatest.funsuite.AnyFunSuiteLike.runTest(AnyFunSuiteLike.scala:236)
            at 
org.scalatest.funsuite.AnyFunSuiteLike.runTest$(AnyFunSuiteLike.scala:218)
            at 
org.apache.spark.SparkFunSuite.org$scalatest$BeforeAndAfterEach$$super$runTest(SparkFunSuite.scala:68)
            at 
org.scalatest.BeforeAndAfterEach.runTest(BeforeAndAfterEach.scala:234)
            at 
org.scalatest.BeforeAndAfterEach.runTest$(BeforeAndAfterEach.scala:227)
            at org.apache.spark.SparkFunSuite.runTest(SparkFunSuite.scala:68)
            at 
org.scalatest.funsuite.AnyFunSuiteLike.$anonfun$runTests$1(AnyFunSuiteLike.scala:269)
            at 
org.scalatest.SuperEngine.$anonfun$runTestsInBranch$1(Engine.scala:413)
            at scala.collection.immutable.List.foreach(List.scala:323)
            at org.scalatest.SuperEngine.traverseSubNodes$1(Engine.scala:401)
            at org.scalatest.SuperEngine.runTestsInBranch(Engine.scala:396)
            at org.scalatest.SuperEngine.runTestsImpl(Engine.scala:475)
            at 
org.scalatest.funsuite.AnyFunSuiteLike.runTests(AnyFunSuiteLike.scala:269)
            at 
org.scalatest.funsuite.AnyFunSuiteLike.runTests$(AnyFunSuiteLike.scala:268)
            at 
org.scalatest.funsuite.AnyFunSuite.runTests(AnyFunSuite.scala:1564)
            at org.scalatest.Suite.run(Suite.scala:1114)
            at org.scalatest.Suite.run$(Suite.scala:1096)
            at 
org.scalatest.funsuite.AnyFunSuite.org$scalatest$funsuite$AnyFunSuiteLike$$super$run(AnyFunSuite.scala:1564)
            at 
org.scalatest.funsuite.AnyFunSuiteLike.$anonfun$run$1(AnyFunSuiteLike.scala:273)
            at org.scalatest.SuperEngine.runImpl(Engine.scala:535)
            at 
org.scalatest.funsuite.AnyFunSuiteLike.run(AnyFunSuiteLike.scala:273)
            at 
org.scalatest.funsuite.AnyFunSuiteLike.run$(AnyFunSuiteLike.scala:272)
            at 
org.apache.spark.SparkFunSuite.org$scalatest$BeforeAndAfterAll$$super$run(SparkFunSuite.scala:68)
            at 
org.scalatest.BeforeAndAfterAll.liftedTree1$1(BeforeAndAfterAll.scala:213)
            at org.scalatest.BeforeAndAfterAll.run(BeforeAndAfterAll.scala:210)
            at org.scalatest.BeforeAndAfterAll.run$(BeforeAndAfterAll.scala:208)
            at org.apache.spark.SparkFunSuite.run(SparkFunSuite.scala:68)
            at org.scalatest.tools.SuiteRunner.run(SuiteRunner.scala:47)
            at 
org.scalatest.tools.Runner$.$anonfun$doRunRunRunDaDoRunRun$13(Runner.scala:1321)
            at 
org.scalatest.tools.Runner$.$anonfun$doRunRunRunDaDoRunRun$13$adapted(Runner.scala:1315)
            at scala.collection.immutable.List.foreach(List.scala:323)
            at 
org.scalatest.tools.Runner$.doRunRunRunDaDoRunRun(Runner.scala:1315)
            at 
org.scalatest.tools.Runner$.$anonfun$runOptionallyWithPassFailReporter$24(Runner.scala:992)
            at 
org.scalatest.tools.Runner$.$anonfun$runOptionallyWithPassFailReporter$24$adapted(Runner.scala:970)
            at 
org.scalatest.tools.Runner$.withClassLoaderAndDispatchReporter(Runner.scala:1481)
            at 
org.scalatest.tools.Runner$.runOptionallyWithPassFailReporter(Runner.scala:970)
            at org.scalatest.tools.Runner$.run(Runner.scala:798)
            at org.scalatest.tools.Runner.run(Runner.scala)
            at 
org.jetbrains.plugins.scala.testingSupport.scalaTest.ScalaTestRunner.runScalaTest2or3(ScalaTestRunner.java:43)
            at 
org.jetbrains.plugins.scala.testingSupport.scalaTest.ScalaTestRunner.main(ScalaTestRunner.java:26)
    
            at 
org.scalatest.matchers.MatchersHelper$.indicateFailure(MatchersHelper.scala:392)
            at 
org.scalatest.matchers.should.Matchers$ShouldMethodHelperClass.shouldMatcher(Matchers.scala:7304)
            at 
org.scalatest.matchers.should.Matchers$AnyShouldWrapper.should(Matchers.scala:7347)
            at 
org.apache.spark.executor.ExecutorSuite.testThrowable$1(ExecutorSuite.scala:524)
            at 
org.apache.spark.executor.ExecutorSuite.$anonfun$new$38(ExecutorSuite.scala:537)
            at scala.collection.immutable.Range.foreach$mVc$sp(Range.scala:256)
            at 
org.apache.spark.executor.ExecutorSuite.$anonfun$new$33(ExecutorSuite.scala:536)
            at 
scala.runtime.java8.JFunction0$mcV$sp.apply(JFunction0$mcV$sp.scala:18)
            at 
org.scalatest.enablers.Timed$$anon$1.timeoutAfter(Timed.scala:127)
            at 
org.scalatest.concurrent.TimeLimits$.failAfterImpl(TimeLimits.scala:282)
            at 
org.scalatest.concurrent.TimeLimits.failAfter(TimeLimits.scala:231)
            at 
org.scalatest.concurrent.TimeLimits.failAfter$(TimeLimits.scala:230)
            at org.apache.spark.SparkFunSuite.failAfter(SparkFunSuite.scala:68)
            at 
org.apache.spark.SparkFunSuite.$anonfun$test$2(SparkFunSuite.scala:154)
            at org.scalatest.OutcomeOf.outcomeOf(OutcomeOf.scala:85)
            at org.scalatest.OutcomeOf.outcomeOf$(OutcomeOf.scala:83)
            at org.scalatest.OutcomeOf$.outcomeOf(OutcomeOf.scala:104)
            at org.scalatest.Transformer.apply(Transformer.scala:22)
            at org.scalatest.Transformer.apply(Transformer.scala:20)
            at 
org.scalatest.funsuite.AnyFunSuiteLike$$anon$1.apply(AnyFunSuiteLike.scala:226)
            at 
org.apache.spark.SparkFunSuite.withFixture(SparkFunSuite.scala:226)
            at 
org.scalatest.funsuite.AnyFunSuiteLike.invokeWithFixture$1(AnyFunSuiteLike.scala:224)
            at 
org.scalatest.funsuite.AnyFunSuiteLike.$anonfun$runTest$1(AnyFunSuiteLike.scala:236)
            at org.scalatest.SuperEngine.runTestImpl(Engine.scala:306)
            at 
org.scalatest.funsuite.AnyFunSuiteLike.runTest(AnyFunSuiteLike.scala:236)
            at 
org.scalatest.funsuite.AnyFunSuiteLike.runTest$(AnyFunSuiteLike.scala:218)
            at 
org.apache.spark.SparkFunSuite.org$scalatest$BeforeAndAfterEach$$super$runTest(SparkFunSuite.scala:68)
            at 
org.scalatest.BeforeAndAfterEach.runTest(BeforeAndAfterEach.scala:234)
            at 
org.scalatest.BeforeAndAfterEach.runTest$(BeforeAndAfterEach.scala:227)
            at org.apache.spark.SparkFunSuite.runTest(SparkFunSuite.scala:68)
            at 
org.scalatest.funsuite.AnyFunSuiteLike.$anonfun$runTests$1(AnyFunSuiteLike.scala:269)
            at 
org.scalatest.SuperEngine.$anonfun$runTestsInBranch$1(Engine.scala:413)
            at scala.collection.immutable.List.foreach(List.scala:323)
            at org.scalatest.SuperEngine.traverseSubNodes$1(Engine.scala:401)
            at org.scalatest.SuperEngine.runTestsInBranch(Engine.scala:396)
            at org.scalatest.SuperEngine.runTestsImpl(Engine.scala:475)
            at 
org.scalatest.funsuite.AnyFunSuiteLike.runTests(AnyFunSuiteLike.scala:269)
            at 
org.scalatest.funsuite.AnyFunSuiteLike.runTests$(AnyFunSuiteLike.scala:268)
            at 
org.scalatest.funsuite.AnyFunSuite.runTests(AnyFunSuite.scala:1564)
            at org.scalatest.Suite.run(Suite.scala:1114)
            at org.scalatest.Suite.run$(Suite.scala:1096)
            at 
org.scalatest.funsuite.AnyFunSuite.org$scalatest$funsuite$AnyFunSuiteLike$$super$run(AnyFunSuite.scala:1564)
            at 
org.scalatest.funsuite.AnyFunSuiteLike.$anonfun$run$1(AnyFunSuiteLike.scala:273)
            at org.scalatest.SuperEngine.runImpl(Engine.scala:535)
            at 
org.scalatest.funsuite.AnyFunSuiteLike.run(AnyFunSuiteLike.scala:273)
            at 
org.scalatest.funsuite.AnyFunSuiteLike.run$(AnyFunSuiteLike.scala:272)
            at 
org.apache.spark.SparkFunSuite.org$scalatest$BeforeAndAfterAll$$super$run(SparkFunSuite.scala:68)
            at 
org.scalatest.BeforeAndAfterAll.liftedTree1$1(BeforeAndAfterAll.scala:213)
            at org.scalatest.BeforeAndAfterAll.run(BeforeAndAfterAll.scala:210)
            at org.scalatest.BeforeAndAfterAll.run$(BeforeAndAfterAll.scala:208)
            at org.apache.spark.SparkFunSuite.run(SparkFunSuite.scala:68)
            at org.scalatest.tools.SuiteRunner.run(SuiteRunner.scala:47)
            at 
org.scalatest.tools.Runner$.$anonfun$doRunRunRunDaDoRunRun$13(Runner.scala:1321)
            at 
org.scalatest.tools.Runner$.$anonfun$doRunRunRunDaDoRunRun$13$adapted(Runner.scala:1315)
            at scala.collection.immutable.List.foreach(List.scala:323)
            at 
org.scalatest.tools.Runner$.doRunRunRunDaDoRunRun(Runner.scala:1315)
            at 
org.scalatest.tools.Runner$.$anonfun$runOptionallyWithPassFailReporter$24(Runner.scala:992)
            at 
org.scalatest.tools.Runner$.$anonfun$runOptionallyWithPassFailReporter$24$adapted(Runner.scala:970)
            at 
org.scalatest.tools.Runner$.withClassLoaderAndDispatchReporter(Runner.scala:1481)
            at 
org.scalatest.tools.Runner$.runOptionallyWithPassFailReporter(Runner.scala:970)
            at org.scalatest.tools.Runner$.run(Runner.scala:798)
            at org.scalatest.tools.Runner.run(Runner.scala)
            at 
org.jetbrains.plugins.scala.testingSupport.scalaTest.ScalaTestRunner.runScalaTest2or3(ScalaTestRunner.java:43)
            at 
org.jetbrains.plugins.scala.testingSupport.scalaTest.ScalaTestRunner.main(ScalaTestRunner.java:26)
    ```
    
    ### Does this PR introduce _any_ user-facing change?
    No
    
    ### How was this patch tested?
    execute affected test
    
    ### Was this patch authored or co-authored using generative AI tooling?
    No
    
    Closes #53022 from vrozov/SPARK-54325.
    
    Authored-by: Vlad Rozov <[email protected]>
    Signed-off-by: Holden Karau <[email protected]>
---
 .../org/apache/spark/executor/ExecutorSuite.scala  | 45 +++++++++++++---------
 1 file changed, 27 insertions(+), 18 deletions(-)

diff --git a/core/src/test/scala/org/apache/spark/executor/ExecutorSuite.scala 
b/core/src/test/scala/org/apache/spark/executor/ExecutorSuite.scala
index fec71a5285ed..f63cf4f05d61 100644
--- a/core/src/test/scala/org/apache/spark/executor/ExecutorSuite.scala
+++ b/core/src/test/scala/org/apache/spark/executor/ExecutorSuite.scala
@@ -17,7 +17,7 @@
 
 package org.apache.spark.executor
 
-import java.io.{Externalizable, ObjectInput, ObjectOutput}
+import java.io.{Externalizable, ObjectInput, ObjectOutput, PrintWriter, 
StringWriter}
 import java.lang.Thread.UncaughtExceptionHandler
 import java.net.URL
 import java.nio.ByteBuffer
@@ -36,9 +36,10 @@ import org.mockito.ArgumentMatchers.{any, eq => meq}
 import org.mockito.Mockito.{inOrder, verify, when}
 import org.mockito.invocation.InvocationOnMock
 import org.mockito.stubbing.Answer
-import org.scalatest.Assertions._
 import org.scalatest.PrivateMethodTester
 import org.scalatest.concurrent.Eventually
+import org.scalatest.matchers.{Matcher, MatchResult}
+import org.scalatest.matchers.should.Matchers._
 import org.scalatestplus.mockito.MockitoSugar
 
 import org.apache.spark._
@@ -507,25 +508,33 @@ class ExecutorSuite extends SparkFunSuite
         e: => Throwable,
         depthToCheck: Int,
         isFatal: Boolean): Unit = {
-      import Executor.isFatalError
+
+      class BeFatalError(isFatal: Boolean) extends Matcher[Throwable] {
+        override def apply(t: Throwable): MatchResult = {
+          val stringWriter = new StringWriter()
+          t.printStackTrace(new PrintWriter(stringWriter))
+          val isFatalError = Executor.isFatalError(t, depthToCheck)
+          MatchResult(
+            isFatalError == isFatal,
+            s"Executor.isFatalError($t) is $isFatalError != $isFatal: " + 
stringWriter.toString,
+            s"Executor.isFatalError($t) is $isFatalError == $isFatal: " + 
stringWriter.toString
+          )
+        }
+      }
+
+      def beFatalError(isFatal: Boolean) = new BeFatalError(isFatal)
+
       // `e`'s depth is 1 so `depthToCheck` needs to be at least 3 to detect 
fatal errors.
-      assert(isFatalError(e, depthToCheck) == (depthToCheck >= 1 && isFatal))
+      e should beFatalError(depthToCheck >= 1 && isFatal)
       // `e`'s depth is 2 so `depthToCheck` needs to be at least 3 to detect 
fatal errors.
-      assert(isFatalError(errorInThreadPool(e), depthToCheck) == (depthToCheck 
>= 2 && isFatal))
-      assert(isFatalError(errorInGuavaCache(e), depthToCheck) == (depthToCheck 
>= 2 && isFatal))
-      assert(isFatalError(
-        new SparkException("foo", e),
-        depthToCheck) == (depthToCheck >= 2 && isFatal))
+      errorInThreadPool(e) should beFatalError(depthToCheck >= 2 && isFatal)
+      errorInGuavaCache(e) should beFatalError(depthToCheck >= 2 && isFatal)
+      new SparkException("foo", e) should beFatalError(depthToCheck >= 2 && 
isFatal)
       // `e`'s depth is 3 so `depthToCheck` needs to be at least 3 to detect 
fatal errors.
-      assert(isFatalError(
-        errorInThreadPool(errorInGuavaCache(e)),
-        depthToCheck) == (depthToCheck >= 3 && isFatal))
-      assert(isFatalError(
-        errorInGuavaCache(errorInThreadPool(e)),
-        depthToCheck) == (depthToCheck >= 3 && isFatal))
-      assert(isFatalError(
-        new SparkException("foo", new SparkException("foo", e)),
-        depthToCheck) == (depthToCheck >= 3 && isFatal))
+      errorInThreadPool(errorInGuavaCache(e)) should beFatalError(depthToCheck 
>= 3 && isFatal)
+      errorInGuavaCache(errorInThreadPool(e)) should beFatalError(depthToCheck 
>= 3 && isFatal)
+      new SparkException("foo", new SparkException("foo", e)) should
+        beFatalError(depthToCheck >= 3 && isFatal)
     }
 
     for (depthToCheck <- 0 to 5) {


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to