BlockchainCommons / BlockchainCommons/bc-lifehash-python

Modes not working -> Extend unit test

オープン
#4 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る
主要言語
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

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。