Copilot commented on code in PR #6742:
URL: https://github.com/apache/texera/pull/6742#discussion_r3627045943
##########
frontend/src/app/workspace/component/result-exportation/result-exportation.component.spec.ts:
##########
@@ -165,4 +169,294 @@ describe("ResultExportationComponent", () => {
expect(component.userAccessibleDatasets[0]).toBe(created);
expect(component.inputDatasetName).toBe("brand-new");
});
+
+ describe("operator-id resolution and downloadability getters (menu source)",
() => {
+ // Reconfigure the graph mock so the "menu" branch of getOperatorIdsToCheck
+ // returns concrete operator IDs (drives the .map() callback on line 114).
Review Comment:
This comment references a specific line number ("line 114") in the component
implementation, which will drift as the component changes and can quickly
become misleading. Prefer describing the code path being exercised (e.g., the
`.getAllOperators().map(op => op.operatorID)` branch) without hard-coded line
numbers.
--
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]
