fix: Start-Htmx ( Fixes #7 )

Registering for PowerShellExiting event and simplifying test
PowerShellWeb · Oct 25, 2024 · fd1c35a · fd1c35a
1 parent a477298
commit fd1c35a
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 15 deletions.
diff --git a/Commands/Start-Htmx.ps1 b/Commands/Start-Htmx.ps1
@@ -477,6 +477,9 @@ function Start-Htmx {
             )
         }
 
+        $null = Register-EngineEvent -SourceIdentifier PowerShell.Exiting -Action {
+            Get-Job | ? HttpListener | Stop-Htmx
+        }
         while ($serverJob.JobStateInfo.State -eq 'NotStarted') {
             Start-Sleep -Milliseconds (Get-Random -Maximum 20 -Minimum 1)
         }

diff --git a/HtmxPS.tests.ps1 b/HtmxPS.tests.ps1
@@ -12,21 +12,12 @@ describe HtmxPS {
     }
 
     context 'Start-Htmx and Stop-Htmx' {
-        it "Will start and stop a small htmx server" {
-            if (-not $env:GITHUB_WORKSPACE) {
-                $startedLocalJob = Start-Htmx -Htmx (
-                    htmx button "Click Me" hx-on:click="alert('Thanks, I needed that!')"
-                )
-                Invoke-RestMethod -Uri $startedLocalJob.ServerUrl
-                $startedLocalJob | Stop-Htmx -PassThru | Remove-Job                 
-            } else {
-                $startedLocalJob = Start-Htmx -Htmx (
-                    htmx button "Click Me" hx-on:click="alert('Thanks, I needed that!')"
-                ) -ServerUrl "http://127.0.0.1:8080/"
-                # Start-Sleep -Seconds 1
-                Invoke-RestMethod -Uri $startedLocalJob.ServerUrl
-                $startedLocalJob | Stop-Htmx -PassThru | Remove-Job                
-            }             
+        it "Will start and stop a small htmx server" {            
+            $startedLocalJob = Start-Htmx -Htmx (
+                htmx button "Click Me" hx-on:click="alert('Thanks, I needed that!')"
+            )
+            Invoke-RestMethod -Uri $startedLocalJob.ServerUrl
+            $startedLocalJob | Stop-Htmx -PassThru | Remove-Job                         
         }
     }
 }