Removing tmp_dir test fixture in favor of pytest's tmp_path.

[upsideon]

Pull Request Check List

Resolves: #6934

These changes remove the custom tmp_dir test fixture in favor of pytest's built-in tmp_path fixture for creating temporary directories. Only test code has been updated, so there was no need to add additional tests or documentation.

• Added tests for changed code.
• Updated documentation for changed code.

[upsideon]

It looks like the Windows builds are failing due to a failure in cloning this repository as you can see in the build log here. I don't believe this is caused by the changes in this pull request as it seems unrelated. Has this been seen before?

[radoering]

It seems to be caused by the changes. See

poetry/tests/conftest.py

Lines 295 to 301 in 0ca8b7e

	@pytest.fixture
	def tmp_dir() -> Iterator[str]:
	dir_ = tempfile.mkdtemp(prefix="poetry_")
	yield Path(dir_).resolve().as_posix()
	remove_directory(dir_, force=True)

and

poetry/src/poetry/utils/helpers.py

Lines 67 to 76 in 0ca8b7e

	def remove_directory(
	path: Path | str, *args: Any, force: bool = False, **kwargs: Any
	) -> None:
	"""
	Helper function handle safe removal, and optionally forces stubborn file removal.
	This is particularly useful when dist files are read-only or git writes read-only
	files on Windows.
	Internally, all arguments are passed to `shutil.rmtree`.
	"""

[radoering]

Closing in favor of #7412

[github-actions]

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.