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.

Sign up for GitHub

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

Jump to bottom

[CT-2877] Convert test project for invalid ephemeral sql into dbt Core test #8225

Closed
gshank opened this issue Jul 27, 2023 · 0 comments · Fixed by #8376
Closed

[CT-2877] Convert test project for invalid ephemeral sql into dbt Core test #8225

gshank opened this issue Jul 27, 2023 · 0 comments · Fixed by #8376
Assignees
@MichelleArk
Labels
backport 1.6.latest dbt tests Issues related to built-in dbt testing functionality tech_debt Behind-the-scenes changes, with little direct impact on end-user functionality

Comments

@gshank
Copy link
Contributor

gshank commented Jul 27, 2023

The project in https://github.com/jeremyyeo/ephemeral-invalid-sql-repro was valuable for observing and fixing a bug in ephemeral cte injection. We should convert this project into a test case.
@github-actions github-actions bot changed the title Convert test project for invalid ephemeral sql into dbt Core test [CT-2877] Convert test project for invalid ephemeral sql into dbt Core test Jul 27, 2023
@gshank gshank added the dbt tests Issues related to built-in dbt testing functionality label Jul 27, 2023
@emmyoop emmyoop added the tech_debt Behind-the-scenes changes, with little direct impact on end-user functionality label Jul 27, 2023
@MichelleArk MichelleArk mentioned this issue Aug 12, 2023
Merged
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@MichelleArk MichelleArk

Labels
backport 1.6.latest dbt tests Issues related to built-in dbt testing functionality tech_debt Behind-the-scenes changes, with little direct impact on end-user functionality
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

add TestLargeEphemeralCompilation dbt-labs/dbt-core
3 participants
@gshank @emmyoop @MichelleArk