Skip to content

Commit

Permalink
Update unit test variable names to match rename
Browse files Browse the repository at this point in the history
  • Loading branch information
PenguinToast committed May 29, 2018
1 parent 586b0e7 commit badbf0e
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions core/src/test/scala/org/apache/spark/SparkConfSuite.scala
Original file line number Diff line number Diff line change
Expand Up @@ -342,7 +342,7 @@ class SparkConfSuite extends SparkFunSuite with LocalSparkContext with ResetSyst
}

val defaultIllegalValue = "SomeIllegalValue"
val illegalArgumentTests : Map[String, (SparkConf, String) => Any] = Map(
val illegalValueTests : Map[String, (SparkConf, String) => Any] = Map(
"getTimeAsSeconds" -> (_.getTimeAsSeconds(_)),
"getTimeAsSeconds with default" -> (_.getTimeAsSeconds(_, defaultIllegalValue)),
"getTimeAsMs" -> (_.getTimeAsMs(_)),
Expand All @@ -362,7 +362,7 @@ class SparkConfSuite extends SparkFunSuite with LocalSparkContext with ResetSyst
"getBoolean" -> (_.getBoolean(_, false))
)

illegalArgumentTests.foreach { case (name, getValue) =>
illegalValueTests.foreach { case (name, getValue) =>
test(s"SPARK-24337: $name throws an useful error message with key name") {
val key = "SomeKey"
val conf = new SparkConf()
Expand Down

0 comments on commit badbf0e

Please sign in to comment.