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 request/response observer via godog attachments #23

Merged
merged 7 commits into from
Oct 31, 2024
Merged

Conversation

vearutop
Copy link
Member

No description provided.

Copy link

Lines Of Code

Language Files Lines Code Comments Blanks Complexity Bytes
Go 3 1386 (+33) 905 (+34) 219 (+4) 262 (-5) 214 (-6) 38.4K (+2.2K)
Go (test) 6 778 (+1) 629 (-1) 14 135 (+2) 44 19.1K (-12B)
YAML 5 349 (-4) 284 (-4) 29 36 0 11.4K (-120B)

Copy link

Go API Changes

# github.com/godogx/httpsteps
## compatible changes
DefaultExposeHTTPDetails: added
HTTPValue: added
LocalClient.ExposeHTTPDetails: added

# summary
Inferred base version: v0.2.14
Suggested version: v0.3.0

Copy link

Unit Test Coverage

total: (statements) 72.3%
changed lines: (statements) 73.0%, coverage is less than 90.0%, consider testing the changes more thoroughly

Coverage of changed lines
File Function Coverage
Total 73.0%
local_client.go 73.0%
local_client.go:37 NewLocalClient 100.0%
local_client.go:231 afterScenario 100.0%
local_client.go:400 tableSetup 100.0%
local_client.go:683 iShouldHaveOtherResponsesWithHeader 100.0%
local_client.go:708 iShouldHaveResponseWithHeader 100.0%
local_client.go:733 iShouldHaveResponseWithBody 100.0%
local_client.go:743 iShouldHaveResponseWithBodyFromFile 100.0%
local_client.go:753 iShouldHaveResponseWithBodyThatMatchesJSON 100.0%
local_client.go:771 iShouldHaveResponseWithBodyThatMatchesJSONPaths 100.0%
local_client.go:781 iShouldHaveResponseWithBodyThatMatchesJSONFromFile 100.0%
local_client.go:791 iShouldHaveOtherResponsesWithBody 100.0%
local_client.go:811 iShouldHaveOtherResponsesWithBodyThatMatchesJSON 100.0%
local_client.go:821 iShouldHaveOtherResponsesWithBodyThatMatchesJSONPaths 100.0%
local_client.go:586 DefaultExposeHTTPDetails 84.0%
local_client.go:672 iShouldHaveResponseWithStatus 81.8%
local_client.go:714 iShouldHaveResponseWithHeaders 76.5%
local_client.go:763 augmentBodyErr 66.7%
local_client.go:647 expectResponse 62.5%
local_client.go:281 LoadBodyFromFile 0.0%
local_client.go:689 iShouldHaveOtherResponsesWithHeaders 0.0%
local_client.go:801 iShouldHaveOtherResponsesWithBodyFromFile 0.0%
local_client.go:831 iShouldHaveOtherResponsesWithBodyThatMatchesJSONFromFile 0.0%
Coverage diff with base branch
File Function Base Coverage Current Coverage
Total 70.5% 72.3% (+1.8%)
local_client.go DefaultExposeHTTPDetails no function 73.9%
local_client.go afterScenario 76.9% 81.8% (+4.9%)
local_client.go expectResponse no function 66.7%
local_client.go iShouldHaveOtherResponsesWithBody 85.7% 100.0% (+14.3%)
local_client.go iShouldHaveOtherResponsesWithBodyThatMatchesJSON 85.7% 100.0% (+14.3%)
local_client.go iShouldHaveOtherResponsesWithBodyThatMatchesJSONPaths 85.7% 100.0% (+14.3%)
local_client.go iShouldHaveOtherResponsesWithHeader 75.0% 100.0% (+25.0%)
local_client.go iShouldHaveResponseWithBody 85.7% 100.0% (+14.3%)
local_client.go iShouldHaveResponseWithBodyFromFile 85.7% 100.0% (+14.3%)
local_client.go iShouldHaveResponseWithBodyThatMatchesJSON 85.7% 100.0% (+14.3%)
local_client.go iShouldHaveResponseWithBodyThatMatchesJSONFromFile 85.7% 100.0% (+14.3%)
local_client.go iShouldHaveResponseWithBodyThatMatchesJSONPaths 85.7% 100.0% (+14.3%)
local_client.go iShouldHaveResponseWithHeader 75.0% 100.0% (+25.0%)
local_client.go iShouldHaveResponseWithHeaders 72.7% 77.8% (+5.1%)
local_client.go iShouldHaveResponseWithStatus 71.4% 80.0% (+8.6%)
local_client.go tableSetup 81.8% 83.3% (+1.5%)

@vearutop vearutop merged commit 1d87b3d into master Oct 31, 2024
6 checks passed
@vearutop vearutop deleted the observer branch October 31, 2024 18:15
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.

None yet

1 participant