From 4aa93d3cbdb71ca6c8eb91bd426fdd5191d03f02 Mon Sep 17 00:00:00 2001 From: yanghua Date: Thu, 19 Aug 2021 15:05:40 +0800 Subject: [PATCH] [KYUUBI #957] Reopen KyuubiServerSuite ### _Why are the changes needed?_ ### _How was this patch tested?_ - [ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible - [ ] Add screenshots for manual tests if appropriate - [ ] [Run test](https://kyuubi.readthedocs.io/en/latest/develop_tools/testing.html#running-tests) locally before make a pull request Closes #960 from yanghua/KYUUBI-957. Closes #957 94cdddda [yanghua] [KYUUBI #957] Reopen KyuubiServerSuite Authored-by: yanghua Signed-off-by: Cheng Pan --- .../scala/org/apache/kyuubi/server/KyuubiServerSuite.scala | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/kyuubi-server/src/test/scala/org/apache/kyuubi/server/KyuubiServerSuite.scala b/kyuubi-server/src/test/scala/org/apache/kyuubi/server/KyuubiServerSuite.scala index cfd2b73d1..0df2cdebd 100644 --- a/kyuubi-server/src/test/scala/org/apache/kyuubi/server/KyuubiServerSuite.scala +++ b/kyuubi-server/src/test/scala/org/apache/kyuubi/server/KyuubiServerSuite.scala @@ -23,7 +23,7 @@ import org.apache.kyuubi.service.ServiceState._ class KyuubiServerSuite extends KyuubiFunSuite { - ignore("kyuubi server basic") { + test("kyuubi server basic") { val server = new KyuubiServer() server.stop() val conf = KyuubiConf().set(KyuubiConf.FRONTEND_BIND_PORT, 0) @@ -35,7 +35,7 @@ class KyuubiServerSuite extends KyuubiFunSuite { server.initialize(conf) assert(server.getServiceState === INITIALIZED) - val backendService = server.getServices(1).asInstanceOf[KyuubiBackendService] + val backendService = server.getServices(2).asInstanceOf[KyuubiBackendService] assert(backendService.getServiceState == INITIALIZED) assert(backendService.getServices.forall(_.getServiceState === INITIALIZED)) assert(server.connectionUrl.split(":").length === 2) @@ -57,7 +57,7 @@ class KyuubiServerSuite extends KyuubiFunSuite { server.stop() } - ignore("invalid port") { + test("invalid port") { val conf = KyuubiConf().set(KyuubiConf.FRONTEND_BIND_PORT, 100) val e = intercept[KyuubiException](new KyuubiServer().initialize(conf)) assert(e.getMessage.contains("Failed to initialize frontend service"))