clickhouse-java icon indicating copy to clipboard operation
clickhouse-java copied to clipboard

Adding cloud tests

Open mzitnik opened this issue 11 months ago • 3 comments

Summary

Adding cloud tests coverage

Checklist

Delete items not relevant to your PR:

  • [ ] Unit and integration tests covering the common scenarios were added
  • [ ] A human-readable description of the changes was provided to include in CHANGELOG
  • [ ] For significant changes, documentation in https://github.com/ClickHouse/clickhouse-docs was updated with further explanations or tutorials

mzitnik avatar Mar 24 '24 09:03 mzitnik

CLA assistant check
All committers have signed the CLA.

CLAassistant avatar Mar 24 '24 09:03 CLAassistant

I think, this change would be good to adjust for running tests against any host not only cloud. Current version assumes that cloud using https protocol but test host may be another cases. At least correcting this thing would be a great addition. As for adding to CI - I think we create a separate workflow. Then we may merge it back. We may not require to run tests against cloud, I think.

chernser avatar Jun 17 '24 22:06 chernser

Quality Gate Failed Quality Gate failed

Failed conditions
19.0% Coverage on New Code (required ≥ 80%)

See analysis details on SonarCloud

sonarqubecloud[bot] avatar Jul 31 '24 15:07 sonarqubecloud[bot]