Node SDK: add unit tests for the delete topic command
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
- 難易度
- 2/5
- 見積もり時間
- 1〜3時間
- 初心者へのやさしさ
- 88/100
- issue の種類
- 機能追加
- 明瞭さ
- 明確に書かれている
- 活発さ
- 活発
- 技術スタック
- node.js, typescript
調査の方向性
foreign/node/src/wire/topic/create-topic.command.test.tsとDELETE_TOPIC.serializeの実装を読みます。数値および文字列の識別子と、0および大きな値のpartitionsCountを対象に、手作業で作成した期待バッファを使ってforeign/node/src/wire/topic/delete-topic.command.test.tsを追加します。foreign/nodeでnpm run test:unitとnpm run lintを実行し、両方が成功すれば完了です。
索引モデルが issue の本文から書いたものです。
説明
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
- 主要言語
- Rust
- スター
- 5.2k
- フォーク
- 516
- 平均マージ
- 2日 12時間
- マージ済み PR（30日）
- 145
環境構築
このプロジェクトの開発コンテナを、あなたの GitHub アカウントでブラウザ上に起動します。
- Dockerfile または Docker Compose ファイルあり
- プルリクエストのテンプレートあり
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
apache/iggy のほかの issue
-
docs good first issue php
難易度 1/5 1〜3時間 初心者へのやさしさ 90/100
メンテナーはふだん 1 日以内に返信
-
good first issue javascript test
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
メンテナーはふだん 1 日以内に返信
-
good first issue javascript test
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
メンテナーはふだん 1 日以内に返信
-
good first issue javascript test
難易度 2/5 1〜3時間 初心者へのやさしさ 90/100
メンテナーはふだん 1 日以内に返信
-
good first issue php
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
メンテナーはふだん 1 日以内に返信
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 67/100
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
0xPlaygrounds/rig#3176 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
packaging
難易度 2/5 1〜3時間 初心者へのやさしさ 74/100
SalomonAvila/memtop#23 ·
メンテナーはふだん 1 日以内に返信
-
Anthropic provider: 400 on tools.N.custom.name after tool_search adds web.run (name contains a dot)オープンarea:install area:mcp area:state area:tui os:macos
難易度 2/5 1〜3時間 初心者へのやさしさ 73/100