Merge pull request #2141 from ClickHouse/returngeneratedvalues-should-noop

Setting a no-op and disabling related tests

Author: Paultagoras

clickhouse-r2dbc/src/main/java/com/clickhouse/r2dbc/ClickHouseStatement.java

@@ -156,6 +156,6 @@ public Flux<? extends Result> execute() {
 
     @Override
     public Statement returnGeneratedValues(String... columns) {
-        throw new UnsupportedOperationException(GENERATED_VALUES_CAN_NOT_BE_RETURNED_FROM_CLICKHOUSE_DATABASE);
+        return this;
     }
 }

clickhouse-r2dbc/src/test/java/com/clickhouse/r2dbc/spi/test/R2DBCTestKitImplTest.java

@@ -207,40 +207,16 @@ public void duplicateColumnNames() {
 
     @Override
     @Test
+    @Disabled
     public void returnGeneratedValues() {
-        getJdbcOperations().execute(expand(TestStatement.DROP_TABLE));
-        getJdbcOperations().execute(getCreateTableWithAutogeneratedKey());
-        Flux.usingWhen(getConnectionFactory().create(),
-                connection -> {
-                    Statement statement = connection.createStatement(getInsertIntoWithAutogeneratedKey());
-
-                    statement.returnGeneratedValues();
-
-                    return Flux.from(statement
-                            .execute())
-                            .flatMap(it -> it.map((row, rowMetadata) -> row.get(0)));
-                },
-                Connection::close)
-                .as(StepVerifier::create)
-                .expectErrorMatches(UnsupportedOperationException.class::isInstance)
-                .verify();
+        // not supported
     }
 
     @Override
     @Test
+    @Disabled
     public void returnGeneratedValuesFails() {
-
-        Flux.usingWhen(getConnectionFactory().create(),
-                connection -> {
-                    Statement statement = connection.createStatement(expand(TestStatement.INSERT_VALUE100));
-
-                    assertThrows(UnsupportedOperationException.class,
-                            () -> statement.returnGeneratedValues((String[]) null));
-                    return Mono.empty();
-                },
-                Connection::close)
-                .as(StepVerifier::create)
-                .verifyComplete();
+        // not supported
     }
 
     @Override