Merge pull request #8409 from francoisfreitag/linkcheck-tests

linkcheck: Remove unused arguments from tests
This commit is contained in:
Takeshi KOMIYA 2020-11-12 02:17:52 +09:00 committed by GitHub
commit 386929ec0f
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

View File

@ -19,7 +19,7 @@ from utils import http_server
@pytest.mark.sphinx('linkcheck', testroot='linkcheck', freshenv=True) @pytest.mark.sphinx('linkcheck', testroot='linkcheck', freshenv=True)
def test_defaults(app, status, warning): def test_defaults(app):
app.builder.build_all() app.builder.build_all()
assert (app.outdir / 'output.txt').exists() assert (app.outdir / 'output.txt').exists()
@ -40,7 +40,7 @@ def test_defaults(app, status, warning):
@pytest.mark.sphinx('linkcheck', testroot='linkcheck', freshenv=True) @pytest.mark.sphinx('linkcheck', testroot='linkcheck', freshenv=True)
def test_defaults_json(app, status, warning): def test_defaults_json(app):
app.builder.build_all() app.builder.build_all()
assert (app.outdir / 'output.json').exists() assert (app.outdir / 'output.json').exists()
@ -101,7 +101,7 @@ def test_defaults_json(app, status, warning):
'https://www.google.com/image2.png', 'https://www.google.com/image2.png',
'path/to/notfound'] 'path/to/notfound']
}) })
def test_anchors_ignored(app, status, warning): def test_anchors_ignored(app):
app.builder.build_all() app.builder.build_all()
assert (app.outdir / 'output.txt').exists() assert (app.outdir / 'output.txt').exists()
@ -111,7 +111,7 @@ def test_anchors_ignored(app, status, warning):
assert not content assert not content
@pytest.mark.sphinx('linkcheck', testroot='linkcheck-localserver', freshenv=True) @pytest.mark.sphinx('linkcheck', testroot='linkcheck-localserver', freshenv=True)
def test_raises_for_invalid_status(app, status, warning): def test_raises_for_invalid_status(app):
class InternalServerErrorHandler(http.server.BaseHTTPRequestHandler): class InternalServerErrorHandler(http.server.BaseHTTPRequestHandler):
def do_GET(self): def do_GET(self):
self.send_error(500, "Internal Server Error") self.send_error(500, "Internal Server Error")
@ -133,7 +133,7 @@ def test_raises_for_invalid_status(app, status, warning):
(r'.+google\.com.+', 'authinfo2'), (r'.+google\.com.+', 'authinfo2'),
] ]
}) })
def test_auth(app, status, warning): def test_auth(app):
mock_req = mock.MagicMock() mock_req = mock.MagicMock()
mock_req.return_value = 'fake-response' mock_req.return_value = 'fake-response'
@ -161,7 +161,7 @@ def test_auth(app, status, warning):
"X-Secret": "open sesami", "X-Secret": "open sesami",
} }
}}) }})
def test_linkcheck_request_headers(app, status, warning): def test_linkcheck_request_headers(app):
mock_req = mock.MagicMock() mock_req = mock.MagicMock()
mock_req.return_value = 'fake-response' mock_req.return_value = 'fake-response'