Open elyobo opened 3 weeks ago
Working on adding unit tests here but it will take a while to migrate all of them
https://github.com/Velir/dbt-ga4/tree/feature/dbt-unit-tests
I'm not suggesting a change of the test suites themselves (although adding unit tests sounds great!), just that the name of the existing tests
config item has moved to data_tests
and it's generating a warning because of that; works fine for now but will presumably go away in a future release.
Fair. I was going to wrap it all into 1 PR but we could update to data_tests
in its own change.
Probably no rush, I don't know when they'll remove them entirely but presumably a 1.9 release at least.
See https://docs.getdbt.com/docs/build/data-tests#new-data_tests-syntax
tests
should be renamed todata_tests
for future compatibility, although this is a BC break and so it's not clear at which point to make this change to me.