From f97da163806277f373e803ebd29ea96de864041a Mon Sep 17 00:00:00 2001 From: Hyukjin Kwon Date: Mon, 11 Mar 2024 10:59:37 -0700 Subject: [PATCH] [SPARK-45245][CONNECT][TESTS][FOLLOW-UP] Remove unneeded Matchers trait in the test ### What changes were proposed in this pull request? This PR is a followup of https://github.com/apache/spark/pull/43023 that addresses a post-review comment. ### Why are the changes needed? It is unnecessary. It also matters with Scala compatibility so should better remove if unused. ### Does this PR introduce _any_ user-facing change? No, test-only. ### How was this patch tested? Manually. ### Was this patch authored or co-authored using generative AI tooling? No. Closes #45459 from HyukjinKwon/SPARK-45245-folllowup. Lead-authored-by: Hyukjin Kwon Co-authored-by: Hyukjin Kwon Signed-off-by: Dongjoon Hyun --- .../apache/spark/api/python/PythonWorkerFactorySuite.scala | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/core/src/test/scala/org/apache/spark/api/python/PythonWorkerFactorySuite.scala b/core/src/test/scala/org/apache/spark/api/python/PythonWorkerFactorySuite.scala index 34c10bd95ed75..359b76c87a2a5 100644 --- a/core/src/test/scala/org/apache/spark/api/python/PythonWorkerFactorySuite.scala +++ b/core/src/test/scala/org/apache/spark/api/python/PythonWorkerFactorySuite.scala @@ -25,15 +25,13 @@ import scala.concurrent.ExecutionContext.Implicits.global import scala.concurrent.Future import scala.concurrent.duration._ -import org.scalatest.matchers.must.Matchers - import org.apache.spark.SharedSparkContext import org.apache.spark.SparkException import org.apache.spark.SparkFunSuite import org.apache.spark.util.ThreadUtils // Tests for PythonWorkerFactory. -class PythonWorkerFactorySuite extends SparkFunSuite with Matchers with SharedSparkContext { +class PythonWorkerFactorySuite extends SparkFunSuite with SharedSparkContext { test("createSimpleWorker() fails with a timeout error if worker does not connect back") { // It verifies that server side times out in accept(), if the worker does not connect back.