Page MenuHomePhabricator

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

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

Details

Summary

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

unit

Diff Detail

Repository
R1 dagster
Lint
Automatic diff as part of commit; lint not applicable.
Unit
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