Commit 4aeaf56
committed
Address review feedback: improve test coverage for all mnemonicPrefix values
Changes per reviewer (Byron) and Copilot suggestions:
- Add h/ (HEAD) to the comment listing supported prefixes
- Update test docstring to include h/ prefix
- Expand test to cover all prefix combinations using subTest:
- c/ (commit) vs w/ (worktree)
- c/ (commit) vs i/ (index)
- i/ (index) vs w/ (worktree)
- o/ (object) vs w/ (worktree)
- h/ (HEAD) vs i/ (index)
- h/ (HEAD) vs w/ (worktree)
This ensures the regex pattern [abciwoh] and decode_path() work correctly
with all supported mnemonicPrefix values.1 parent 6e77e26 commit 4aeaf56
2 files changed
+34
-20
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
371 | 371 | | |
372 | 372 | | |
373 | 373 | | |
374 | | - | |
| 374 | + | |
375 | 375 | | |
376 | 376 | | |
377 | 377 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
290 | 290 | | |
291 | 291 | | |
292 | 292 | | |
| 293 | + | |
293 | 294 | | |
294 | 295 | | |
295 | 296 | | |
296 | | - | |
297 | | - | |
298 | | - | |
299 | | - | |
300 | | - | |
301 | | - | |
302 | | - | |
303 | | - | |
304 | | - | |
305 | | - | |
306 | | - | |
307 | | - | |
308 | | - | |
309 | | - | |
310 | | - | |
311 | | - | |
312 | | - | |
313 | | - | |
314 | | - | |
| 297 | + | |
| 298 | + | |
| 299 | + | |
| 300 | + | |
| 301 | + | |
| 302 | + | |
| 303 | + | |
| 304 | + | |
| 305 | + | |
| 306 | + | |
| 307 | + | |
| 308 | + | <
C0D0
/tr>|
| 309 | + | |
| 310 | + | |
| 311 | + | |
| 312 | + | |
| 313 | + | |
| 314 | + | |
| 315 | + | |
| 316 | + | |
| 317 | + | |
| 318 | + | |
| 319 | + | |
| 320 | + | |
| 321 | + | |
| 322 | + | |
| 323 | + | |
| 324 | + | |
| 325 | + | |
| 326 | + | |
| 327 | + | |
| 328 | + | |
315 | 329 | | |
316 | 330 | | |
317 | 331 | | |
| |||
0 commit comments