Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[SPARK-18296][SQL] Use consistent naming for expression test suites #15793

Closed
wants to merge 1 commit into from
Closed

[SPARK-18296][SQL] Use consistent naming for expression test suites #15793

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion ...t/expressions/BitwiseFunctionsSuite.scala → ...expressions/BitwiseExpressionsSuite.scala
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@ import org.apache.spark.SparkFunSuite
import org.apache.spark.sql.types._


class BitwiseFunctionsSuite extends SparkFunSuite with ExpressionEvalHelper {
class BitwiseExpressionsSuite extends SparkFunSuite with ExpressionEvalHelper {

import IntegralLiteralTestUtils._

Expand Down
3 changes: 1 addition & 2 deletions ...xpressions/CollectionFunctionsSuite.scala → ...ressions/CollectionExpressionsSuite.scala
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -20,8 +20,7 @@ package org.apache.spark.sql.catalyst.expressions
import org.apache.spark.SparkFunSuite
import org.apache.spark.sql.types._


class CollectionFunctionsSuite extends SparkFunSuite with ExpressionEvalHelper {
class CollectionExpressionsSuite extends SparkFunSuite with ExpressionEvalHelper {

test("Array and Map Size") {
val a0 = Literal.create(Seq(1, 2, 3), ArrayType(IntegerType))
Expand Down
2 changes: 1 addition & 1 deletion ...lyst/expressions/MathFunctionsSuite.scala → ...st/expressions/MathExpressionsSuite.scala
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@ import org.apache.spark.sql.catalyst.optimizer.SimpleTestOptimizer
import org.apache.spark.sql.catalyst.plans.logical.{OneRowRelation, Project}
import org.apache.spark.sql.types._

class MathFunctionsSuite extends SparkFunSuite with ExpressionEvalHelper {
class MathExpressionsSuite extends SparkFunSuite with ExpressionEvalHelper {

import IntegralLiteralTestUtils._

Expand Down
2 changes: 1 addition & 1 deletion ...lyst/expressions/MiscFunctionsSuite.scala → ...st/expressions/MiscExpressionsSuite.scala
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ package org.apache.spark.sql.catalyst.expressions
import org.apache.spark.SparkFunSuite
import org.apache.spark.sql.types._

class MiscFunctionsSuite extends SparkFunSuite with ExpressionEvalHelper {
class MiscExpressionsSuite extends SparkFunSuite with ExpressionEvalHelper {

test("assert_true") {
intercept[RuntimeException] {
Expand Down
2 changes: 1 addition & 1 deletion ...lyst/expressions/NullFunctionsSuite.scala → ...st/expressions/NullExpressionsSuite.scala
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@ import org.apache.spark.sql.catalyst.expressions.objects.AssertNotNull
import org.apache.spark.sql.catalyst.plans.logical.{LocalRelation, Project}
import org.apache.spark.sql.types._

class NullFunctionsSuite extends SparkFunSuite with ExpressionEvalHelper {
class NullExpressionsSuite extends SparkFunSuite with ExpressionEvalHelper {

def testAllTypes(testFunc: (Any, DataType) => Unit): Unit = {
testFunc(false, BooleanType)
Expand Down
6 changes: 3 additions & 3 deletions ...ache/spark/sql/MathExpressionsSuite.scala → ...apache/spark/sql/MathFunctionsSuite.scala
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -23,13 +23,13 @@ import org.apache.spark.sql.functions._
import org.apache.spark.sql.functions.{log => logarithm}
import org.apache.spark.sql.test.SharedSQLContext

private object MathExpressionsTestData {
private object MathFunctionsTestData {
case class DoubleData(a: java.lang.Double, b: java.lang.Double)
case class NullDoubles(a: java.lang.Double)
}

class MathExpressionsSuite extends QueryTest with SharedSQLContext {
import MathExpressionsTestData._
class MathFunctionsSuite extends QueryTest with SharedSQLContext {
import MathFunctionsTestData._
import testImplicits._

private lazy val doubleData = (1 to 10).map(i => DoubleData(i * 0.2 - 1, i * -0.2 + 1)).toDF()
Expand Down