Fix CompletesAttributeValue Test

JustinGrote · JustinGrote · commit 127b083830e0 · 2025-03-01T21:52:30.000-08:00
diff --git a/test/PowerShellEditorServices.Test.Shared/Completion/CompletionExamples.psm1 b/test/PowerShellEditorServices.Test.Shared/Completion/CompletionExamples.psm1
@@ -13,7 +13,7 @@ Import-Module PowerShellGet
 Get-Rand
 
 function Test-Completion {
-    param([Parameter(Mandatory, Value)])
+    param([Parameter(Mandatory, Value)]$test)
 }
 
 Get-ChildItem /
diff --git a/test/PowerShellEditorServices.Test/Language/CompletionHandlerTests.cs b/test/PowerShellEditorServices.Test/Language/CompletionHandlerTests.cs
@@ -105,7 +105,6 @@ public async Task CompletesVariableInFile()
         [SkippableFact]
         public async Task CompletesAttributeValue()
         {
-            Skip.If(VersionUtils.IsPS74, "PowerShell 7.4 isn't returning these!");
             (_, IEnumerable<CompletionItem> results) = await GetCompletionResultsAsync(CompleteAttributeValue.SourceDetails);
             // NOTE: Since the completions come through un-ordered from PowerShell, their SortText
             // (which has an index prepended from the original order) will mis-match our assumed

Original file line number	Diff line number	Diff line change
`@@ -13,7 +13,7 @@ Import-Module PowerShellGet`
`13`	`13`	`Get-Rand`
`14`	`14`
`15`	`15`	`function Test-Completion {`
`16`		`- param([Parameter(Mandatory, Value)])`
	`16`	`+ param([Parameter(Mandatory, Value)]$test)`
`17`	`17`	`}`
`18`	`18`
`19`	`19`	`Get-ChildItem /`
Original file line number	Diff line number	Diff line change
`@@ -105,7 +105,6 @@ public async Task CompletesVariableInFile()`
`105`	`105`	`[SkippableFact]`
`106`	`106`	`public async Task CompletesAttributeValue()`
`107`	`107`	`{`
`108`		`- Skip.If(VersionUtils.IsPS74, "PowerShell 7.4 isn't returning these!");`
`109`	`108`	`(_, IEnumerable<CompletionItem> results) = await GetCompletionResultsAsync(CompleteAttributeValue.SourceDetails);`
`110`	`109`	`// NOTE: Since the completions come through un-ordered from PowerShell, their SortText`
`111`	`110`	`// (which has an index prepended from the original order) will mis-match our assumed`