-
Notifications
You must be signed in to change notification settings - Fork 4.6k
test: add test for invalid streamID #6940
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
Merged
Merged
Changes from 4 commits
Commits
Show all changes
28 commits
Select commit
Hold shift + click to select a range
24816f2
add test with invalid client id
AnandInguva 4300406
add test for invalid frame
AnandInguva 71b7ee7
Check if the err is nil or not
AnandInguva 9815dbc
Add comment for the test
AnandInguva 3bd77df
Run the test against a single env
AnandInguva bee7003
Remove redundant comments
AnandInguva 6b13297
Remove funcServer and try to assert the error
AnandInguva 5fff887
Add test for illegal stream id on server operateHeader
AnandInguva ac91174
assert for io.EOF error
AnandInguva 577a1b7
Address comments
AnandInguva f25503b
Fix assertion
AnandInguva c68ded3
Add back new line
AnandInguva 3e863de
Send GoAwayFrame when illegal streamID is received
AnandInguva 4338789
Remove comment
AnandInguva f1855d3
Merge remote-tracking branch 'upstream/master' into add_test
AnandInguva 34dd481
Use servertester wantGoFrame for assertion
AnandInguva 14ecbb8
Return error and add GoAwayFrame
AnandInguva cfa206b
Update test/end2end_test.go
AnandInguva 4d3bc72
Add a comment on error case
AnandInguva 26c215b
Refactor test
AnandInguva fcff403
Break comment
AnandInguva ad742c3
Add space
AnandInguva 7a1a6f8
Modify assertion
AnandInguva 7bbea1f
Fix condition
AnandInguva fa61215
Add test for stream ID lower than previous frame's streamID
AnandInguva 6b79110
Fix test
AnandInguva a1d33da
rewrite error message
AnandInguva 3522f84
Update error message
AnandInguva File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.