Node SDK: add unit tests for the delete topic command
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ó
- 2/5
- Thời gian dự kiến
- 1-3 giờ
- Mức phù hợp với người mới
- 88/100
- Loại issue
- Tính năng
- Độ rõ ràng
- Đặc tả rõ ràng
- Mức độ hoạt động
- Sôi nổi
- Công nghệ
- node.js, typescript
Hướng nghiên cứu
Đọc foreign/node/src/wire/topic/create-topic.command.test.ts và phần triển khai DELETE_TOPIC.serialize. Thêm foreign/node/src/wire/topic/delete-topic.command.test.ts với các buffer kỳ vọng được tạo thủ công cho định danh dạng số và dạng chuỗi, cùng các giá trị partitionsCount là 0 và một giá trị lớn. Chạy npm run test:unit và npm run lint trong foreign/node; hoàn thành khi cả hai đều thành công.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Description
DELETE_TOPIC.serialize in the Node SDK has no unit test. It writes the stream identifier, the topic identifier and a uint32 partitionsCount, with no range check. The sibling create-topic.command.test.ts and topic.utils.test.ts are tested.
Affected area / component
Node SDK
Proposed solution
Add foreign/node/src/wire/topic/delete-topic.command.test.ts, modelled on create-topic.command.test.ts.
- The exact bytes for numeric and string stream and topic identifiers.
- A
partitionsCountof 0 and of a large value. - Build the expected buffers by hand, so the test does not call the code it checks.
Adding validation of partitionsCount is out of scope: it changes behaviour and needs a maintainer's view first.
How to test: npm run test:unit and npm run lint pass in foreign/node.
Good first issue
- I think this could be a good first issue for a new contributor
- Ngôn ngữ chính
- Rust
- Star
- 5.2k
- Fork
- 516
- Merge trung bình
- 2 ngày 12 giờ
- Pull request đã merge (30 ngày)
- 145
Chuẩn bị môi trường
Khởi chạy dev container của dự án ngay trên trình duyệt, bằng tài khoản GitHub của bạn.
- Có Dockerfile hoặc tệp Docker Compose
- Có mẫu pull request
- Đọc hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- 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.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của apache/iggy
-
docs good first issue php
Độ khó 1/5 1-3 giờ Mức phù hợp với người mới 90/100
Maintainer thường phản hồi trong vòng 1 ngày
-
good first issue javascript test
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 88/100
Maintainer thường phản hồi trong vòng 1 ngày
-
good first issue javascript test
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 82/100
Maintainer thường phản hồi trong vòng 1 ngày
-
good first issue javascript test
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 90/100
Maintainer thường phản hồi trong vòng 1 ngày
-
good first issue php
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 88/100
Maintainer thường phản hồi trong vòng 1 ngày
Issue tương tự
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 82/100
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 67/100
Maintainer thường phản hồi trong vòng 1 ngày
-
bug(core): partial tool arguments drop existing string content during a split surrogate pairĐang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 76/100
0xPlaygrounds/rig#3176 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Publish memtop on crates.ioĐang mởpackaging
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 74/100
SalomonAvila/memtop#23 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Anthropic provider: 400 on tools.N.custom.name after tool_search adds web.run (name contains a dot)Đang mởarea:install area:mcp area:state area:tui os:macos
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 73/100