aglinxinyuan opened a new issue, #7771:
URL: https://github.com/apache/texera/issues/7771

   ### Task Summary
   
   Nine small frontend templates plus one parser service sit just below their 
neighbours. In every one of these cases the specs drive the component's methods 
directly and never render or click, so the templates' event bindings are 
genuinely unexecuted.
   
   | File | Coverage |
   |---|---|
   | 
`workspace/component/dataset-version-selector/dataset-version-selector.component.html`
 | 71.4% |
   | `workspace/component/left-panel/settings/settings.component.html` | 90.0% |
   | 
`workspace/component/left-panel/versions-list/versions-list.component.html` | 
91.3% |
   | `workspace/component/workflow-editor/mini-map/mini-map.component.html` | 
91.7% |
   | `dashboard/component/user/sort-button/sort-button.component.html` | 92.6% |
   | `dashboard/.../user-dataset-version-creator.component.html` | 93.1% |
   | 
`common/service/user/registration-request-modal/registration-request-modal.component.html`
 | 89.5% |
   | 
`workspace/component/dataset-selection-modal/dataset-selection-modal.component.html`
 | 89.7% |
   | `workspace/service/code-editor/ui-udf-parameters-parser.service.ts` | 
90.2% |
   
   **Worth stating up front: none of these is an instance of #7458.** No target 
spec uses `TestBed.overrideComponent`, so there is no attribution loss to 
recover — the remedy applied in #7535/#7627/#7629/#7661/#7681/#7727 is not what 
is needed. These are simply untested bindings, and the fix is to render and 
interact.
   
   Regions that are genuinely unreachable and should be left alone:
   
   - **`hugging-face.component.html:201-203` is gated on a typo** (see below), 
so its validation message can never render. Do not cover it; covering it 
cements the bug.
   - **The parser has six unreachable branch arms**, verified by probing 
`@lezer/python`'s error recovery across ~20 malformed sources: a dangling 
`name=` always gains a zero-width error node, so the `!value` guard cannot 
fire; an `AssignOp` inside an `ArgList` is always preceded by a `VariableName`; 
`|| "parameter"` only fires for `""`, which an earlier guard already rejects; 
`/^[ \t]*/` always matches so its `?? ""` is dead; and both `parts` guards are 
fed by nodes that always carry the relevant child.
   
   ### Task Type
   
   - [ ] Refactor / Cleanup
   - [ ] DevOps / Deployment / CI
   - [x] Testing / QA
   - [ ] Documentation
   - [ ] Performance
   - [ ] Other
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to