Commite40fc2c

committed

Use TemporaryFile as a context manager

(This also uses assertRaises as a context manager, but that isjust for improved clarity.)

1 parentafa5754 commite40fc2cCopy full SHA for e40fc2c

File tree

-7

lines changed

-7

lines changed

Lines changed: 3 additions & 7 deletions

Original file line number	Diff line number	Diff line change
`@@ -260,13 +260,9 @@ def test_it_ignores_false_kwargs(self, git):`
`260`	`260`	`self.assertTrue("pass_this_kwarg"notingit.call_args[1])`
`261`	`261`
`262`	`262`	`deftest_it_raises_proper_exception_with_output_stream(self):`
`263`		`-tmp_file=TemporaryFile()`
`264`		`-self.assertRaises(`
`265`		`-GitCommandError,`
`266`		`-self.git.checkout,`
`267`		`-"non-existent-branch",`
`268`		`-output_stream=tmp_file,`
`269`		`- )`
	`263`	`+withTemporaryFile()astmp_file:`
	`264`	`+withself.assertRaises(GitCommandError):`
	`265`	`+self.git.checkout("non-existent-branch",output_stream=tmp_file)`
`270`	`266`
`271`	`267`	`deftest_it_accepts_environment_variables(self):`
`272`	`268`	`filename=fixture_path("ls_tree_empty")`

Comments

(0)