-
Notifications
You must be signed in to change notification settings - Fork 1.6k
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
- Loading branch information
1 parent
407f6ca
commit c089d6f
Showing
1 changed file
with
62 additions
and
0 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,62 @@ | ||
|
||
from dataclasses import dataclass | ||
import pytest | ||
|
||
|
||
from dbt.artifacts.schemas.base import ( | ||
ArtifactMixin, | ||
BaseArtifactMetadata, | ||
schema_version, | ||
) | ||
|
||
@dataclass | ||
@schema_version("manifest", 12) | ||
class Artifact(ArtifactMixin): | ||
a: int | ||
|
||
|
||
@dataclass | ||
@schema_version("manifest", 12) | ||
class ArtifactMinorSchemaChange(ArtifactMixin): | ||
a: int | ||
b: int = 2 | ||
|
||
|
||
class TestMinorSchemaChange: | ||
@pytest.fixture | ||
def artifact(self): | ||
return Artifact(a = 1, metadata=BaseArtifactMetadata(dbt_schema_version="1.9.0")) | ||
|
||
@pytest.fixture | ||
def artifact_minor_schema_change(self): | ||
return ArtifactMinorSchemaChange(a=1, metadata=BaseArtifactMetadata(dbt_schema_version="1.9.0")) | ||
|
||
def test_serializing_new_default_field_is_backward_compatabile( | ||
self, artifact_minor_schema_change | ||
): | ||
# old code (using old class) can create an instance of itself given new data (new class) | ||
artifact = Artifact.from_dict(artifact_minor_schema_change.to_dict()) | ||
|
||
assert artifact.a == 1 | ||
# Additional fields are ignored | ||
assert not hasattr(artifact, "b") | ||
|
||
def test_serializing_new_default_field_is_forward_compatible(self, artifact): | ||
# new code (using new class) can create an instance of itself given old data (old class) | ||
artifact = ArtifactMinorSchemaChange.from_dict(artifact.to_dict()) | ||
|
||
assert artifact.a == 1 | ||
assert artifact.b == 2 | ||
|
||
def test_serializing_removed_default_field_is_backward_compatabile(self, artifact): | ||
# old code (using old class with default field) can create an instance of itself given new data (class w/o default field) | ||
old_artifact = ArtifactMinorSchemaChange.from_dict(artifact.to_dict()) | ||
|
||
# set to the default value when not provided in data | ||
assert old_artifact.b == 2 | ||
|
||
def test_serializing_removed_default_field_is_forward_compatible( | ||
self, artifact_minor_schema_change | ||
): | ||
# new code (using class without default field) can create an instance of itself given old data (class with old field) | ||
Artifact.from_dict(artifact_minor_schema_change.to_dict()) |