Added readonly profile tests

Author: chernser

jdbc-v2/src/test/java/com/clickhouse/jdbc/JdbcDataTypeTests.java

@@ -2863,12 +2863,23 @@ public void testJSONRead(String json, Object expected) throws Exception {
 
                 assertTrue(rs.next());
                 Object jsonObj = rs.getObject(1);
+                assertTrue(jsonObj instanceof Map, "JSON should be read as a Map");
+                Map<String, Object> jsonMap = (Map<String, Object>) jsonObj;
+                
                 if (expected == null) {
                     expected = jsonToClientMap(json);
                 }
-                assertEquals(jsonObj, expected);
+                Map<String, Object> expectedMap = (Map<String, Object>) expected;
+                
+                assertEquals(jsonMap, expectedMap);
+                for (Map.Entry<String, Object> entry : expectedMap.entrySet()) {
+                    assertTrue(jsonMap.containsKey(entry.getKey()), "Map should contain key: " + entry.getKey());
+                    assertEquals(jsonMap.get(entry.getKey()), entry.getValue(), "Values should match for key: " + entry.getKey());
+                }
+
                 assertTrue(rs.next());
                 Object emptyJsonObj = rs.getObject(1);
+                assertTrue(emptyJsonObj instanceof Map, "Empty JSON should be returned as Map");
                 assertEquals(emptyJsonObj, EMPTY_JSON);
                 assertFalse(rs.next());
             }

jdbc-v2/src/test/java/com/clickhouse/jdbc/ReadonlyProfileTest.java

@@ -0,0 +1,80 @@
+package com.clickhouse.jdbc;
+
+import com.clickhouse.client.api.ClientConfigProperties;
+import com.clickhouse.client.api.internal.ServerSettings;
+import org.testng.Assert;
+import org.testng.annotations.AfterClass;
+import org.testng.annotations.BeforeClass;
+import org.testng.annotations.Test;
+
+import java.sql.Connection;
+import java.sql.ResultSet;
+import java.sql.SQLException;
+import java.sql.Statement;
+import java.util.Properties;
+
+public class ReadonlyProfileTest extends JdbcIntegrationTest {
+
+    @BeforeClass(groups = { "integration" })
+    public void setup() throws SQLException {
+        com.clickhouse.client.ClickHouseServerForTest.beforeSuite();
+        try (Connection conn = getJdbcConnection();
+             Statement stmt = conn.createStatement()) {
+            stmt.execute("CREATE SETTINGS PROFILE IF NOT EXISTS profile_readonly_1 SETTINGS readonly=1");
+            stmt.execute("CREATE SETTINGS PROFILE IF NOT EXISTS profile_readonly_2 SETTINGS readonly=2");
+            stmt.execute("CREATE USER IF NOT EXISTS user_readonly_1 IDENTIFIED WITH no_password SETTINGS PROFILE profile_readonly_1");
+            stmt.execute("CREATE USER IF NOT EXISTS user_readonly_2 IDENTIFIED WITH no_password SETTINGS PROFILE profile_readonly_2");
+            stmt.execute("GRANT SELECT ON *.* TO user_readonly_1");
+            stmt.execute("GRANT SELECT ON *.* TO user_readonly_2");
+        }
+    }
+
+    @AfterClass(groups = { "integration" })
+    public void teardown() throws SQLException {
+        try (Connection conn = getJdbcConnection();
+             Statement stmt = conn.createStatement()) {
+            stmt.execute("DROP USER IF EXISTS user_readonly_1");
+            stmt.execute("DROP USER IF EXISTS user_readonly_2");
+            stmt.execute("DROP SETTINGS PROFILE IF EXISTS profile_readonly_1");
+            stmt.execute("DROP SETTINGS PROFILE IF EXISTS profile_readonly_2");
+        }
+    }
+
+    @Test(groups = { "integration" })
+    public void testReadonly1CannotChangeSettings() throws Exception {
+        Properties properties = new Properties();
+        properties.setProperty("user", "user_readonly_1");
+        properties.setProperty("password", "");
+        properties.put(ClientConfigProperties.serverSetting(ServerSettings.OUTPUT_FORMAT_BINARY_WRITE_JSON_AS_STRING), "1");
+        
+        try (Connection conn = getJdbcConnection(properties)) {
+            try (Statement stmt = conn.createStatement();
+                 ResultSet rs = stmt.executeQuery("SELECT 1")) {
+                Assert.fail("Should have thrown an exception because readonly=1 prevents changing settings");
+            }
+        } catch (SQLException e) {
+            Assert.assertTrue(e.getMessage().contains("Cannot modify"), "Exception message should indicate setting cannot be modified: " + e.getMessage());
+        }
+    }
+
+    @Test(groups = { "integration" })
+    public void testReadonly2CanChangeSettings() throws Exception {
+        if (isVersionMatch("(,24.8]")) {
+            return; // JSON was introduced in 24.10
+        }
+
+        Properties properties = new Properties();
+        properties.setProperty("user", "user_readonly_2");
+        properties.setProperty("password", "");
+        properties.put(ClientConfigProperties.serverSetting(ServerSettings.OUTPUT_FORMAT_BINARY_WRITE_JSON_AS_STRING), "1");
+        properties.put(ClientConfigProperties.serverSetting("allow_experimental_json_type"), "1");
+        
+        try (Connection conn = getJdbcConnection(properties)) {
+            try (Statement stmt = conn.createStatement();
+                 ResultSet rs = stmt.executeQuery("SELECT '{\"key\":\"value\"}'::JSON")) {
+                Assert.assertTrue(rs.next());
+                Assert.assertEquals(rs.getString(1), "{\"key\":\"value\"}");
+            }
+        }
+    }
+}