Fix #5086 - Invalid JSON in the snippets file (#5087)

* Fixed JSON

* Added test for making sure all JSON files are valid JSON

* Run in correct dir

* Simplify title of the new step

Co-authored-by: Andy Jordan <[email protected]>

* Fix path double nesting

Co-authored-by: Andy Jordan <[email protected]>

* Add spaces after commas

Co-authored-by: Andy Jordan <[email protected]>

* Use $PWD instead of '.\' for current dir 🐧

* Only test the snippets JSON file

* Oops, typo

---------

Co-authored-by: Andy Jordan <[email protected]>

Author: o-l-a-v

.github/workflows/ci-test.yml

@@ -2,19 +2,19 @@ name: CI Tests
 
 on:
   push:
-    branches: [ main ]
+    branches: [main]
   pull_request:
     # The branches below must be a subset of the branches above
-    branches: [ main ]
+    branches: [main]
   merge_group:
-    types: [ checks_requested ]
+    types: [checks_requested]
 
 jobs:
   ci:
     name: node
     strategy:
       matrix:
-        os: [ windows-latest, macos-latest, ubuntu-latest ]
+        os: [windows-latest, macos-latest, ubuntu-latest]
     runs-on: ${{ matrix.os }}
     env:
       DOTNET_NOLOGO: true
@@ -33,6 +33,11 @@ jobs:
         with:
           path: vscode-powershell
 
+      - name: Validate snippets JSON file
+        shell: pwsh
+        run: $null = ConvertFrom-Json -InputObject (Get-Content -Raw -Path './snippets/PowerShell.json')
+        working-directory: vscode-powershell
+
       - name: Install dotnet
         uses: actions/setup-dotnet@v4
         with:

snippets/PowerShell.json

@@ -162,24 +162,23 @@
     ]
   },
   "Foreach with Progress": {
-		"prefix": "foreach-progress",
+    "prefix": "foreach-progress",
     "description": "Insert a foreach loop with Write-Progress initialized",
-		"body": [
-\\$progPercent = \"{0:n2}\" -f ([math]::round(\\$i/\\$array.count,4) * 100)",
-
-Write-Progress -Activity \"${3:activityName}\" -Status \"\\$i of \\$array.count - \\$progPercent% Complete:\" -PercentComplete \\$progPercent",
-		  "\\$i = 1",
-		  "foreach ($${2:item} in $${1:array}) {",
-		  "    \\$progPercent = \"{0:n2}\" -f ([math]::round(\\$i/\\$total,4) * 100)",
-		  "    Write-Progress -Activity \"${3:activityName}\" -Status \"\\$i of \\$total - \\$progPercent% Complete:\" -PercentComplete \\$progPercent",
-		  "    # Insert Code Here",
-		  "    ${0}",
-		  "    ",
-		  "    \\$i++",
-		  "}",
-		  ""
-		]
-	},
+    "body": [
+      "\\$progPercent = \"{0:n2}\" -f ([math]::round(\\$i/\\$array.count,4) * 100)",
+      "Write-Progress -Activity \"${3:activityName}\" -Status \"\\$i of \\$array.count - \\$progPercent% Complete:\" -PercentComplete \\$progPercent",
+      "\\$i = 1",
+      "foreach ($${2:item} in $${1:array}) {",
+      "    \\$progPercent = \"{0:n2}\" -f ([math]::round(\\$i/\\$total,4) * 100)",
+      "    Write-Progress -Activity \"${3:activityName}\" -Status \"\\$i of \\$total - \\$progPercent% Complete:\" -PercentComplete \\$progPercent",
+      "    # Insert Code Here",
+      "    ${0}",
+      "    ",
+      "    \\$i++",
+      "}",
+      ""
+    ]
+  },
   "ForEach-Object -Parallel": {
     "prefix": "foreach-parallel",
     "description": "[PS 7+] Process multiple objects in parallel using runspaces. This has some limitations compared to a regular ForEach-Object. More: Get-Help ForEach-Object",