packages/components/cascader-panel/__tests__/cascader-panel.test.tsx (1)

602-631: ⚡ Quick win

Well-written test that correctly validates the lazy-load race condition fix.

The test properly verifies that loadLazyRootNodes() prevents duplicate lazyLoad invocations both during the pending state and after completion. The use of fake timers and clear assertions makes the test easy to understand and maintain.

Optional enhancement: Verify loaded state after timer completion

Consider adding an assertion after line 627 to verify that the nodes were actually loaded before the final reload attempt. This would make the test more robust:

 vi.runAllTimers()
 await nextTick()
+const nodes = wrapper.findAll(NODE)
+expect(nodes.length).toBe(1)
 vm.loadLazyRootNodes()
 expect(lazyLoad).toHaveBeenCalledTimes(1)

This ensures the component reached the expected "loaded" state before testing the post-load behavior.

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@packages/components/cascader-panel/__tests__/cascader-panel.test.tsx` around
lines 602 - 631, Add a post-timer assertion to confirm the lazy nodes were
actually loaded before the final reload attempt: after vi.runAllTimers() and
await nextTick(), assert the component state (e.g., check vm.rootNodes or
wrapper.findComponent(CascaderPanel).vm data that stores loaded nodes, or verify
the rendered node with value 'loaded') to ensure the lazyLoad resolution took
effect prior to calling vm.loadLazyRootNodes() and the final
expect(lazyLoad).toHaveBeenCalledTimes(1).