Label db sync failures by real cause - #213
Conversation
|
Reviewed head 18568b8 (on main a95344f). Approved. Worker, plex and db suites pass (161/161, 29 files). Two of the three new tests fail against main's repository.ts. The third, the Plex token case, passes on main as well, so it only guards against regressions. I ran 15 error shapes through failSyncRun on main and on this head. These are the changes:
These are unchanged and still authentication: Two things to be aware of, neither blocking:
CI at the time I checked: TypeScript workspace, Compose config and CodeFactor green. The four container builds were still running, so it is not CI-green yet. |
Codecov Report❌ Patch coverage is
📢 Thoughts on this report? Let us know! |
Fixes #212. Mirrors the #211 worker fix in the db classifier: SyntaxError is upstream_response, and 'token' only means authentication when Plex is named. Fail-first tests in sync-failure-label.test.ts.