Skip to content

fix: Update the Http() stub so facades.Http() return client.Factory - #1584

Closed
vendion wants to merge 1 commit into
goravel:v1.18.xfrom
vendion:v1.18.0
Closed

vendion wants to merge 1 commit into
goravel:v1.18.xfrom
vendion:v1.18.0

Conversation

@vendion

@vendion vendion commented Oct 19, 2026

Copy link
Copy Markdown
Contributor

📑 Description

Currently the stub was generating the Http() facade with a response of client.Request rather than the expected client.Factory.

Closes goravel/goravel#1021

✅ Checks

  • Added test cases for my code

Currently the stub was generating the `Http()` facade with a response of
`client.Request` rather than the expected `client.Factory`.

Signed-off-by: Adam Jimerson <vendion@gmail.com>
@vendion
vendion requested a review from a team as a code owner October 19, 2026 12:42
@codecov

codecov Bot commented Oct 19, 2026 •

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 72.99%. Comparing base (a554ced) to head (39520df).
⚠️ Report is 1 commits behind head on v1.18.x.

Files with missing lines Patch % Lines
http/setup/stubs.go 0.00% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           v1.18.x    #1584      +/-   ##
===========================================
+ Coverage    69.60%   72.99%   +3.39%     
===========================================
  Files          384      412      +28     
  Lines        30242    28197    -2045     
===========================================
- Hits         21050    20583     -467     
+ Misses        8195     7612     -583     
+ Partials       997        2     -995     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@hwbrzzl hwbrzzl left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks, could you set the base branch to master? I'll handle other version if needed.

@vendion

vendion commented Oct 20, 2026

Copy link
Copy Markdown
Contributor Author

Thanks, could you set the base branch to master? I'll handle other version if needed.

Yeah, I'll have to close this PR and open a new one due to not having write permissions.

@vendion vendion closed this Oct 20, 2026
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.

2 participants