Copilot commented on code in PR #7351:
URL: https://github.com/apache/texera/pull/7351#discussion_r3726526003
##########
frontend/src/app/workspace/component/property-editor/operator-property-edit-frame/operator-property-edit-frame.component.spec.ts:
##########
@@ -30,6 +30,13 @@ import { CustomJSONSchema7 } from
"../../../types/custom-json-schema.interface";
import { OperatorMetadataService } from
"../../../service/operator-metadata/operator-metadata.service";
import { StubOperatorMetadataService } from
"../../../service/operator-metadata/stub-operator-metadata.service";
import { FORM_DEBOUNCE_TIME_MS } from
"../../../service/execute-workflow/execute-workflow.service";
+
+/**
+ * Drains the short timers scheduled while the fixture is being set up, before
the form change
+ * under test. Unrelated to FORM_DEBOUNCE_TIME_MS, which is what the
assertions below actually wait
+ * on; this previously borrowed COLLAB_DEBOUNCE_TIME_MS (also 10ms) from an
unrelated component.
+ */
+const SETUP_FLUSH_MS = 10;
import { DatePipe } from "@angular/common";
Review Comment:
`SETUP_FLUSH_MS` is declared between import statements. Keeping all imports
contiguous (and defining module constants after the import block) makes the
file easier to scan and avoids issues with tooling that assumes imports come
first.
--
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]
