Skip to content

Conversation

@deviantintegral
Copy link
Owner

No description provided.

@deviantintegral deviantintegral changed the base branch from main to test-cleanup December 18, 2025 00:45
@deviantintegral deviantintegral changed the title Claude/remove duplicate tests 71hbp test: remove redundant test/assertion in ResponseTest Dec 18, 2025
@deviantintegral deviantintegral force-pushed the claude/remove-duplicate-tests-71hbp branch from 95fffc0 to c16f0d3 Compare December 18, 2025 00:45
@deviantintegral deviantintegral marked this pull request as ready for review December 18, 2025 00:48
Remove testGetSetStatus as it is redundant - the same code paths
are already tested by testFromPsr7 and testSerialize.
Remove testGetSetStatusText as it is redundant - the same code paths
are already tested by testFromPsr7 and testSerialize.
Remove testGetSetContent as it is redundant - the same code paths
are already tested by testFromPsr7 and testSerialize.
Remove testGetSetRedirectURL as it is redundant - the same code paths
are already tested by testSerialize.
Remove testSetHeadersCalculatesCorrectSize as it is redundant -
the same code paths are already tested by testSetHeadersWithMultipleHeaders
and testSetHeadersWithSingleCharacterValues.
Remove testSetHeadersWithSingleCharacterValues as it is redundant -
the same code paths are already tested by testFromPsr7 (which also
calls setHeaders with a single header) and testSetHeadersWithMultipleHeaders.
@deviantintegral deviantintegral force-pushed the claude/remove-duplicate-tests-71hbp branch from c16f0d3 to 6c2f4d6 Compare December 18, 2025 00:48
@deviantintegral deviantintegral merged commit b68077b into test-cleanup Dec 18, 2025
9 checks passed
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.

3 participants