Skip to content

#8516 fix causes other issues elsewhere #10757

@cjw296

Description

@cjw296

8516 includes a hard coded ~0o077:

https://github.com/pytest-dev/pytest/pull/8516/files#diff-10ee70cc91700b16983baadc71b08d4239920ea83c1041a06adbb4302ae80c5bR145

This is also arguably wrong in that it re-permissions a directory potentially not even created by the current text (think pytest-xdist, etc) and breaks some existing use cases for these fixtures...

In our case, we have tests that ssh to localhost as a different user to the one running the tests and then work inside a directory created inside the one created by the tmp_path fixtures. Since this change, those tests are broken and we can no longer use the tmp_path fixture as a result.

Please could this specific change be reverted or, alternatively, something added so that we can turn this off?

Metadata

Metadata

Assignees

No one assigned

    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