Conversation
7d1b138 to
1f09287
Compare
Added a logic fix: previous docker deployment integration test used essentially vanilla redis.conf configuration; this is now fixed so that we will load the LRR redis config into the redis container instead.
Added client-side optimizations. Mainly moved everything (including all of my other Python projects) over to uv; it (by it I mean my build times) is generally 20-30% faster. It's been pretty good. Another client-side optimization is the request-handling, previously validation was unnecessarily done in Python, now Pydantic (with rust core) handles it, which also made request handling faster (though it's not a big improvement considering it's not a bottleneck in testing). And some windows test-side improvements, such as not spawning console windows, which (heuristically) have alleviated the test pressure problems? Unrelated: tests can be run with a different dockerfile now, if you ever need that for some fork-side reason. |
31d5186 to
e07cd4b
Compare
Thanks to some clankRCA™, instrumentation-spamming, hypothesis testing and process of elimination, the cause of Windows test-side failures and slowness were (practically) identified; see psilabs-dev/aio-lanraragi#173 and psilabs-dev/aio-lanraragi#172. But anyways, the tests (ignoring Shinobu) should be more stable and faster now. Also added/extended test coverage:
Added various other tests which don't run by default (OpenAPI feature + bypass, preloading check, etc.) |
No new tests planned atm, instead I've stabilized the existing test cases and make conditions stricter.
The following updates apply to all Playwright UI tests:
Log health assertions are occasionally capturing Shinobu error logs ("giving up adding it to the filesystem"); this is an existing race condition which can be ignored, or we can suppress it in the future test-side or LRR-side.