Skip to content

Add test coverage to fixRequestBody and responseInterceptor#608

Merged
chimurai merged 2 commits intochimurai:masterfrom
leonardobazico:add-tests
May 19, 2021
Merged

Add test coverage to fixRequestBody and responseInterceptor#608
chimurai merged 2 commits intochimurai:masterfrom
leonardobazico:add-tests

Conversation

@leonardobazico
Copy link
Copy Markdown
Contributor

@leonardobazico leonardobazico commented May 14, 2021

Description

Just add unit tests to cover some use cases for fixRequestBody and responseInterceptor

Motivation and Context

Unit tests for handlers

How has this been tested?

United

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.

@ghost
Copy link
Copy Markdown

ghost commented May 14, 2021 •
edited by ghost
Loading

Congratulations 🎉. DeepCode analyzed your code in 2.577 seconds and we found no issues. Enjoy a moment of no bugs ☀️.

👉 View analysis in DeepCode’s Dashboard | Configure the bot

@coveralls
Copy link
Copy Markdown

coveralls commented May 14, 2021 •
edited
Loading

Coverage Status

Coverage increased (+0.6%) to 98.857% when pulling 4ec3130 on leonardobazico:add-tests into a6f8e0d on chimurai:master.

@leonardobazico leonardobazico force-pushed the add-tests branch 3 times, most recently from d87cf57 to 639e380 Compare May 18, 2021 16:15
@leonardobazico
Copy link
Copy Markdown
Contributor Author

leonardobazico commented May 18, 2021

Hi @chimurai, do you know why the checks are failing?

@chimurai
Copy link
Copy Markdown
Owner

chimurai commented May 18, 2021

Looks like github-actions is having some issues: https://www.githubstatus.com/incidents/m16jzl31gnqt

@chimurai chimurai merged commit 6fd75f7 into chimurai:master May 19, 2021
@chimurai
Copy link
Copy Markdown
Owner

chimurai commented May 19, 2021

Thanks a bunch!

@iamsuganthi iamsuganthi mentioned this pull request Aug 6, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@leonardobazico @coveralls @chimurai