test: got 100% coverage (branch and line)

[bougyman]

No description provided.

[Copilot]

Pull request overview

This PR adds comprehensive test coverage to achieve 100% branch and line coverage for the Kalshi API client library. The tests cover previously untested code paths including error handling, validation edge cases, and class-level method invocations.

• Adds validation tests for invalid filter parameters (both keys and values)
• Adds tests for the Contract class's propertize method to cover the inherited class path
• Adds tests for Client error handling and the singleton client accessor

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File	Description
test/kalshi/market/series_list_test.rb	Improves test naming clarity and adds validation test for invalid filter value types to achieve branch coverage
test/kalshi/contract_test.rb	New test file covering the Contract class's property definition mechanism including edge cases
test/kalshi/client_test.rb	New test file covering Client error handling and the module-level client singleton pattern

[Copilot]

The test comment is redundant and states the obvious. Either remove the comment entirely or make it more informative by explaining why this edge case matters for coverage (e.g., "Ensures propertize handles properties that already exist via Reform's property method").

Suggested change

	# This test just ensures the class definition above runs and exercises the code path
	# Ensures propertize correctly adds properties on a class that already defines properties via property