Skip to content

Modernize pytest temporary directory handling #698

@ethanholz

Description

@ethanholz

Is your feature request related to a problem? Please describe.

pytest migrated from using tmp_dir to tmp_path in more recent versions of pytest (for more info see here). I adopted this in the work done in #690. We should try and make it more uniform for the rest of the tests.

Describe the solution you'd like

This should be fairly easy to swap, you can run pytest -p no:legacypath which will trigger an error on outdated convention. See here for more info.

Describe alternatives you've considered

We can keep it the same, but it would be nice to make things more uniform.

Metadata

Metadata

Assignees

Labels

No labels
No labels

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions