thevar1able/ClickHouse
1
0
Fork 0
mirror of https://github.com/ClickHouse/ClickHouse.git synced 2024-11-29 02:52:13 +00:00
Code Issues Packages Projects Releases Wiki Activity

Fix test_allow_introspection.

Browse Source
This commit is contained in:
Vitaly Baranov 2020-06-28 21:41:41 +03:00
parent eb27814fbe
commit ace9c99e14
4 changed files with 21 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

0
tests/integration/test_settings_constraints_distributed/configs/remote_servers.xml → tests/integration/test_settings_constraints_distributed/configs/config.d/remote_servers.xml
View File

7
tests/integration/test_settings_constraints_distributed/configs/users.d/allow_introspection_functions.xml Normal file
View File

@ -0,0 +1,7 @@
<yandex>
<profiles>
<default>
<allow_introspection_functions>1</allow_introspection_functions>
</default>
</profiles>
</yandex>

6
tests/integration/test_settings_constraints_distributed/test.py
View File

@ -8,9 +8,9 @@ from helpers.test_tools import assert_eq_with_retry
cluster = ClickHouseCluster(__file__) cluster = ClickHouseCluster(__file__)
node1 = cluster.add_instance('node1') node1 = cluster.add_instance('node1', config_dir="configs")
node2 = cluster.add_instance('node2') node2 = cluster.add_instance('node2', config_dir="configs")
distributed = cluster.add_instance('distributed', main_configs=["configs/remote_servers.xml"], stay_alive=True) distributed = cluster.add_instance('distributed', config_dir="configs", stay_alive=True)
@pytest.fixture(scope="module", autouse=True) @pytest.fixture(scope="module", autouse=True)

13
tests/integration/test_settings_profile/test.py
View File

@ -177,11 +177,18 @@ def test_allow_ddl():
def test_allow_introspection(): def test_allow_introspection():
assert "Introspection functions are disabled" in instance.query_and_get_error("SELECT demangle('a')")
assert "Not enough privileges" in instance.query_and_get_error("SELECT demangle('a')", user="robin") assert "Not enough privileges" in instance.query_and_get_error("SELECT demangle('a')", user="robin")
assert "Not enough privileges" in instance.query_and_get_error("SELECT demangle('a')", user="robin", settings={"allow_introspection_functions":1})
assert "Introspection functions are disabled" in instance.query_and_get_error("GRANT demangle ON *.* TO robin")
assert "Not enough privileges" in instance.query_and_get_error("GRANT demangle ON *.* TO robin", user="robin")
assert "Not enough privileges" in instance.query_and_get_error("GRANT demangle ON *.* TO robin", user="robin", settings={"allow_introspection_functions":1})
assert instance.query("SELECT demangle('a')", settings={"allow_introspection_functions":1}) == "signed char\n"
instance.query("GRANT demangle ON *.* TO robin", settings={"allow_introspection_functions":1})
instance.query("GRANT ALL ON *.* TO robin")
assert "Introspection functions are disabled" in instance.query_and_get_error("SELECT demangle('a')", user="robin") assert "Introspection functions are disabled" in instance.query_and_get_error("SELECT demangle('a')", user="robin")
instance.query("ALTER USER robin SETTINGS allow_introspection_functions=1") instance.query("ALTER USER robin SETTINGS allow_introspection_functions=1")
assert instance.query("SELECT demangle('a')", user="robin") == "signed char\n" assert instance.query("SELECT demangle('a')", user="robin") == "signed char\n"
@ -194,5 +201,5 @@ def test_allow_introspection():
instance.query("DROP SETTINGS PROFILE xyz") instance.query("DROP SETTINGS PROFILE xyz")
assert "Introspection functions are disabled" in instance.query_and_get_error("SELECT demangle('a')", user="robin") assert "Introspection functions are disabled" in instance.query_and_get_error("SELECT demangle('a')", user="robin")
instance.query("REVOKE ALL ON *.* FROM robin") instance.query("REVOKE demangle ON *.* FROM robin", settings={"allow_introspection_functions":1})
assert "Not enough privileges" in instance.query_and_get_error("SELECT demangle('a')", user="robin") assert "Not enough privileges" in instance.query_and_get_error("SELECT demangle('a')", user="robin")