DrRataplan / xq-lsp / 29051709947

93%

push

github

fix: don't blanket-skip XQUpdate-tagged QT4 tests

The XQUpdate feature exclusion added in the previous commit was too coarse:
QT4 tags a whole test-set file with feature="XQUpdate" even when most of
its test-cases just exercise the plain `copy $x := ... modify ... return`
transform expression, which is core XQuery 3.0/3.1 grammar we already
support (variable-diagnostics.ts handles XQUF_TransformExpr directly) and
is unrelated to the actual Update Facility operations (insert/delete/
rename/replace, `declare updating function`).

Removing the exclusion and letting the real parse behavior decide restores
correct coverage for all 39 upd-* test-sets. Only 17 false positives
reappear across ~470 restored cases: 14 are genuine `declare updating
function` cases in upd-FunctionDeclaration (the one syntax that actually
fails to parse, confirmed directly against xq-parser), plus 3 unrelated
one-off cases. Total false positives: 109 -> 126; false negatives
unchanged at 534.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01NywD7BhMcq8yyRG3b6pTba

1010 of 1183 branches covered (85.38%)

Branch coverage included in aggregate %.

3672 of 3846 relevant lines covered (95.48%)

440430.56 hits per line