[spark] branch master updated: [SPARK-40163][SQL][TESTS][FOLLOWUP] Use Junit `Assert` api instead of Java `assert` in `JavaSparkSessionSuite.java`

[sr...@apache.org]

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 <ya...@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 <ya...@baidu.com>
    Signed-off-by: Sean Owen <sr...@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-unsubscribe@spark.apache.org
For additional commands, e-mail: commits-help@spark.apache.org