diff options
author | John Thacker <johnthacker@gmail.com> | 2022-12-06 07:21:58 -0500 |
---|---|---|
committer | John Thacker <johnthacker@gmail.com> | 2022-12-06 07:21:58 -0500 |
commit | 74b1c07f913fcedcf9c28795b93ac53ee654fb29 (patch) | |
tree | e85f7c2f115abd720d448dbfa2fc377040c537f2 | |
parent | 2fcc819366b63d5d393250096125fe767ab3db5f (diff) |
test: Skip Follow HTTP2 test without Nghttp2
The expected test output is with the headers decompressed, which
we can't do without Nghttp2. (It outputs the compressed headers
if we don't have it, so we could test for that instead.)
Fix #18707
-rw-r--r-- | test/suite_follow_multistream.py | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/test/suite_follow_multistream.py b/test/suite_follow_multistream.py index c233695b47..852f2ec436 100644 --- a/test/suite_follow_multistream.py +++ b/test/suite_follow_multistream.py @@ -12,9 +12,13 @@ import fixtures @fixtures.mark_usefixtures('test_env') @fixtures.uses_fixtures class case_follow_multistream(subprocesstest.SubprocessTestCase): - def test_follow_http2_multistream(self, cmd_tshark, capture_file): + def test_follow_http2_multistream(self, cmd_tshark, capture_file, features): '''Checks whether Follow HTTP2 correctly handles multiple streams on the same packet.''' - + # If we don't have nghttp2, we output the compressed headers. + # We could test against the expected output in that case, but + # just skip for now. + if not features.have_nghttp2: + self.skipTest('Requires nghttp2.') # Test 1: # 1. While following stream 25 we should ignore stream 21 at frame 65 proc = self.assertRun((cmd_tshark, |