Commit 703ff5d
committed
Address review feedback: refactor to use dictionary and parametrize tests
- Refactor get_sample_short_name_by_md5() to use dictionary lookup for O(1) performance
- Add _MD5_TO_SHORT_NAME module-level constant for efficient lookups
- Refactor tests to use pytest.mark.parametrize for better readability
- Split tests into separate functions for different concerns
- Fix CHANGELOG: use @kamranulhaq2002 instead of email
- Update CHANGELOG: use PR #2802 instead of #TBD
- Apply code formatting (isort, black, ruff)
Addresses feedback from Gemini Code Assist bot review.1 parent 280f020 commit 703ff5d
3 files changed
+814
-208
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
50 | 50 | | |
51 | 51 | | |
52 | 52 | | |
53 | | - | |
| 53 | + | |
54 | 54 | | |
55 | 55 | | |
56 | 56 | | |
| |||
0 commit comments