Skip to content

Conversation

@Twilikiss
Copy link
Contributor

1.Comparison before and after correction

  • Before correction:
fix01
  • After correction
image

2. Unit Test Description

2.1 PostJson Testing (TestIsPostJson)

  • Verify the correct identification of the POST method and JSON tags

  • Key test: Non-POST methods should not return true (bug fixed before)

2.2 Parameter Generation Test (TestParameters from Type)

  • Test the key parameter generation scenarios

  • Verify the correct generation of the body parameter

2.3 Boundary Condition Test (TestParameters from type _edgecases)

  • Test the handling of nil and non-struct types

@kesonan kesonan added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Jul 13, 2025
@codecov
Copy link

codecov bot commented Jul 13, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 94.89%. Comparing base (8690859) to head (16500b1).
Report is 377 commits behind head on master.

Additional details and impacted files

see 21 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@kevwan kevwan force-pushed the fixbug-goctl-postjson branch from 16500b1 to 564b80c Compare July 13, 2025 02:55
@kevwan kevwan added this pull request to the merge queue Jul 13, 2025
Merged via the queue into zeromicro:master with commit 8cf4f95 Jul 13, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm "Looks good to me", indicates that a PR is ready to be merged.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants