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

[CT-2413] [Bug] Non-functional testing for string_type and numeric_type static class methods #81

Closed
2 of 6 tasks
dbeatty10 opened this issue Apr 13, 2023 · 5 comments
Closed
2 of 6 tasks
Labels
Stale Mark an issue or PR as stale, to be closed type:bug Something isn't working as documented

Comments

@dbeatty10
Copy link
Contributor

dbeatty10 commented Apr 13, 2023

Is this a new bug in dbt-core?

  • I believe this is a new bug in dbt-core
  • I have searched the existing issues, and I could not find an existing issue for this bug

Current Behavior

Based on the documentation for the string_type static class method here, we'd expect the following query to work across adapters:

select cast(10 as {{ api.Column.string_type(4000) }})

But a package maintainer reported here that it doesn't work for dbt-bigquery, dbt-spark, or dbt-databricks.

Expected Behavior

See above.

Steps To Reproduce

See here for steps to reproduce.

Which database adapter are you using with dbt?

bigquery, spark

Suspected root cause

The primary testing for these two methods appears tautological to me (rather than actually testing platform-specific data types).

Acceptance criteria

  • add functional tests for the string_type and numeric_type adapter class methods that adapter maintainers can inherit
  • inherit those new adapter tests for all adapters maintained by dbt Labs
  • add instructions for maintainer adapters in the next version migration guide
  • either update or remove these tests
@dbeatty10 dbeatty10 added type:bug Something isn't working as documented triage:product In Product's queue labels Apr 13, 2023
@github-actions github-actions bot changed the title [Bug] Non-functional testing for string_type and numeric_type static class methods [CT-2413] [Bug] Non-functional testing for string_type and numeric_type static class methods Apr 13, 2023
@dbeatty10 dbeatty10 removed the triage:product In Product's queue label Apr 13, 2023
@github-actions
Copy link
Contributor

This issue has been marked as Stale because it has been open for 180 days with no activity. If you would like the issue to remain open, please comment on the issue or else it will be closed in 7 days.

@github-actions github-actions bot added the Stale Mark an issue or PR as stale, to be closed label Oct 12, 2023
@rlh1994
Copy link

rlh1994 commented Oct 12, 2023

Stale label - a haiku
GitHub actions bot
Please do not close this issue
It still needs a fix

@dbeatty10 dbeatty10 removed the Stale Mark an issue or PR as stale, to be closed label Oct 12, 2023
@dbeatty10
Copy link
Contributor Author

Your words have been heard,
Stale label swept away, friend,
Gratitude expressed

@mikealfare mikealfare transferred this issue from dbt-labs/dbt-core Feb 13, 2024
Copy link
Contributor

This issue has been marked as Stale because it has been open for 180 days with no activity. If you would like the issue to remain open, please comment on the issue or else it will be closed in 7 days.

@github-actions github-actions bot added the Stale Mark an issue or PR as stale, to be closed label Aug 12, 2024
Copy link
Contributor

Although we are closing this issue as stale, it's not gone forever. Issues can be reopened if there is renewed community interest. Just add a comment to notify the maintainers.

@github-actions github-actions bot closed this as not planned Won't fix, can't repro, duplicate, stale Aug 20, 2024
mikealfare pushed a commit that referenced this issue Jan 23, 2025
Co-authored-by: Colin Rogers <111200756+colin-rogers-dbt@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Stale Mark an issue or PR as stale, to be closed type:bug Something isn't working as documented
Projects
None yet
Development

No branches or pull requests

2 participants