Conversation
|
@tynes looks like it's trying to run |
|
Works fine locally but maybe I just did something wrong |
|
:-/ adds a full hour to our integration test time |
Any thoughts on speeding up the integration tests here @snario? I think this is very high priority to get in to prevent regressions. Just looked at the Synthetix CI and looks like it spends ~12 min running the same tests, so I feel like we can get pretty far by just beefing up CI |
Seems like in the very near term the best we can do is make this run once a day and ask developers to try running it once locally if making a substantial change to the codebase. In the medium term we should figure out which tests are included here that we can rip out and put into our own testing infrastructure in a way that tests very specific functionality. There are also some other kinds of CI improvements we should consider focusing some attention on, maybe next week, like more caching and less duplication of work between jobs and test suites. |
Description
Dead simple method of adding the Synthetix L2 test suite into our integration tests. Won't be particularly fast but that's not a huge priority right now. We should be able to speed this up later on.
Additional context
We've been wanting this for a while, would be nice to have this even if it's a bit jank.