Page MenuHomeElementl

Move not_graphql_context_test_suite to root tox file

Authored by sashank on Jan 13 2021, 5:03 PM.



Previously, all the tests that were not part of the dagster-graphql context test matrix were included in the postgres_tox.ini file and run as a part of the dagster-graphql-postgres BK step. This was due to one test: test_readonly_variants, which had a dagster_postgres dependency.

This diff updates the tox files to run all tests not part of the test matrix as a part of the dagster-graphql step, and adds a variant mark that only tests the test_readonly_variants in the dagster-graphql-postgres step.

Test Plan


Diff Detail

R1 dagster
Lint Not Applicable
Tests Not Applicable

Event Timeline

sashank created this revision.

forgot a file

sashank edited the summary of this revision. (Show Details)


Harbormaster returned this revision to the author for changes because remote builds failed.Jan 13 2021, 5:35 PM
Harbormaster failed remote builds in B24251: Diff 29503!
Harbormaster failed remote builds in B24253: Diff 29505!
alangenfeld added inline comments.

i dont really know what variant is supposed to mean on its own, maybe a slightly more descriptive name ?

This revision is now accepted and ready to land.Jan 13 2021, 7:21 PM

update name of "variants" mark