diff --git a/core/src/index/cmake/ThirdPartyPackagesCore.cmake b/core/src/index/cmake/ThirdPartyPackagesCore.cmake index 35b9f49b019d..5caf49465980 100644 --- a/core/src/index/cmake/ThirdPartyPackagesCore.cmake +++ b/core/src/index/cmake/ThirdPartyPackagesCore.cmake @@ -545,6 +545,8 @@ macro(build_faiss) --without-cuda) endif () + message(STATUS "Building FAISS with configure args -${FAISS_CONFIGURE_ARGS}") + if (DEFINED ENV{FAISS_SOURCE_URL}) set(FAISS_SOURCE_URL "$ENV{FAISS_SOURCE_URL}") externalproject_add(faiss_ep diff --git a/tests/milvus_python_test/test_ping.py b/tests/milvus_python_test/test_ping.py index c3d382efd92b..c26b2c2c666f 100644 --- a/tests/milvus_python_test/test_ping.py +++ b/tests/milvus_python_test/test_ping.py @@ -9,7 +9,7 @@ def test_server_version(self, connect): ''' target: test get the server version method: call the server_version method after connected - expected: version should be the pymilvus version + expected: version should be the milvus version ''' status, res = connect.server_version() assert res == __version__ @@ -54,6 +54,58 @@ def test_connected(self, connect): assert connect +class TestPingWithTimeout: + def test_server_version_legal_timeout(self, connect): + ''' + target: test get the server version with legal timeout + method: call the server_version method after connected with altering timeout + expected: version should be the milvus version + ''' + status, res = connect.server_version(20) + assert res == __version__ + + def test_server_version_negative_timeout(self, connect): + ''' + target: test get the server version with negative timeout + method: call the server_version method after connected with altering timeout + expected: when timeout is illegal raises an error; + ''' + status, res = connect.server_version(-1) + assert not status.OK() + + def test_server_cmd_with_params_version_with_legal_timeout(self, connect): + ''' + target: test cmd: version and timeout + method: cmd = "version" , timeout=10 + expected: when cmd = 'version', return version of server; + ''' + cmd = "version" + status, msg = connect._cmd(cmd, 10) + logging.getLogger().info(status) + logging.getLogger().info(msg) + assert status.OK() + assert msg == __version__ + + def test_server_cmd_with_params_version_with_illegal_timeout(self, connect): + ''' + target: test cmd: version and timeout + method: cmd = "version" , timeout=-1 + expected: when timeout is illegal raises an error; + ''' + status, res = connect.server_version(-1) + assert not status.OK() + + def test_server_cmd_with_params_others_with_illegal_timeout(self, connect): + ''' + target: test cmd: lalala, timeout = -1 + method: cmd = "lalala", timeout = -1 + expected: when timeout is illegal raises an error; + ''' + cmd = "rm -rf test" + status, res = connect.server_version(-1) + assert not status.OK() + + class TestPingDisconnect: def test_server_version(self, dis_connect): ''' @@ -76,3 +128,14 @@ def test_server_status(self, dis_connect): with pytest.raises(Exception) as e: status, msg = connect.server_status() assert status is None + + def test_server_version_with_timeout(self, dis_connect): + ''' + target: test get the server status with timeout settings after disconnect + method: call the server_status method after connected + expected: status returned should be not ok + ''' + status = None + with pytest.raises(Exception) as e: + status, msg = connect.server_status(100) + assert status is None