Page MenuHomePhabricator

(dagster-cli-api-run-launcher-5.5) Revamp dagster-graphql scheduler test infrastructure
ClosedPublic

Authored by schrockn on May 22 2020, 2:06 AM.

Details

Summary

In preparation for having all graphql tests launch run out-of-process,
we need to clean up some more graphql test infra, because we need better control
over the graphql context. These tests require the filesystem scheduler so we
just configure that on the test graphql context by default

Depends on D3038

Test Plan

BK

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

schrockn retitled this revision from (dagster-cli-api-run-launcher-8) Revamp dagster-graphql scheduler test infrastructure to (dagster-cli-api-run-launcher-5.5) Revamp dagster-graphql scheduler test infrastructure.
schrockn edited the summary of this revision. (Show Details)

up

Looks great, thanks for this

This revision is now accepted and ready to land.May 22 2020, 5:13 PM