fix pytest-dev#4649 - also transfer markers to keywordmapping

as it turns out it is distinct from nodekeywords and behaves completely different

src/_pytest/mark/legacy.py

@@ -45,13 +45,14 @@ def from_item(cls, item):
                 mapped_names.add(item.name)
 
         # Add the names added as extra keywords to current or parent items
-        for name in item.listextrakeywords():
-            mapped_names.add(name)
+        mapped_names.update(item.listextrakeywords())
 
         # Add the names attached to the current function through direct assignment
         if hasattr(item, "function"):
-            for name in item.function.__dict__:
-                mapped_names.add(name)
+            mapped_names.update(item.function.__dict__)
+
+        # add the markers to the keywords as we no longer handle them correctly
+        mapped_names.update(mark.name for mark in item.iter_markers())
 
         return cls(mapped_names)
 

testing/example_scripts/marks/marks_considered_keywords/test_marks_as_keywords.py

@@ -0,0 +1,6 @@
+import pytest
+
+
+@pytest.mark.foo
+def test_mark():
+    pass