Page MenuHomePhabricator

[dagster-airflow 2/2] Remove class-based airflow test fixtures

Authored by nate on Jan 3 2020, 12:06 AM.



This replaces the class-based test fixtures with function-based ones to make the tests cleaner / more straightforward. Will have another follow-up to set up the airflow tests like the kubernetes tests, doing test setup in pytest instead of shell scripts

Depends on D1767

Test Plan


Diff Detail

R1 dagster
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

nate created this revision.Jan 3 2020, 12:06 AM
Harbormaster failed remote builds in B6759: Diff 8329!
nate updated this revision to Diff 8334.Jan 3 2020, 12:25 AM

update docstrings

nate edited the summary of this revision. (Show Details)Jan 3 2020, 12:30 AM
nate added reviewers: alangenfeld, max.
This revision is now accepted and ready to land.Jan 3 2020, 3:53 PM