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

Add MultipartRequest log to CurlInterceptor #435

Merged
merged 2 commits into from
May 27, 2023
Merged

Add MultipartRequest log to CurlInterceptor #435

merged 2 commits into from
May 27, 2023

Conversation

JosephNK
Copy link
Contributor

When using CurlInterceptor, the log output for MultipartRequest was missing. So, I added the Log of MultipartRequest. :)

@codecov
Copy link

codecov bot commented May 27, 2023

Codecov Report

Merging #435 (ca778fd) into develop (0546452) will increase coverage by 0.09%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##           develop     #435      +/-   ##
===========================================
+ Coverage    93.65%   93.75%   +0.09%     
===========================================
  Files            8        8              
  Lines          441      448       +7     
===========================================
+ Hits           413      420       +7     
  Misses          28       28              
Impacted Files Coverage Δ
chopper/lib/src/interceptor.dart 76.11% <100.00%> (+2.78%) ⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@techouse
Copy link
Collaborator

Would you mind please adding some tests for this?

@techouse techouse self-assigned this May 27, 2023
@techouse techouse added the enhancement New feature or request label May 27, 2023
@JosephNK
Copy link
Contributor Author

How can I do it?

@techouse
Copy link
Collaborator

Check these out and make some similar test cases covering them. You can run them using make tests inside the chopper directory.

@techouse techouse merged commit ae8bff5 into lejard-h:develop May 27, 2023
@techouse techouse mentioned this pull request May 28, 2023
techouse added a commit that referenced this pull request May 28, 2023
# chopper

## 6.1.3

* Add follow redirects to toHttpRequest (#430)
* Update http constraint to ">=0.13.0 <2.0.0" (#431)
* Add MultipartRequest log to CurlInterceptor (#435)

---

# chopper_built_value

## 1.2.2

* Update http constraint to ">=0.13.0 <2.0.0" (#431)

---

# example

* Update squadron example (#432)

---

# Github actions

* Add cleanup step to publish workflow (#434)

---------

Co-authored-by: Joran Dob <joran998@gmail.com>
Co-authored-by: Joseph, NamKung <nkw0608@gmail.com>
Co-authored-by: Klemen Tusar <techouse@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants