Skip to content

Comments

[plugin-web-app] Add ability to capture http response in proxy#2147

Merged
valfirst merged 1 commit intomasterfrom
capture-proxy-response
Nov 8, 2021
Merged

[plugin-web-app] Add ability to capture http response in proxy#2147
valfirst merged 1 commit intomasterfrom
capture-proxy-response

Conversation

@uarlouski
Copy link
Member

@uarlouski uarlouski commented Nov 4, 2021

Closes #1772

@codecov
Copy link

codecov bot commented Nov 4, 2021

Codecov Report

Merging #2147 (cd4a995) into master (c2b7a48) will decrease coverage by 0.00%.
The diff coverage is 100.00%.

❗ Current head cd4a995 differs from pull request most recent head 2cf8960. Consider uploading reports for the commit 2cf8960 to get more accurate results
Impacted file tree graph

@@             Coverage Diff              @@
##             master    #2147      +/-   ##
============================================
- Coverage     96.21%   96.21%   -0.01%     
+ Complexity     5362     5361       -1     
============================================
  Files           757      757              
  Lines         15322    15317       -5     
  Branches       1031     1029       -2     
============================================
- Hits          14742    14737       -5     
  Misses          450      450              
  Partials        130      130              
Impacted Files Coverage Δ
...c/main/java/org/vividus/softassert/SoftAssert.java 89.78% <100.00%> (-0.36%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c2b7a48...2cf8960. Read the comment docs.

@lgtm-com
Copy link

lgtm-com bot commented Nov 4, 2021

This pull request introduces 2 alerts when merging 0f7f7b213d82fbaa2d6f1f8a429867f333f052be into 050b53d - view on LGTM.com

new alerts:

  • 2 for Spurious Javadoc @param tags

@ikalinin1 ikalinin1 mentioned this pull request Nov 5, 2021
@uarlouski uarlouski force-pushed the capture-proxy-response branch 2 times, most recently from a2d9745 to 1fc4b0a Compare November 5, 2021 13:43
@uarlouski uarlouski marked this pull request as ready for review November 5, 2021 14:35
@uarlouski uarlouski force-pushed the capture-proxy-response branch from 1fc4b0a to 32f7abf Compare November 5, 2021 14:50
@vividus-framework vividus-framework deleted a comment from lgtm-com bot Nov 5, 2021
@vividus-framework vividus-framework deleted a comment from lgtm-com bot Nov 5, 2021
@vividus-framework vividus-framework deleted a comment from lgtm-com bot Nov 5, 2021
@vividus-framework vividus-framework deleted a comment from lgtm-com bot Nov 5, 2021
@uarlouski uarlouski force-pushed the capture-proxy-response branch from 32f7abf to 8d5455f Compare November 5, 2021 15:59
@uarlouski uarlouski requested a review from valfirst November 5, 2021 16:01
@uarlouski uarlouski force-pushed the capture-proxy-response branch from 8d5455f to 2cf8960 Compare November 5, 2021 16:12
@uarlouski uarlouski requested a review from valfirst November 5, 2021 16:13
@sonarqubecloud
Copy link

sonarqubecloud bot commented Nov 5, 2021

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

100.0% 100.0% Coverage
0.0% 0.0% Duplication

@lgtm-com
Copy link

lgtm-com bot commented Nov 5, 2021

This pull request introduces 2 alerts when merging 2cf8960 into cd4a995 - view on LGTM.com

new alerts:

  • 2 for Spurious Javadoc @param tags

@valfirst valfirst merged commit 35b0f6c into master Nov 8, 2021
@valfirst valfirst deleted the capture-proxy-response branch November 8, 2021 11:05
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.

Ability to save response body to the variable to check it

3 participants