Repository: spark Updated Branches: refs/heads/master 736fc0393 -> 65a8d1b87
[SPARK-25812][UI][TEST] Fix test failure in PagedTableSuite ## What changes were proposed in this pull request? In https://github.com/apache/spark/pull/22668, the PR was merged without PR builder test. And there is a test failure: https://amplab.cs.berkeley.edu/jenkins/view/Spark%20QA%20Test%20(Dashboard)/job/spark-master-test-sbt-hadoop-2.7/5070/testReport/org.apache.spark.ui/PagedTableSuite/pageNavigation/ This PR is to fix it. ## How was this patch tested? Update the test case. Closes #22808 from gengliangwang/fixPagedTableSuite. Authored-by: Gengliang Wang <gengliang.w...@databricks.com> Signed-off-by: Dongjoon Hyun <dongj...@apache.org> Project: http://git-wip-us.apache.org/repos/asf/spark/repo Commit: http://git-wip-us.apache.org/repos/asf/spark/commit/65a8d1b8 Tree: http://git-wip-us.apache.org/repos/asf/spark/tree/65a8d1b8 Diff: http://git-wip-us.apache.org/repos/asf/spark/diff/65a8d1b8 Branch: refs/heads/master Commit: 65a8d1b87f98b4160c3e4039c97d6a1a5096aaf2 Parents: 736fc03 Author: Gengliang Wang <gengliang.w...@databricks.com> Authored: Tue Oct 23 12:37:45 2018 -0700 Committer: Dongjoon Hyun <dongj...@apache.org> Committed: Tue Oct 23 12:37:45 2018 -0700 ---------------------------------------------------------------------- core/src/test/scala/org/apache/spark/ui/PagedTableSuite.scala | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/spark/blob/65a8d1b8/core/src/test/scala/org/apache/spark/ui/PagedTableSuite.scala ---------------------------------------------------------------------- diff --git a/core/src/test/scala/org/apache/spark/ui/PagedTableSuite.scala b/core/src/test/scala/org/apache/spark/ui/PagedTableSuite.scala index 74eeca2..cda98ae 100644 --- a/core/src/test/scala/org/apache/spark/ui/PagedTableSuite.scala +++ b/core/src/test/scala/org/apache/spark/ui/PagedTableSuite.scala @@ -73,7 +73,7 @@ class PagedTableSuite extends SparkFunSuite { override def goButtonFormPath: String = "" } - assert(pagedTable.pageNavigation(1, 10, 1) === Nil) + assert((pagedTable.pageNavigation(1, 10, 1).head \\ "li").map(_.text.trim) === Seq("1")) assert( (pagedTable.pageNavigation(1, 10, 2).head \\ "li").map(_.text.trim) === Seq("1", "2", ">")) assert( --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@spark.apache.org For additional commands, e-mail: commits-h...@spark.apache.org
