Commita8591a0

authored and

committed

Exclude kwarg when None

1 parentb3d9b8d commita8591a0Copy full SHA for a8591a0

File tree

-3

lines changed

-3

lines changed

Lines changed: 2 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -885,15 +885,15 @@ def transform_kwarg(self, name, value, split_single_char_options):`
`885`	`885`	`iflen(name)==1:`
`886`	`886`	`ifvalueisTrue:`
`887`	`887`	`return ["-%s"%name]`
`888`		`-eliftype(value)isnotbool:`
	`888`	`+elifvaluenotin (False,None):`
`889`	`889`	`ifsplit_single_char_options:`
`890`	`890`	`return ["-%s"%name,"%s"%value]`
`891`	`891`	`else:`
`892`	`892`	`return ["-%s%s"% (name,value)]`
`893`	`893`	`else:`
`894`	`894`	`ifvalueisTrue:`
`895`	`895`	`return ["--%s"%dashify(name)]`
`896`		`-eliftype(value)isnotbool:`
	`896`	`+elifvaluenotin (False,None):`
`897`	`897`	`return ["--%s=%s"% (dashify(name),value)]`
`898`	`898`	`return []`
`899`	`899`

Lines changed: 3 additions & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -82,13 +82,15 @@ def test_it_raises_errors(self):`
`82`	`82`	`deftest_it_transforms_kwargs_into_git_command_arguments(self):`
`83`	`83`	`assert_equal(["-s"],self.git.transform_kwargs(**{'s':True}))`
`84`	`84`	`assert_equal(["-s","5"],self.git.transform_kwargs(**{'s':5}))`
	`85`	`+assert_equal([],self.git.transform_kwargs(**{'s':None}))`
`85`	`86`
`86`	`87`	`assert_equal(["--max-count"],self.git.transform_kwargs(**{'max_count':True}))`
`87`	`88`	`assert_equal(["--max-count=5"],self.git.transform_kwargs(**{'max_count':5}))`
	`89`	`+assert_equal([],self.git.transform_kwargs(**{'max_count':None}))`
`88`	`90`
`89`	`91`	`# Multiple args are supported by using lists/tuples`
`90`	`92`	`assert_equal(["-L","1-3","-L","12-18"],self.git.transform_kwargs(**{'L': ('1-3','12-18')}))`
`91`		`-assert_equal(["-C","-C"],self.git.transform_kwargs(**{'C': [True,True]}))`
	`93`	`+assert_equal(["-C","-C"],self.git.transform_kwargs(**{'C': [True,True,None,False]}))`
`92`	`94`
`93`	`95`	`# order is undefined`
`94`	`96`	`res=self.git.transform_kwargs(**{'s':True,'t':True})`

Comments

(0)