Commit d78f84d

committed

refactor: pytest_ignore_collect to return False, which after needle match

1 parent 65c1b43 commit d78f84dCopy full SHA for d78f84d

File tree

-7

lines changed

-7

lines changed

Lines changed: 8 additions & 7 deletions

Original file line number	Diff line number	Diff line change
`@@ -106,16 +106,17 @@ def __next__(self) -> str:`
`106`	`106`
`107`	`107`	`def pytest_ignore_collect(collection_path: pathlib.Path, config: pytest.Config) -> bool:`
`108`	`108`	`"""Skip tests if VCS binaries are missing."""`
`109`		`- if notshutil.which("svn") andany(`
	`109`	`+ if any(`
`110`	`110`	`needle in str(collection_path) for needle in ["svn", "subversion"]`
`111`		`- ):`
	`111`	`+ )andnotshutil.which("svn"):`
`112`	`112`	`return True`
`113`		`- if notshutil.which("git") and "git"instr(collection_path):`
	`113`	`+ if "git"instr(collection_path) and notshutil.which("git"):`
`114`	`114`	`return True`
`115`		`- return bool(`
`116`		`- not shutil.which("hg")`
`117`		`- and any(needle in str(collection_path) for needle in ["hg", "mercurial"]),`
`118`		`- )`
	`115`	`+ if any( # NOQA: SIM103`
	`116`	`+ needle in str(collection_path) for needle in ["hg", "mercurial"]`
	`117`	`+ ) and not shutil.which("hg"):`
	`118`	`+ return True`
	`119`	`+ return False`
`119`	`120`
`120`	`121`
`121`	`122`	`@pytest.fixture(scope="session")`

Comments

(0)