Test serializability of `SerializableDataclass` subclasses

dbt-labs / dbt-semantic-interfaces

The shared semantic layer definitions that dbt-core and MetricFlow use.

Apache License 2.0

66 stars 13 forks source link

Test serializability of `SerializableDataclass` subclasses #315

Closed plypaul closed 1 month ago

plypaul commented 1 month ago

Description

Since there are no tests that verify the serializability of SerializableDataclass subclasses, this PR adds a mechanism for enumerating them and a test for them.

Checklist

[x] I have read the contributing guide and understand what's expected of me
[x] I have signed the CLA
[x] This PR includes tests, or tests are not required/relevant for this PR
[x] I have run changie new to create a changelog entry

github-actions[bot] commented 1 month ago

Thank you for your pull request! We could not find a changelog entry for this change. For details on how to document a change, see the contributing guide.

plypaul commented 1 month ago

Curious if you will add something similar to MF / MFS?

Yes, I'm planning to add similar tests for those projects.