Skip to content

Commit

Permalink
test_800_websockets: AH10467 is an expected error for the _fail_proto…
Browse files Browse the repository at this point in the history
… test.

So ignore it, and move the _fail_proto test first to avoid messing up with
the error logs of the other tests.



git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1910667 13f79535-47bb-0310-9956-ffa450edef68
  • Loading branch information
ylavic committed Jun 28, 2023
1 parent 5b6275f commit 56c4cfc
Showing 1 changed file with 8 additions and 7 deletions.
15 changes: 8 additions & 7 deletions test/modules/http2/test_800_websockets.py
Original file line number Diff line number Diff line change
Expand Up @@ -145,20 +145,21 @@ def ws_server(self, env):
yield
p.terminate()

# CONNECT with invalid :protocol header, must fail
def test_h2_800_01_fail_proto(self, env: H2TestEnv, ws_server):
r, infos, frames = ws_run(env, path='/ws/echo/', scenario='fail-proto')
assert r.exit_code == 0, f'{r}'
assert infos == ['[1] :status: 501', '[1] EOF'], f'{r}'
env.httpd_error_log.ignore_recent()

# a correct CONNECT, send CLOSE, expect CLOSE, basic success
def test_h2_800_01_ws_empty(self, env: H2TestEnv, ws_server):
def test_h2_800_02_ws_empty(self, env: H2TestEnv, ws_server):
r, infos, frames = ws_run(env, path='/ws/echo/')
assert r.exit_code == 0, f'{r}'
assert infos == ['[1] :status: 200', '[1] EOF'], f'{r}'
assert len(frames) == 1, f'{frames}'
assert frames[0].opcode == WsFrame.CLOSE, f'{frames}'

# CONNECT with invalid :protocol header, must fail
def test_h2_800_02_fail_proto(self, env: H2TestEnv, ws_server):
r, infos, frames = ws_run(env, path='/ws/echo/', scenario='fail-proto')
assert r.exit_code == 0, f'{r}'
assert infos == ['[1] :status: 501', '[1] EOF'], f'{r}'

# CONNECT to a URL path that does not exist on the server
def test_h2_800_03_not_found(self, env: H2TestEnv, ws_server):
r, infos, frames = ws_run(env, path='/does-not-exist')
Expand Down

0 comments on commit 56c4cfc

Please sign in to comment.