Thanks to visit codestin.com
Credit goes to github.com

Skip to content

The Subprocess.run() method can now override the cwd #35

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Sep 23, 2022

Conversation

s0undt3ch
Copy link
Contributor

Fixes #33

@s0undt3ch s0undt3ch requested a review from a team as a code owner September 23, 2022 06:25
@s0undt3ch s0undt3ch requested a review from cmcmarrow September 23, 2022 06:25
@codecov
Copy link

codecov bot commented Sep 23, 2022

Codecov Report

Merging #35 (6f41d5d) into main (09b1cc2) will increase coverage by 0.10%.
The diff coverage is 100.00%.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main      #35      +/-   ##
==========================================
+ Coverage   87.96%   88.05%   +0.10%     
==========================================
  Files          31       31              
  Lines        2407     2426      +19     
  Branches      355      355              
==========================================
+ Hits         2117     2136      +19     
  Misses        220      220              
  Partials       70       70              
Flag Coverage Δ
87.60% <100.00%> (+0.10%) ⬆️
src 83.21% <100.00%> (ø)
tests 96.41% <100.00%> (+0.09%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
src/pytestshellutils/downgraded/shell.py 84.12% <100.00%> (ø)
src/pytestshellutils/shell.py 88.68% <100.00%> (ø)
tests/functional/shell/test_fixture.py 100.00% <100.00%> (ø)
tests/functional/shell/test_subprocess.py 100.00% <100.00%> (ø)

@s0undt3ch s0undt3ch merged commit 258df8d into saltstack:main Sep 23, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Cannot pass a different cwd to shell.run()
2 participants