pytest-postgresql icon indicating copy to clipboard operation
pytest-postgresql copied to clipboard

support `\` in linux temporary path

Open bhelgs opened this issue 1 year ago • 2 comments

What action do you want to perform

By default pytest temporary paths on AWS workspaces will contain a \ in the temporary path due to their username conventions.

Maybe you might want to support \ in paths? Some additional command quotes seem fix the \ issue and may help with other special linux path characters.

What are the results

Temporary data is generated under /tmp/ACTIVEDIRECTORY\user.name/pytest-101/pytest-postgresql-postgresql_proc0/ but some commands lost the \

What are the expected results

I was able to run without issue after quoting the paths in PostgreSQLExecutor.BASE_PROC_START_COMMAND and PostgreSQLExecutor.running()

Workaround

set --basetemp= argument in pytest command

bhelgs avatar Jul 11 '24 23:07 bhelgs

Thank you very much for creating such a great project! 🏆 Please let me know if you would like for me to open a couple line PR.

bhelgs avatar Jul 11 '24 23:07 bhelgs

@bhelgs if you're able to probide PR with fix (and test preferably), please go ahead :)

fizyk avatar Jul 16 '24 10:07 fizyk

And we have a release.

fizyk avatar Sep 04 '24 08:09 fizyk