Fixes #82

AbsaOSS · Mar 16, 2023 · cb93a03 · cb93a03
1 parent 6c04072
commit cb93a03
Showing 1 changed file with 7 additions and 1 deletion.
diff --git a/spark-commons/src/test/scala/za/co/absa/spark/commons/OncePerSparkSessionTest.scala b/spark-commons/src/test/scala/za/co/absa/spark/commons/OncePerSparkSessionTest.scala
@@ -66,7 +66,13 @@ class OncePerSparkSessionTest extends AnyFunSuite with MockitoSugar with SparkTe
     assert(UdfLibrary.libraryAInitCounter == 1)
   }
 
-  test("should return false if the library is registered successfully and false if provided that spark session has started") {
+  test("should return false for both results if provided that spark session has started") {
+    /*
+    * It returns false because the constructor will actually register the library before the call for the register
+    * method on line 77.
+    * Up for discussion though.
+    */
+
     var libraryAInitCounter = 0
     val anotherSpark: SparkSession =  mock[SparkSession]
     class UDFLibrary()(implicit sparkToRegisterTo: SparkSession) extends OncePerSparkSession(sparkToRegisterTo) {