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

   ### Task Summary
   
   `OperatorPropertyEditFrameComponent` sits at codecov ~72% (117 uncovered 
lines). Goal: **cover the file's remaining unit-testable methods to bring its 
coverage toward complete** — mock the injected services and exercise every 
method that doesn't require the Quill rich-text editor. The families below are 
the likely gap; cover whichever the spec doesn't already hit, plus any other 
non-Quill method. Only the Quill binding is out of scope. jsdom/Vitest; see 
`frontend/TESTING.md`.
   
   ### Behavior to add
   
   **OperatorPropertyEditFrameComponent** 
(`frontend/src/app/workspace/component/property-editor/operator-property-edit-frame/operator-property-edit-frame.component.ts`,
 codecov ~72% — EXTEND; cover all remaining non-Quill methods)
   
   - Modify-logic gating: `checkOperatorProperty` / `allowModifyOperatorLogic` 
/ `confirmModifyOperatorLogic` — assert whether editing is allowed for a 
running vs idle execution and that confirm applies the change.
   - Predicates / transforms (pure): `isHuggingFaceOperator` / 
`formatTaskTitle` / `hideEnvNameWhenDefaultEnvChecked` / 
`patchPythonUdfEnvironmentSchema` / `typeInferenceOnLambdaFunction` — assert 
the returned value/schema for representative operator inputs.
   - Form state: `onFormChanges` / `setInteractivity` / `setFormlyFormBinding` 
/ `rerenderEditorForm` — assert the form model / interactivity flag updates.
   - Subscription handlers: `registerOperatorPropertyChangeHandler` / 
`registerOperatorSchemaChangeHandler` / 
`registerOperatorDisplayNameChangeHandler` / `registerOnFormChangeHandler` / 
`registerDisableEditorInteractivityHandler` — drive the source event through a 
mocked service and assert the component reacts.
   
   Out of scope (rich-text binding, needs Quill): `registerQuillBinding` / 
`connectQuillToText` / `disconnectQuillFromText` — leave those; do not mount a 
real Quill editor.
   
   ### 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