fix poetry shell for zsh if space character in venv path

[dimstunt]

fix zsh: poetry shell doesn't work if space character in .venv path

Pull Request Check List

Resolves: #7244

Confirmed working on bash, zsh

[PabloEmidio]

On PEP 8, there is the following say about triple quote

For triple-quoted strings, always use double quote characters to be consistent with the docstring convention in PEP 257.

[jaklan]

Or even simpler:
c.sendline(f'emulate bash -c ". {shlex.quote(str(activate_path))}"')
(so we get emulate bash -c ". '/some path/activate'")

But the safest one would be:
c.sendline(f"emulate bash -c {shlex.quote(f'. {shlex.quote(str(activate_path))}')}")
(so we get emulate bash -c '. '"'"'/some path/activate'"'"'')

[DennisTheMenace780]

Any updates on whether this will get merged in the near future? I am running into the same issue on macOS unfortuantely.

[dimstunt]

@PabloEmidio when you have a chance, please check this.

[PabloEmidio]

@PabloEmidio when you have a chance, please check this.

For what I saw, it should do it. Could any test be added to validate this scenary?

[dimstunt]

@PabloEmidio when you have a chance, please check this.

For what I saw, it should do it. Could any test be added to validate this scenary?

Not fast because the executing bash script is moked, and I don't have enough expertise in pytest to change that.

[radoering]

pre-commit.ci autofix

[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.