Commite436690

authored

Merge pull request#3584 from pre-commit/exitstack

use ExitStack instead of start + stop

2 parents9c7ea88 +8d34f95 commite436690Copy full SHA for e436690

File tree

-5

lines changed

-5

lines changed

Lines changed: 4 additions & 5 deletions

Original file line number	Diff line number	Diff line change
`@@ -1,6 +1,7 @@`
`1`	`1`	`from __future__importannotations`
`2`	`2`
`3`	`3`	`importargparse`
	`4`	`+importcontextlib`
`4`	`5`	`importos.path`
`5`	`6`	`fromunittestimportmock`
`6`	`7`
`@@ -97,11 +98,9 @@ def test_adjust_args_try_repo_repo_relative(in_git_dir):`
`97`	`98`
`98`	`99`	`@pytest.fixture`
`99`	`100`	`defmock_commands():`
`100`		`-mcks= {fn:mock.patch.object(main,fn).start()forfninFNS}`
`101`		`-ret=auto_namedtuple(**mcks)`
`102`		`-yieldret`
`103`		`-formckinret:`
`104`		`-mck.stop()`
	`101`	`+withcontextlib.ExitStack()asctx:`
	`102`	`+mcks= {f:ctx.enter_context(mock.patch.object(main,f))forfinFNS}`
	`103`	`+yieldauto_namedtuple(**mcks)`
`105`	`104`
`106`	`105`
`107`	`106`	`@pytest.fixture`

Comments

(0)