Skip to content

Commit

Permalink
change expected_stat types to match agate_helper change in dbt-core (#…
Browse files Browse the repository at this point in the history
…598) (#599)

(cherry picked from commit 4c5e49b)

Co-authored-by: Matthew McKnight <[email protected]>
  • Loading branch information
github-actions[bot] and McKnight-42 authored Sep 11, 2023
1 parent 859fe61 commit 8d31620
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 3 deletions.
6 changes: 6 additions & 0 deletions .changes/unreleased/Fixes-20230908-113019.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,6 @@
kind: Fixes
body: changes expected value types to AnyInteger to take into account changes in core
time: 2023-09-08T11:30:19.77143-05:00
custom:
Author: McKnight-42
Issue: "597"
6 changes: 3 additions & 3 deletions tests/functional/adapter/expected_stats.py
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
from dbt.tests.util import AnyStringWith, AnyFloat, AnyString
from dbt.tests.util import AnyStringWith, AnyInteger, AnyString, AnyFloat


def redshift_stats():
Expand Down Expand Up @@ -27,14 +27,14 @@ def redshift_stats():
"max_varchar": {
"id": "max_varchar",
"label": "Max Varchar",
"value": AnyFloat(),
"value": AnyInteger(),
"description": "Size of the largest column that uses a VARCHAR data type.",
"include": True,
},
"size": {
"id": "size",
"label": "Approximate Size",
"value": AnyFloat(),
"value": AnyInteger(),
"description": "Approximate size of the table, calculated from a count of 1MB blocks",
"include": True,
},
Expand Down

0 comments on commit 8d31620

Please sign in to comment.