Repository navigation
Replace httpbin.org/gzip Tests with WebListener and Re-Enable Deflate Tests - #4948
Conversation
| } | ||
| } | ||
| # Validate response content | ||
| $result.Output.Headers.'Content-Encoding'[0] | Should Be $dataEncoding |
There was a problem hiding this comment.
Maybe, Should BeExactly?
There was a problem hiding this comment.
Fixed
| # Validate response content | ||
| $result.Output.Headers.'Content-Encoding'[0] | Should Be $dataEncoding | ||
| $jsonContent = $result.Output.Content | ConvertFrom-Json | ||
| $jsonContent.Headers.Host | Should Be $uri.Authority |
There was a problem hiding this comment.
Same as above.
There was a problem hiding this comment.
Fixed
| } | ||
| } | ||
| # Validate response content | ||
| $headers.'Content-Encoding'[0] | Should Be $dataEncoding |
There was a problem hiding this comment.
Same as above
There was a problem hiding this comment.
Fixed
| } | ||
| # Validate response content | ||
| $headers.'Content-Encoding'[0] | Should Be $dataEncoding | ||
| $result.Headers.Host | Should Be $uri.Authority |
There was a problem hiding this comment.
Same as above
There was a problem hiding this comment.
Fixed
| "args": {}, | ||
| "origin": "127.0.0.1", | ||
| "headers": { | ||
| "User-Agent": "Mozilla/5.0 (Windows NT; Windows NT 10.0; en-US) WindowsPowerShell/5.1.15063.608", |
There was a problem hiding this comment.
Should this be PowerShell/6.0.0 instead of WindowsPowerShell?
There was a problem hiding this comment.
Fixed
| "args": {}, | ||
| "origin": "127.0.0.1", | ||
| "headers": { | ||
| "User-Agent": "Mozilla/5.0 (Windows NT; Windows NT 10.0; en-US) WindowsPowerShell/5.1.15063.608", |
There was a problem hiding this comment.
Should this be PowerShell/6.0.0 instead of WindowsPowerShell?
There was a problem hiding this comment.
Fixed
|
test fails unrelated (#4958) |
|
Mark Kraus (@markekraus) those numbers probably refer to old bugs we used to track internally in TFS, quite possibly no longer relevant |
|
Mark Kraus (@markekraus) The tests seemed to have hanged in Invoke-WebRequest tests. Please have a look. |
|
Aditya Patwardhan (@adityapatwardhan) I'm not sure it is related. https://travis-ci.org/PowerShell/PowerShell/jobs/281240156 had passed and the only changes since were superficial. I think it was a random macOS issue on Travis CI. I've pushed an empty commit to rerun the tests |
|
Aditya Patwardhan (@adityapatwardhan) speaking of unrelated macOS failures in Travis CI https://travis-ci.org/PowerShell/PowerShell/jobs/282354703 :( Does someone have access to restart just that Job? Or should just push another empty commit? |
|
Mark Kraus (@markekraus) I restarted it for you |
|
Steve Lee (@SteveL-MSFT) Thanks! |
Aditya Patwardhan (adityapatwardhan)
left a comment
There was a problem hiding this comment.
LGTM
… Tests (PowerShell#4948) * Add Gzip and Deflate Support to WebListener * [Feature] Run Feature tests * [Feature] Address PR Feedback * [Feature] Re-Run CI * [feature] Update WebListener Index page * [Feature] Run Feature tests * [Feature] Re-run CI
Compression/Gziptest to WebListenerCompression/Deflatetest to WebListener-TestCasefromforeachI have no idea what ''7976639:Invoke-WebRequest does not support -TransferEncoding deflate'' and "7976639:Invoke-RestMethod does not support -TransferEncoding deflate" are, but in my limited testing it appears to no longer be true.
reference #2504