gleonSun commented on code in PR #8464:
URL: https://github.com/apache/incubator-gluten/pull/8464#discussion_r1908290264
##########
gluten-core/src/test/scala/org/apache/gluten/component/ComponentSuite.scala:
##########
@@ -53,6 +56,18 @@ class ComponentSuite extends AnyFunSuite with
BeforeAndAfterAll {
new DummyBackendA().ensureRegistered()
}
}
+
+ test("Discover without component class") {
+ System.setProperty("java.class.path", classPath)
+ assertThrows[GlutenException](Discovery.discoverAll())
+ }
+
+ // Gluten attempts to load from the "classes" directory instead of
"test-classes,"
+ // but the META-INF is not located in the "classes" directory.
+ test("Discover without component class again") {
+ System.setProperty("java.class.path", "")
Review Comment:
The unit tests are too tricky, and this part of the code will be refactored
later.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
