Page MenuHomePhabricator

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

Authored by schrockn on Fri, May 22, 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 created this revision.Fri, May 22, 2:06 AM
schrockn updated this revision to Diff 14509.Fri, May 22, 2:13 AM
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

schrockn requested review of this revision.Fri, May 22, 2:32 AM
sashank accepted this revision.Fri, May 22, 5:13 PM

Looks great, thanks for this

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