mirror of
https://github.com/BerriAI/litellm.git
synced 2026-09-08 22:21:35 +00:00
a5f9410681
key_edit_view opened a select and then clicked the option it found by title text or by raw text. Both queries match the moment the option enters the DOM, which is one render before the popup finishes entering. Until then the positioner still carries an inline pointer-events: none, and user-event refuses to click through it. That is a race, and a fast machine loses it. Five of the file's 84 tests failed on every local run while CI stayed green, which is the worst shape for a test to have: it is only ever red on the machine of whoever is trying to change the code. tests/test-utils.tsx already ships chooseSelectOption for exactly this. It finds the option by role and waits for the positioner to release pointer events before clicking. The five call sites now use it, and the helper takes the direct user-event API as well as a setup() instance so callers do not have to restructure to use it. Five consecutive full-file runs pass where every previous run failed. Also finishes this file's screen queries, which brings prefer-screen-queries to its target of 18.
11 lines
548 B
JSON
11 lines
548 B
JSON
{
|
|
"@typescript-eslint/no-explicit-any": { "max": 2040, "target": 1500 },
|
|
"no-console": { "max": 12, "target": 0 },
|
|
"complexity": { "max": 140, "target": 80 },
|
|
"max-depth": { "max": 70, "target": 30 },
|
|
"local/no-large-inline-object-arg": { "max": 559, "target": 300 },
|
|
"local/no-long-condition-chain": { "max": 265, "target": 120 },
|
|
"testing-library/no-container": { "max": 133, "target": 50 },
|
|
"testing-library/no-node-access": { "max": 716, "target": 500 },
|
|
"testing-library/prefer-screen-queries": { "max": 18, "target": 18 }
|
|
}
|