Renderer suites flake under CPU load: fixed delays before async state (git-components, file-find-dom, mailbox-inbox-components, others)

Đang mở
#815 0 bình luận 0 reaction 0 người được giao Xem trên GitHub

Maintainer thường phản hồi trong vòng 1 ngày

Chưa có ai nhận issue này.

Đánh giá

Độ khó
3/5
Thời gian dự kiến
1-2 ngày
Mức phù hợp với người mới
70/100
Loại issue
Lỗi
Độ rõ ràng
Khá rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
electron, typescript
Lĩnh vực
desktop-dev, testing

Hướng nghiên cứu

Run node scripts/test-renderer.mjs to reproduce the flaky suites. Locate the test files for git-components, file-find-dom, and mailbox-inbox-components (likely under a renderer test dir) and replace fixed delay() calls with waits for rendered state or attached listeners. Confirm by running the suites alone and under load until they pass consistently.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

type:bug

Found while fixing #802. On a heavily loaded Mac (load average about 40), a full node scripts/test-renderer.mjs run failed these suites on both attempts. None of them mount the Composer or the editor touched in #802:

  • git-components: environment and branch dropdowns: This Mac (0/2 when run alone). The suite waits a fixed delay(120) after rendering EnvironmentFooter, before the branch/environment fetch lands.
  • file-find-dom: the Find bar opens (1/2). It dispatches Cmd+F 60 ms after mount, so the window keydown listener's effect may not be attached yet.
  • mailbox-inbox-components: expected 3, got 0 (1/2).
  • git-history-components (0 painted of 150 known), governance-components and html-preview-components failed in the full run but passed 2/2 when run alone.

These are the same kind of problem as #802: a fixed delay before asserting on state that arrives through an effect or a promise. Fix: wait for the real signal (the rendered state), and for key or click input, wait until the listener or handler is attached.

Ngôn ngữ chính
TypeScript
Star
3
Fork
0
Merge trung bình
17 giờ 33 phút
Pull request đã merge (30 ngày)
163

Chuẩn bị môi trường

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của musterhq/muster-code

Tất cả issue của musterhq/muster-code

Issue tương tự

Thêm issue về TypeScript

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.