Gives api_tests/, cli_tests/, core_tests/ their own tox envs, putting the rest under the -general suffix.
Adds these suffices to the corresponding Buildkite pipeline step.
If there's a nicer way to bundle these ignores together, I'd love to make use of it. But I don't see anything reading through the tox docs that seems immediately useful...
hmm, can you just move all of the other dagster tests under dagster_tests/general_tests? then you shouldn't need the ignores, right?
+1 to this. The only thing we have to be careful about is thinking about tests people add outside one of these folders. They might not realize that they're not being run by BK.
Looks great! I’d recommend creating a new diff that just moves the tests in to /general_tests, and rebasing this off of that. There have been quite a few tests modified in dagster_tests, and the merge conflict would be unnecessarily complicated to work through.
Also make sure none of the .db, .db-shm, and .db-wal files are committed, there shouldn’t be any changes in them.