This is an automated email from the ASF dual-hosted git repository. srowen 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 339d6380e1d [SPARK-40163][SQL][TESTS][FOLLOWUP] Use Junit `Assert` api instead of Java `assert` in `JavaSparkSessionSuite.java` 339d6380e1d is described below commit 339d6380e1d00cac5821ddc44349cbcd3f58ad7d Author: yangjie01 <yangji...@baidu.com> AuthorDate: Fri Sep 2 14:49:23 2022 -0500 [SPARK-40163][SQL][TESTS][FOLLOWUP] Use Junit `Assert` api instead of Java `assert` in `JavaSparkSessionSuite.java` ### What changes were proposed in this pull request? This pr is a minor fix of https://github.com/apache/spark/pull/37478, just change to use Junit api to assert in Java suite. ### Why are the changes needed? In Java suites, should use JUnit API to make assertions. ### Does this PR introduce _any_ user-facing change? No ### How was this patch tested? Pass GitHub Actions Closes #37772 from LuciferYang/SPARK-40163-followup. Authored-by: yangjie01 <yangji...@baidu.com> Signed-off-by: Sean Owen <sro...@gmail.com> --- .../src/test/java/test/org/apache/spark/sql/JavaSparkSessionSuite.java | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/sql/core/src/test/java/test/org/apache/spark/sql/JavaSparkSessionSuite.java b/sql/core/src/test/java/test/org/apache/spark/sql/JavaSparkSessionSuite.java index 00f744f4d86..b1df377936d 100644 --- a/sql/core/src/test/java/test/org/apache/spark/sql/JavaSparkSessionSuite.java +++ b/sql/core/src/test/java/test/org/apache/spark/sql/JavaSparkSessionSuite.java @@ -19,6 +19,7 @@ package test.org.apache.spark.sql; import org.apache.spark.sql.*; import org.junit.After; +import org.junit.Assert; import org.junit.Test; import java.util.HashMap; @@ -50,7 +51,7 @@ public class JavaSparkSessionSuite { .getOrCreate(); for (Map.Entry<String, Object> e : map.entrySet()) { - assert(spark.conf().get(e.getKey()).equals(e.getValue().toString())); + Assert.assertEquals(spark.conf().get(e.getKey()), e.getValue().toString()); } } } --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@spark.apache.org For additional commands, e-mail: commits-h...@spark.apache.org