liquibase / liquibase-databricks

Apache License 2.0
26 stars 7 forks source link

feat(DAT-18853): Add custom diff change generator for Databricks #203

Closed filipelautert closed 2 weeks ago

filipelautert commented 4 weeks ago
filipelautert commented 4 weeks ago

@KushnirykOleh any idea on how to add tests here?

KushnirykOleh commented 3 weeks ago

diff-changelog test from test-harness doesn't work properly in extensions. To check parsing I would only add unit test with Strings that looks like these https://github.com/liquibase/liquibase-databricks/blob/2cd2f400ba4a7c67c761b4cbdd7141a867541aef/src/test/resources/liquibase/harness/change/expectedSnapshot/databricks/createTable.json#L7-L16 - actual tblProperties returned from databricks

filipelautert commented 3 weeks ago

To check parsing I would only add unit test with Strings that looks like these

Thanks!

sonarcloud[bot] commented 2 weeks ago

Quality Gate Passed Quality Gate passed

Issues
0 New issues
2 Accepted issues

Measures
0 Security Hotspots
45.7% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud