Commitd831fba

authored and

committed

Rules>PSAlignAssignmentStatement: Treat single kvp hashtables as being on a single line, and not checked for violations.

1 parentaa7a582 commitd831fbaCopy full SHA for d831fba

File tree

+32

-0

lines changed

+32

-0

lines changed

Lines changed: 5 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -314,6 +314,11 @@ private static List<Tuple<IScriptExtent, IScriptExtent>> GetExtents(`
`314`	`314`
`315`	`315`	`privateboolHasPropertiesOnSeparateLines(IEnumerable<Tuple<IScriptExtent,IScriptExtent>>tuples)`
`316`	`316`	`{`
	`317`	`+if(tuples.Count()==1)`
	`318`	`+{`
	`319`	`+// If the hashtable has just a single key-value pair, it does not have properties on separate lines`
	`320`	`+returnfalse;`
	`321`	`+}`
`317`	`322`	`varlines=newHashSet<int>();`
`318`	`323`	`foreach(varkvpintuples)`
`319`	`324`	`{`

Lines changed: 27 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -75,6 +75,33 @@ $x = @{ }`
`75`	`75`	`Invoke-ScriptAnalyzer-ScriptDefinition$def-Settings$settings\|Get-Count\| Should-Be0`
`76`	`76`
`77`	`77`	`}`
	`78`	`+`
	`79`	`+ It"Should ignore if a hashtable has a single key-value pair on a single line" {`
	`80`	`+$def=@'`
	`81`	`+$x = @{ 'key'="value" }`
	`82`	`+'@`
	`83`	`+Invoke-ScriptAnalyzer-ScriptDefinition$def-Settings$settings\|Get-Count\| Should-Be0`
	`84`	`+`
	`85`	`+ }`
	`86`	`+`
	`87`	`+ It"Should ignore if a hashtable has a single key-value pair across multiple lines" {`
	`88`	`+$def=@'`
	`89`	`+$x = @{`
	`90`	`+ 'key'="value"`
	`91`	`+}`
	`92`	`+'@`
	`93`	`+Invoke-ScriptAnalyzer-ScriptDefinition$def-Settings$settings\|Get-Count\| Should-Be0`
	`94`	`+`
	`95`	`+ }`
	`96`	`+`
	`97`	`+ It"Should ignore if a hashtable has multiple key-value pairs on a single line" {`
	`98`	`+$def=@'`
	`99`	`+$x = @{ 'key'="value"; 'key2'="value2"; 'key3WithLongerName'="value3" }`
	`100`	`+'@`
	`101`	`+Invoke-ScriptAnalyzer-ScriptDefinition$def-Settings$settings\|Get-Count\| Should-Be0`
	`102`	`+`
	`103`	`+ }`
	`104`	`+`
`78`	`105`	`}`
`79`	`106`
`80`	`107`	`Context"When assignment statements are in DSC Configuration" {`

Comments

(0)