Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Bug] Test for date_spine can give a false positives #210

Open
3 tasks
dbeatty10 opened this issue May 14, 2024 · 0 comments · May be fixed by #211
Open
3 tasks

[Bug] Test for date_spine can give a false positives #210

dbeatty10 opened this issue May 14, 2024 · 0 comments · May be fixed by #211
Labels
bug Something isn't working

Comments

@dbeatty10
Copy link
Contributor

Housekeeping

  • I am a maintainer of dbt-adapters

Short description

The test case can pass even if the expected and actual data sets are not perfectly equal.

Context

History: this test was migrated effectively as-is from dbt-utils to dbt-core to dbt-adapters.

This issue was discovered while working on dbt-labs/dbt-core#10075.

Acceptance criteria

Tasks

Testing

Manually alter the implementation of date_spine in order to check that the test case fails as expected in the following scenarios.

Tasks

Security

No security concerns.

Docs

No doc updates needed.

Consequences

This change impacts any adapter that imports this test.

We'd hope that no adapters are currently getting a false positive, and this change is a no-op for them.

For any adapters that were getting a false positive, this test would start failing for them, and they need to make updates accordingly.

@dbeatty10 dbeatty10 added the bug Something isn't working label May 14, 2024
@dbeatty10 dbeatty10 linked a pull request May 14, 2024 that will close this issue
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant