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

   ### Task Summary
   
   `joint-graph-wrapper.ts` is the adapter between the Texera graph model and 
the JointJS paper —
   positioning, layers, highlighting, and the co-editor presence rings all go 
through it. Live Codecov
   puts it at 76.0% lines with 19 partial branches, and the branch figure is 
the real gap: **43 of 85
   branches taken (50.6%)**. A line counts as covered when only one side of its 
condition ever runs.
   
   Everything in the file is reachable under jsdom with no extra 
infrastructure, which is worth
   recording because it is not obvious: a real `joint.dia.Paper` constructs and 
renders against a
   detached `div`, `paper.findViewByModel()` returns a real view, 
`joint.highlighters.mask` applies and
   removes, and `joint.layout.DirectedGraph.layout` runs with dagre/graphlib. 
No browser-mode test is
   needed.
   
   The `HTMLCanvasElement.prototype.getContext` warnings in the log are 
pre-existing and non-fatal —
   they come from `JointUIService.truncateOperatorDisplayName` measuring text.
   
   ### 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