acoolname/neotest-python
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix(pytest): don't reverse namespaces

Browse Source 
See #26
This commit is contained in:
Rónán Carrigan
2022-10-31 14:14:52 +00:00
parent 2dc9c95fe9
commit ba7069a030
2 changed files with 7 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
neotest_python/base.py
View File

@@ -31,7 +31,6 @@ else:
class NeotestAdapter(abc.ABC): class NeotestAdapter(abc.ABC):
def update_result( def update_result(
self, base: Optional[NeotestResult], update: NeotestResult self, base: Optional[NeotestResult], update: NeotestResult
) -> NeotestResult: ) -> NeotestResult:

12
neotest_python/pytest.py
View File

@@ -10,7 +10,6 @@ if TYPE_CHECKING:
class PytestNeotestAdapter(NeotestAdapter): class PytestNeotestAdapter(NeotestAdapter):
def run( def run(
self, self,
args: List[str], args: List[str],
@@ -24,7 +23,6 @@ class PytestNeotestAdapter(NeotestAdapter):
class NeotestResultCollector: class NeotestResultCollector:
def __init__( def __init__(
self, self,
adapter: PytestNeotestAdapter, adapter: PytestNeotestAdapter,
@@ -36,7 +34,9 @@ class NeotestResultCollector:
self.pytest_config: "Config" = None # type: ignore self.pytest_config: "Config" = None # type: ignore
self.results: Dict[str, NeotestResult] = {} self.results: Dict[str, NeotestResult] = {}
def _get_short_output(self, config: "Config", report: "TestReport") -> Optional[str]: def _get_short_output(
self, config: "Config", report: "TestReport"
) -> Optional[str]:
from _pytest.terminal import TerminalReporter from _pytest.terminal import TerminalReporter
buffer = StringIO() buffer = StringIO()
@@ -62,9 +62,9 @@ class NeotestResultCollector:
for report in items: for report in items:
file_path, *name_path = report.nodeid.split("::") file_path, *name_path = report.nodeid.split("::")
abs_path = str(Path(self.pytest_config.rootdir, file_path)) abs_path = str(Path(self.pytest_config.rootdir, file_path))
test_name, *namespaces = reversed(name_path) *namespaces, test_name = name_path
valid_test_name, *params = test_name.split("[") # ] valid_test_name, *params = test_name.split("[") # ]
pos_id = "::".join([abs_path, *namespaces, valid_test_name]) pos_id = "::".join([abs_path, *(namespaces), valid_test_name])
result = self.adapter.update_result( result = self.adapter.update_result(
self.results.get(pos_id), self.results.get(pos_id),
{ {
@@ -88,7 +88,7 @@ class NeotestResultCollector:
file_path, *name_path = report.nodeid.split("::") file_path, *name_path = report.nodeid.split("::")
abs_path = str(Path(self.pytest_config.rootdir, file_path)) abs_path = str(Path(self.pytest_config.rootdir, file_path))
test_name, *namespaces = reversed(name_path) *namespaces, test_name = name_path
valid_test_name, *params = test_name.split("[") # ] valid_test_name, *params = test_name.split("[") # ]
pos_id = "::".join([abs_path, *namespaces, valid_test_name]) pos_id = "::".join([abs_path, *namespaces, valid_test_name])