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

Unit testing docs #6344

Open
1 task done
dbeatty10 opened this issue Oct 23, 2024 · 1 comment · May be fixed by #6345
Open
1 task done

Unit testing docs #6344

dbeatty10 opened this issue Oct 23, 2024 · 1 comment · May be fixed by #6345
Labels
content Improvements or additions to content improvement Use this when an area of the docs needs improvement as it's currently unclear

Comments

@dbeatty10
Copy link
Contributor

Contributions

  • I have read the contribution docs, and understand what's expected of me.

Link to the page on docs.getdbt.com requiring updates

https://docs.getdbt.com/docs/build/unit-tests

What part(s) of the page would you like to see updated?

We got some feedback that suggested the following updates to our documentation:

  • Clarify fixture requirements for all dependencies (to avoid null values in output)
  • Emphasize satisfying all model logic conditions (i.e. join / where clauses to avoid nulls)
  • Under-the-hood code example of how unit tests compile

Additional information

No response

@dbeatty10 dbeatty10 added content Improvements or additions to content improvement Use this when an area of the docs needs improvement as it's currently unclear labels Oct 23, 2024
@dbeatty10 dbeatty10 linked a pull request Oct 23, 2024 that will close this issue
1 task
@runleonarun
Copy link
Collaborator

Thanks Doug! Let us know when you want a review on that PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
content Improvements or additions to content improvement Use this when an area of the docs needs improvement as it's currently unclear
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants