Convert list tests to pytest #5178

stu-k · 2022-04-27T19:06:24Z

resolves #5049

Description

Converted dbt list tests to pytest

Checklist

I have read the contributing guide and understand what's expected of me
I have signed the CLA
I have run this code in development and it appears to resolve the stated issue
This PR includes tests, or tests are not required/relevant for this PR
I have opened an issue to add/update docs, or docs changes are not required/relevant for this PR
I have run changie new to create a changelog entry

ChenyuLInx

LGTM! Maybe just one minor adjustment

tests/functional/list/test_list.py

Convert list tests to pytest

e6fd100

stu-k requested a review from a team as a code owner April 27, 2022 19:06

stu-k requested review from ChenyuLInx and iknox-fa April 27, 2022 19:06

cla-bot bot added the cla:yes label Apr 27, 2022

stu-k added 2 commits April 27, 2022 14:06

Add changie entry

5b059bc

Remove unused unittest tests

6fd3e23

ChenyuLInx approved these changes Apr 28, 2022

View reviewed changes

tests/functional/list/test_list.py Show resolved Hide resolved

tests/functional/list/test_list.py Show resolved Hide resolved

Remove unused fixture

4607b82

stu-k merged commit a2b3602 into main Apr 28, 2022

stu-k deleted the stu/CT-484-convert-ls-tests branch April 28, 2022 20:00

agoblet pushed a commit to BigDataRepublic/dbt-core that referenced this pull request May 20, 2022

Convert list tests to pytest (dbt-labs#5178)

5783026

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Convert list tests to pytest #5178

Convert list tests to pytest #5178

stu-k commented Apr 27, 2022 •

edited

Loading

ChenyuLInx left a comment

Convert list tests to pytest #5178

Convert list tests to pytest #5178

Conversation

stu-k commented Apr 27, 2022 • edited Loading

Description

Checklist

ChenyuLInx left a comment

Choose a reason for hiding this comment

stu-k commented Apr 27, 2022 •

edited

Loading