Merge pull request #383 from pytest-dev/update-pytests

Update pytests

Author: youtux

.pre-commit-config.yaml

@@ -2,11 +2,11 @@
 # See https://pre-commit.com/hooks.html for more hooks
 repos:
 -   repo: https://github.com/psf/black
-    rev: 19.3b0
+    rev: 20.8b1
     hooks:
     - id: black
 -   repo: https://github.com/pre-commit/pre-commit-hooks
-    rev: v2.3.0
+    rev: v3.2.0
     hooks:
     -   id: trailing-whitespace
     -   id: end-of-file-fixer

pytest_bdd/gherkin_terminal_reporter.py

@@ -51,11 +51,6 @@ class GherkinTerminalReporter(TerminalReporter):
     def __init__(self, config):
         TerminalReporter.__init__(self, config)
 
-    def pytest_runtest_logstart(self, nodeid, location):
-        # Prevent locationline from being printed since we already
-        # show the module_name & in verbose mode the test name.
-        pass
-
     def pytest_runtest_logreport(self, report):
         rep = report
         res = self.config.hook.pytest_report_teststatus(report=rep, config=self.config)

pytest_bdd/scenario.py

@@ -198,7 +198,12 @@ def scenario_wrapper(request):
 
 
 def scenario(
-    feature_name, scenario_name, encoding="utf-8", example_converters=None, caller_module=None, features_base_dir=None,
+    feature_name,
+    scenario_name,
+    encoding="utf-8",
+    example_converters=None,
+    caller_module=None,
+    features_base_dir=None,
 ):
     """Scenario decorator.
 

requirements-testing.txt

@@ -1 +1 @@
-mock
+packaging

tests/feature/test_cucumber_json.py

@@ -155,7 +155,7 @@ def test_passing_outline():
                             "line": 12,
                             "match": {"location": ""},
                             "name": "a failing step",
-                            "result": {"error_message": OfType(string), "status": "failed", "duration": OfType(int),},
+                            "result": {"error_message": OfType(string), "status": "failed", "duration": OfType(int)},
                         },
                     ],
                     "tags": [{"name": "scenario-failing-tag", "line": 9}],

tests/feature/test_outline.py

@@ -1,6 +1,7 @@
 """Scenario Outline tests."""
 import textwrap
 
+from tests.utils import assert_outcomes
 
 STEPS = """\
 from pytest_bdd import given, when, then
@@ -111,11 +112,11 @@ def test_outline(request):
         )
     )
     result = testdir.runpytest()
-    result.assert_outcomes(error=1)
+    assert_outcomes(result, errors=1)
     result.stdout.fnmatch_lines(
         '*ScenarioExamplesNotValidError: Scenario "Outlined with wrong examples"*has not valid examples*',
     )
-    result.stdout.fnmatch_lines("*should match set of example values [[]'eat', 'left', 'start', 'unknown_param'[]].*",)
+    result.stdout.fnmatch_lines("*should match set of example values [[]'eat', 'left', 'start', 'unknown_param'[]].*")
 
 
 def test_wrong_vertical_examples_scenario(testdir):
@@ -151,7 +152,7 @@ def test_outline(request):
         )
     )
     result = testdir.runpytest()
-    result.assert_outcomes(error=1)
+    assert_outcomes(result, errors=1)
     result.stdout.fnmatch_lines(
         "*Scenario has not valid examples. Example rows should contain unique parameters. "
         '"start" appeared more than once.*'
@@ -192,7 +193,7 @@ def test_outline(request):
         )
     )
     result = testdir.runpytest()
-    result.assert_outcomes(error=1)
+    assert_outcomes(result, errors=1)
     result.stdout.fnmatch_lines(
         "*Feature has not valid examples. Example rows should contain unique parameters. "
         '"start" appeared more than once.*'

tests/feature/test_scenario.py

@@ -2,6 +2,8 @@
 
 import textwrap
 
+from tests.utils import assert_outcomes
+
 
 def test_scenario_not_found(testdir):
     """Test the situation when scenario is not found."""
@@ -30,7 +32,7 @@ def test_not_found():
     )
     result = testdir.runpytest()
 
-    result.assert_outcomes(error=1)
+    assert_outcomes(result, errors=1)
     result.stdout.fnmatch_lines('*Scenario "NOT FOUND" in feature "Scenario is not found" in*')
 
 

tests/feature/test_wrong.py

@@ -2,6 +2,8 @@
 
 import textwrap
 
+from tests.utils import assert_outcomes
+
 
 def test_multiple_features_single_file(testdir):
     """Test validation error when multiple features are placed in a single file."""
@@ -49,5 +51,5 @@ def test_wrong():
         )
     )
     result = testdir.runpytest()
-    result.assert_outcomes(error=1)
+    assert_outcomes(result, errors=1)
     result.stdout.fnmatch_lines("*FeatureError: Multiple features are not allowed in a single feature file.*")

tests/generation/test_generate_missing.py

@@ -3,6 +3,7 @@
 import textwrap
 
 from pytest_bdd.scenario import get_python_name_generator
+from tests.utils import assert_outcomes
 
 
 def test_python_name_generator():
@@ -60,7 +61,7 @@ def test_missing_steps():
     )
 
     result = testdir.runpytest("--generate-missing", "--feature", "generation.feature")
-    result.assert_outcomes(passed=0, failed=0, error=0)
+    assert_outcomes(result, passed=0, failed=0, errors=0)
     assert not result.stderr.str()
     assert result.ret == 0
 

tests/steps/test_steps.py

@@ -42,7 +42,8 @@ def test_when_then(request):
 
 
 @pytest.mark.parametrize(
-    ("step", "keyword"), [("given", "Given"), ("when", "When"), ("then", "Then")],
+    ("step", "keyword"),
+    [("given", "Given"), ("when", "When"), ("then", "Then")],
 )
 def test_preserve_decorator(testdir, step, keyword):
     """Check that we preserve original function attributes after decorating it."""