apache / apache/iggy

Go SDK: Add unit tests for command serialization/deserialization

Open
#2,883 3 comments 0 reactions 0 assignees View on GitHub
go sdk test
Dominant language
Rust
Stars
4.9k
Forks
432
Avg merge
2d 10h
Merged PRs (30d)
173

Description

## Description
The Go SDK contains numerous command types that implement the `MarshalBinary` interface to handle binary serialization. Currently, many of these implementations lack comprehensive unit tests, which is essential for ensuring data integrity and protocol compatibility.

## Task
Add unit tests to verify the correctness of the `MarshalBinary` implementations for all command types. The tests should ensure that the serialized output matches the expected binary format used by the Iggy server.

Go Command Types: [foreign/go/internal/command](https://github.com/apache/iggy/tree/master/foreign/go/internal/command). You should write tests for them.

Contributor guide

Open the contributing guide

Research direction

Start in foreign/go/internal/command and inventory the command types that implement MarshalBinary. Read the existing serialization code and identify the expected binary format for each command, then add unit tests covering all command types and verify their serialized output matches the Iggy server protocol.

Written by the indexing model from the issue text.

Assessment

Tech stack
go
Domain
testing
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Active
Clarity
Mostly clear
Newbie friendliness
66/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.