WIP: Adds Bugfixing kata #344
Conversation
indented-automation
requested review from
steviecoaster and
vexx32
as code owners
vexx32
added
Category-Koans
| function Debug-Me { | ||
| $process = Get—Process -Id $PID | ||
| if ($process -and $process.Name -eq 'powershell') { | ||
| Write-Host 'you must be using PowerShell' | ||
| } | ||
| } |
There was a problem hiding this comment.
This won't throw, will it? Might need to swap your Write-Host for a throw :)
There was a problem hiding this comment.
It will throw (as-is) until the bug is fixed :)
There was a problem hiding this comment.
Let me tweak it just a little, to make it less obvious.
There was a problem hiding this comment.
Even the new example here doesn't look like it'll throw? Pester's Should -Throw will expect a terminating error and nothing else... what here is going to throw? XD
Co-Authored-By: Joel Sallow (/u/ta11ow) <32407840+vexx32@users.noreply.github.com>
|
@indented-automation came across this in the Discord just now -- need one where misplacing the array index sometimes causes a bug. So, for example, sometimes something returns two objects with a string property each, sometimes just one, so an expression like |
PR Summary
Fixes #282
Context
A draft for bug fixing kata's. This is going to take a little time. Please add a WIP.
I would, however, appreciate any feedback as and when anyone feels like commenting :)
Changes
Adds Bug fixing kata
Checklist