BlockchainCommons / BlockchainCommons/bc-lifehash-python
Modes not working -> Extend unit test
- 主要言語
- Python
- スター
- 1
- フォーク
- 1
- PR マージ指標
- 30日以内にマージされた PR はありません
説明
Hy dear LifeHash developers,
nice project. Yet, using you code with python 3.13 sadly made some issues.
It seems you are expecting to have the "+" operator to be defined on a hashlib.sha256 as well as hashlib.sha256 on another hashlib.sha256 ( sha256(sha256(bytes)) ).
lifehash/lifehash.py line 1024 thus fails. I think you just need to add a few .digest() here and there.
I get why you did not see that. I strongly suggest to extend you unit tests to test *all* modes for you LifeHash. Currently you only test version2.
コントリビューションガイド
調査の方向性
lifehash/lifehash.py の 1024 行目から始め、Python 3.13 で hashlib.sha256 の値がどのように結合されるかを確認してください。既存のユニットテストを確認します。現在は version2 のみを対象としています。その後、他の LifeHash モードも確認しながらテストを実行してください。失敗しているハッシュ処理のパスが処理され、テストがすべてのモードをカバーすれば完了です。
索引モデルが issue の本文から書いたものです。
評価
- 技術スタック
- python
- 領域
- testing
- issue の種類
- バグ
- 難易度
- 3/5
- 見積もり時間
- 1〜2日
- 活発さ
- 停滞
- 明瞭さ
- おおむね明確
- 初心者へのやさしさ
- 48/100