Page MenuHomeElementl

Make run storage test a bit less strict
AbandonedPublic

Authored by dgibson on Jul 28 2021, 2:49 PM.

Details

Summary

Test for a specific error message to encompass different types of run storage

Test Plan

BK

Diff Detail

Repository
R1 dagster
Branch
graphqlstorage (branched from master)
Lint
Lint Passed
Unit
No Test Coverage

Event Timeline

dgibson published this revision for review.Jul 28 2021, 2:50 PM
jordansanders added inline comments.
python_modules/dagster/dagster_tests/core_tests/storage_tests/utils/run_storage.py
900

Why are some run storage implementations raising an exception that doesn't inherit from DagsterRunNotFoundError? This feels like the test accurately describes behavior and we have implementations that should change.

This revision now requires changes to proceed.Jul 28 2021, 3:06 PM