Skip to content

Use SDK test suite and harness for C# SDK - #5684

Draft
lisandroct wants to merge 16 commits into
masterfrom
lisandro/chsarp-sdk-testing
Draft

Use SDK test suite and harness for C# SDK#5684
lisandroct wants to merge 16 commits into
masterfrom
lisandro/chsarp-sdk-testing

Conversation

@lisandroct

@lisandroct lisandroct commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

Description of Changes

Adds tests for the C# SDK using the test suite and harness following closely what we do for Rust and Unreal.

Implementing this, I found (and fixed) three problems in the C# SDK:

  • Enums could not be used as Primary Keys or Index
  • Unsubscribing from a pending subscription was not supported
  • Zero-width rows were not supported

API and ABI breaking changes

No breaking changes. It just adds tests and fixes three bugs.

Expected complexity level and risk

  1. Very low risk. It's mostly tests (that should be passing) and only 3 source files that should be reviewed more closely.

Testing

  • The new tests all pass

@lisandroct
lisandroct marked this pull request as draft August 7, 2026 14:33
@lisandroct lisandroct linked an issue Aug 7, 2026 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Use SDK test suite and harness for C# SDK

1 participant