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

Feature/add dbt unit tests #349

Draft
wants to merge 46 commits into
base: main
Choose a base branch
from
Draft

Feature/add dbt unit tests #349

wants to merge 46 commits into from

Conversation

davidbooke4
Copy link
Collaborator

Description & motivation

Checklist

  • I have verified that these changes work locally
  • I have updated the README.md (if applicable)
  • I have added tests & descriptions to my models (and macros if applicable)
  • I have run dbt test and python -m pytest . to validate existing tests

adamribaudo-velir and others added 30 commits June 7, 2024 07:18
@adamribaudo-velir
Replaced python unit test with dbt 1.8 unit test
9d8f0c8 
For stg_ga4__page_conversions
@adamribaudo-velir
refactored unit tests for stg_ga4__session_conversions_daily
298f373
@adamribaudo-velir
update test name
906bec2
@adamribaudo-velir
Replaced Python unit test with dbt unit test
e994408
@adamribaudo-velir
variable override working properly
34bbab9
@adamribaudo-velir
using overrides properly
4b66d1f
@adamribaudo-velir
replaced another unit test
79f7e27
@adamribaudo-velir
replaced python unit test
cecf337
@adamribaudo-velir
add unit test for stg_ga4__client_key_first_last_pageviews
63a7d86
@adamribaudo-velir
replace unit test
6e709db
@adamribaudo-velir
unit test for stg_ga4__sessions_traffic_sources_last_non_direct_daily…
9d53c9a 
…. Not working yet
@davidbooke4
Add package-lock.yml to .gitignore
3425fdf
@davidbooke4
Add vars to dbt_project.yml for testing
c3ba7f7
@davidbooke4
Merge branch 'main' into feature/dbt-unit-tests
10456ef
@davidbooke4
Add unit tests to stg_ga4__events.yml for the url_parsing macros
a1f10df
@davidbooke4
Add conditions for cases when event_source is null for session parame…
5972788 
…ters
@davidbooke4
Add unit test to stg_ga4__sessions_traffic_sources_daily for testing …
20598fb 
…the default channel grouping macro
@davidbooke4
Add unit test to stg_ga4__user_id_mapping to test the latest mapping …
282eeee 
…logic
@davidbooke4
Add descriptions for unit tests that were missing them
c321197
@davidbooke4
Remove python unit tests that have been migrated to dbt unit tests
8a1796e
@davidbooke4
Add unit test to stg_ga4__events for testing transformations in stg_g…
c0aba5f 
…a4__events
@davidbooke4
Remove todo and example stg_ga4__events unit test files
922ba07
@davidbooke4
Add sessions_traffic_sources_last_non_direct_daily python unit test back
3a4f677
@davidbooke4
Comment out unit tests for disabled models
c870130
@davidbooke4
Remove edits from dbt_project.yml
7386371
@davidbooke4
Comment out unit test for sessions_traffic_sources_last_non_direct_da…
76f2c7f 
…ily until unit test var bug fix
@davidbooke4
Update unit test section in README
697bafd
@davidbooke4
Simplify event_params construction in test_base_to_stg_ga4__events in…
616da99 
…put and expect
@davidbooke4
Update yml files to use consistent new line convention
653e1ae
@adamribaudo-velir
update PR template
50ff2e8
@davidbooke4
Update default channel grouping test to use seed instead of fixture a…
68f9f87 
…nd delete fixture csv
@davidbooke4
Comment out unit tests for disabled models
a3d9c1e
@davidbooke4
Un-comment unit tests
1dd415e
@davidbooke4
Add profiles.yml for Github Actions to execute dbt commands and add .…
4ef2503 
…user.yml to gitignore
@davidbooke4
Add profile and variables to dbt_project.yml so Github Action can run…
83bd23b 
… unit tests
@davidbooke4
Add dbt unit tests job to github CI workflow
6f4335e
@davidbooke4
Remove empty step
947868d
@davidbooke4
Add repo to checkout step so PR code is checked out to test adding ne…
8c879f7 
…w dbt unit test job
@davidbooke4
Change workflow on behavior for testing changes
ca215a1
@davidbooke4
Remove source project and property ID variables from dbt_project.yml
984c311
@davidbooke4
Use environment variables for project, dataset, property ID when not …
6493051 
…set as project variables
@davidbooke4
Add conditional logic to allow for use of --empty flag
637e166
@davidbooke4
Add step to materialize tables/views needed to run dbt unit tests
845e1df
@davidbooke4
Add environment variables related to project variables so models are …
48f2a14 
…enabled for CI
@davidbooke4
Update models to look for environment variables before project variables
561c3e0
@davidbooke4
Add environment variables to CI workflow
c32d699
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@davidbooke4 @adamribaudo-velir