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

Update naming conventions for automated tests #3648

Closed
kwigley opened this issue Jul 29, 2021 · 0 comments · Fixed by #4509
Closed

Update naming conventions for automated tests #3648

kwigley opened this issue Jul 29, 2021 · 0 comments · Fixed by #4509
Assignees
@emmyoop
Labels
tech_debt Behind-the-scenes changes, with little direct impact on end-user functionality

Comments

@kwigley
Copy link
Contributor

kwigley commented Jul 29, 2021

Some testing modules are similarly or inconsistently named. We use both *_test and *_tests when defining test modules. This makes test discoverability difficult. We should update naming where possible to align with a single convention.

Implementation suggestions:

  • find and update testing modules to follow a single conventions
  • identify any confusingly named tests and update for discoverability
@kwigley kwigley added the tech_debt Behind-the-scenes changes, with little direct impact on end-user functionality label Jul 29, 2021
@emmyoop emmyoop self-assigned this Dec 15, 2021
@emmyoop emmyoop mentioned this issue Jan 3, 2022
Merged
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@emmyoop emmyoop

Labels
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.

test name standardization dbt-labs/dbt-core
2 participants
@emmyoop @kwigley