Skip to content

Commit

Permalink
Remove config paramter from pytest_terminal_summary as discussed duri…
Browse files Browse the repository at this point in the history
…ng review
  • Loading branch information
nicoddemus committed Feb 27, 2018
1 parent 4e405dd commit 94050a8
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 3 deletions.
3 changes: 1 addition & 2 deletions _pytest/hookspec.py
Original file line number Diff line number Diff line change
Expand Up @@ -489,10 +489,9 @@ def pytest_report_teststatus(report):
Stops at first non-None result, see :ref:`firstresult` """


def pytest_terminal_summary(config, terminalreporter, exitstatus):
def pytest_terminal_summary(terminalreporter, exitstatus):
"""Add a section to terminal summary reporting.
:param _pytest.config.Config config: pytest config object
:param _pytest.terminal.TerminalReporter terminalreporter: the internal terminal reporter object
:param int exitstatus: the exit status that will be reported back to the OS
Expand Down
1 change: 0 additions & 1 deletion _pytest/terminal.py
Original file line number Diff line number Diff line change
Expand Up @@ -480,7 +480,6 @@ def pytest_sessionfinish(self, exitstatus):
EXIT_NOTESTSCOLLECTED)
if exitstatus in summary_exit_codes:
self.config.hook.pytest_terminal_summary(terminalreporter=self,
config=self.config,
exitstatus=exitstatus)
if exitstatus == EXIT_INTERRUPTED:
self._report_keyboardinterrupt()
Expand Down

0 comments on commit 94050a8

Please sign in to comment.