Skip to content

fix: add ok checks to type assertions in HTTP plugin tests#1338

Open
MD-Mushfiqur123 wants to merge 1 commit into
floatpane:masterfrom
MD-Mushfiqur123:fix/unchecked-type-assertions-http-tests
Open

fix: add ok checks to type assertions in HTTP plugin tests#1338
MD-Mushfiqur123 wants to merge 1 commit into
floatpane:masterfrom
MD-Mushfiqur123:fix/unchecked-type-assertions-http-tests

Conversation

@MD-Mushfiqur123
Copy link
Copy Markdown

Fixes #721. Added ok checks to type assertions in HTTP plugin tests so they fail gracefully with descriptive errors instead of panicking on type mismatch.

@MD-Mushfiqur123 MD-Mushfiqur123 requested a review from a team as a code owner May 22, 2026 06:10
@floatpanebot floatpanebot added the bug Something isn't working label May 22, 2026
Copy link
Copy Markdown
Member

@floatpanebot floatpanebot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @MD-Mushfiqur123! Please fix the following issues with your PR:

  • Title: Is too long (58 characters). The PR title must be strictly under 40 characters.
  • Body: Missing the ## What? or ## Why? headings required by the PR template.

@floatpanebot floatpanebot added area/plugin Plugin system size/S Diff: 11–50 lines and removed bug Something isn't working labels May 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/plugin Plugin system size/S Diff: 11–50 lines

Projects

None yet

Development

Successfully merging this pull request may close these issues.

BUG: Unchecked type assertions in HTTP plugin tests

2 participants