Compare commits
4 Commits
master
...
231908da78
| Author | SHA1 | Date | |
|---|---|---|---|
|
|
231908da78 | ||
|
|
72223525c4 | ||
|
|
ab80f41d3f | ||
|
|
9f6fbd6e04 |
@@ -139,7 +139,7 @@ end
|
||||
---@param positions neotest.Tree
|
||||
---@param root string
|
||||
local function discover_params(python, script, path, positions, root)
|
||||
local cmd = vim.iter({ python, script, "--pytest-collect", path }):flatten():totable()
|
||||
local cmd = vim.tbl_flatten({ python, script, "--pytest-collect", path })
|
||||
logger.debug("Running test instance discovery:", cmd)
|
||||
|
||||
local test_params = {}
|
||||
|
||||
@@ -1,7 +1,9 @@
|
||||
import inspect
|
||||
import os
|
||||
import subprocess
|
||||
import sys
|
||||
import traceback
|
||||
import unittest
|
||||
from argparse import ArgumentParser
|
||||
from pathlib import Path
|
||||
from types import TracebackType
|
||||
@@ -31,26 +33,12 @@ class CaseUtilsMixin:
|
||||
|
||||
|
||||
class DjangoNeotestAdapter(CaseUtilsMixin, NeotestAdapter):
|
||||
def get_django_root(self, path: str) -> Path:
|
||||
"""
|
||||
Traverse the file system to locate the nearest manage.py parent
|
||||
from the location of a given path.
|
||||
|
||||
This is the location of the django project
|
||||
"""
|
||||
test_file_path = Path(path).resolve()
|
||||
for parent in [test_file_path] + list(test_file_path.parents):
|
||||
if (parent / "manage.py").exists():
|
||||
return parent
|
||||
raise FileNotFoundError("manage.py not found")
|
||||
|
||||
def convert_args(self, case_id: str, args: List[str]) -> List[str]:
|
||||
"""Converts a neotest ID into test specifier for unittest"""
|
||||
path, *child_ids = case_id.split("::")
|
||||
if not child_ids:
|
||||
child_ids = []
|
||||
django_root = self.get_django_root(path)
|
||||
relative_file = os.path.relpath(path, django_root)
|
||||
relative_file = os.path.relpath(path, os.getcwd())
|
||||
relative_stem = os.path.splitext(relative_file)[0]
|
||||
relative_dotted = relative_stem.replace(os.sep, ".")
|
||||
return [*args, ".".join([relative_dotted, *child_ids])]
|
||||
@@ -129,16 +117,10 @@ class DjangoNeotestAdapter(CaseUtilsMixin, NeotestAdapter):
|
||||
+ len(suite_results.unexpectedSuccesses)
|
||||
)
|
||||
|
||||
# Add the location of the django project to system path
|
||||
# to ensure we have the same import paths as if the tests were ran
|
||||
# by manage.py
|
||||
case_id = args[-1]
|
||||
path, *_ = case_id.split("::")
|
||||
manage_py_location = self.get_django_root(path)
|
||||
sys.path.insert(0, str(manage_py_location))
|
||||
|
||||
# Make sure we can import relative to current path
|
||||
sys.path.insert(0, os.getcwd())
|
||||
# Prepend an executable name which is just used in output
|
||||
argv = ["neotest-python"] + self.convert_args(case_id, args[:-1])
|
||||
argv = ["neotest-python"] + self.convert_args(args[-1], args[:-1])
|
||||
# parse args
|
||||
parser = ArgumentParser()
|
||||
DjangoUnittestRunner.add_arguments(parser)
|
||||
|
||||
@@ -1,7 +1,6 @@
|
||||
from io import StringIO
|
||||
import json
|
||||
from pathlib import Path
|
||||
import re
|
||||
from typing import Callable, Dict, List, Optional, Union
|
||||
from . import params_getter
|
||||
|
||||
@@ -12,7 +11,6 @@ from _pytest.fixtures import FixtureLookupErrorRepr
|
||||
|
||||
from .base import NeotestAdapter, NeotestError, NeotestResult, NeotestResultStatus
|
||||
|
||||
ANSI_ESCAPE = re.compile(r'\x1B(?:[@-Z\\-_]|\[[0-?]*[ -/]*[@-~])')
|
||||
|
||||
class PytestNeotestAdapter(NeotestAdapter):
|
||||
def __init__(self, emit_parameterized_ids: bool):
|
||||
@@ -134,7 +132,7 @@ class NeotestResultCollector:
|
||||
errors.append({"message": msg_prefix + exc_repr, "line": None})
|
||||
# Test failed internally
|
||||
elif isinstance(exc_repr, ExceptionRepr):
|
||||
error_message = ANSI_ESCAPE.sub('', exc_repr.reprcrash.message) # type: ignore
|
||||
error_message = exc_repr.reprcrash.message # type: ignore
|
||||
error_line = None
|
||||
for traceback_entry in reversed(call.excinfo.traceback):
|
||||
if str(traceback_entry.path) == abs_path:
|
||||
|
||||
Reference in New Issue
Block a user