Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

unit tests for parseRequestURL function #709

Merged
merged 1 commit into from
Sep 25, 2023

Conversation

SVilgelm
Copy link
Contributor

  • unit tests for the PathParams, RawPathParams and QueryParams added by client and request
  • unit tests for client.HostURL and client.BaseURL
  • use client.BaseURL by default and fallback to client.HostURL if empty

* unit tests for the `PathParams`, `RawPathParams` and `QueryParams` added by client and request
* unit tests for `client.HostURL` and `client.BaseURL`
* use `client.BaseURL` by default and fallback to `client.HostURL` if empty
@codecov
Copy link

codecov bot commented Sep 25, 2023

Codecov Report

Patch coverage: 100.00% and project coverage change: +0.13% 🎉

Comparison is base (1ae2aac) 95.65% compared to head (546a0e3) 95.78%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #709      +/-   ##
==========================================
+ Coverage   95.65%   95.78%   +0.13%     
==========================================
  Files          12       12              
  Lines        1612     1615       +3     
==========================================
+ Hits         1542     1547       +5     
+ Misses         42       41       -1     
+ Partials       28       27       -1     
Files Changed Coverage Δ
middleware.go 92.97% <100.00%> (+0.74%) ⬆️

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

@SVilgelm
Copy link
Contributor Author

@jeevatkm as promised

@jeevatkm jeevatkm added this to the v2.9.0 milestone Sep 25, 2023
Copy link
Member

@jeevatkm jeevatkm left a comment

Choose a reason for hiding this comment

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

@SVilgelm Thank you for your unit test PR, appreciated!

@jeevatkm jeevatkm merged commit 106e689 into go-resty:master Sep 25, 2023
4 checks passed
@SVilgelm
Copy link
Contributor Author

yw

@SVilgelm SVilgelm deleted the parseRequestURL branch September 26, 2023 13:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

2 participants