keep-network / keep-core

The smart contracts and reference client behind the Keep network
https://keep.network
MIT License
118 stars 73 forks source link

Investigate failing `WalletRegistry - Wallets` test in ECDSA #3707

Open michalinacienciala opened 1 year ago

michalinacienciala commented 1 year ago

We've observed that sporadically the nightly ECDSA tests fail on the following assertion:

     WalletRegistry - Wallets
       isWalletMember
         when the given operator address is an actual sortition pool operator
           when the passed wallet members identifiers are valid
             when the wallet member index is in correct range
               when the given operator is not the member of the wallet signing group at the given position
                 should return false

An example of a failing job: https://github.com/keep-network/keep-core/actions/runs/7647915998/job/20839754741.

We should investigate if the problem is about how the test is written or about how our system behaves.

Dates of recent failures: 2023-09-06, 2024-01-25