Page MenuHomeElementl
Differential D8833

make sure pipeline_failure_sensor is under test
ClosedPublic
Actions

Authored by • prha on Jul 12 2021, 8:35 PM.
Tags
None
Referenced Files
F2439266: D8833.id41533.diff
Tue, Aug 16, 7:30 PM
F2439265: D8833.id.diff
Tue, Aug 16, 7:30 PM
Unknown Object (File)
Mon, Aug 15, 8:07 AM
Unknown Object (File)
Fri, Aug 12, 12:05 PM
Unknown Object (File)
Sun, Aug 7, 4:53 AM
Unknown Object (File)
Sun, Aug 7, 3:01 AM
Unknown Object (File)
Sun, Aug 7, 2:52 AM
Unknown Object (File)
Thu, Aug 4, 6:05 PM
View All 29 Files
Subscribers
None

Details

Reviewers
• yuhan
alangenfeld
Commits
R1:d9f5cafa1f0c: make sure pipeline_failure_sensor is under test
Summary

Would have caught the get_event_record callsite error sooner. :(

Test Plan

ran unittest

Diff Detail

Repository
R1 dagster
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Revision Contents
Changeset List

PathSize
python_modules/
dagster/
dagster/
core/
definitions/
19 lines
7 lines
dagster_tests/
daemon_tests/
92 lines

Diff 41533

View Options

python_modules/dagster/dagster/core/definitions/pipeline_sensor.py

Loading...
View Options

python_modules/dagster/dagster/core/definitions/sensor.py

Loading...
View Options

python_modules/dagster/dagster_tests/daemon_tests/test_sensor_run.py

Loading...