ML-dev-crypto commented on issue #2628: URL: https://github.com/apache/iggy/issues/2628#issuecomment-3841738443
Hi maintainers 👋 Please assign this issue to me (@ML-dev-crypto). Proposed Solution / Plan of Action Review Existing BDD Scenario Analyze the leader_redirection scenario defined in bdd/scenarios/basic_messaging.feature. Understand the expected behavior and steps used in other SDKs (if applicable) for consistency. C# BDD Test Implementation Add the missing scenario to the Iggy_SDK.Tests.BDD project. Implement corresponding step definitions in C# using the existing BDD test framework and patterns. Ensure proper handling of leader redirection logic in the test flow (e.g., client reconnects or retries against the leader node). Integration & Validation Run the full BDD test suite locally to confirm the new scenario passes. Verify that the test fails appropriately when leader redirection is not handled, ensuring it provides meaningful coverage. Deliverables New BDD scenario implementation for leader_redirection in the C# SDK tests. Any required step definition or helper additions. A PR with clean commits and test results confirming no regressions. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
