Fix for Transfer Stake Overwrite

computablelabs / goest

Golang testing and sandbox for the Computable Protocol. 0xb00.

MIT License

8 stars 0 forks source link

Fix for Transfer Stake Overwrite #144

Closed rbharath closed 5 years ago

rbharath commented 5 years ago

Working towards fixing https://github.com/computablelabs/goest/issues/132

It looks like the existing tests don't check for this issue. Will add in a new test before this is ready for merge.

rbharath commented 5 years ago

This PR is now ready for review. It makes the fix to #132. It turns out that an existing test checks for the bad aggregation behavior noted in #132, but this code adds a second more straightforward test to check that double challenges work as expected wrt aggregation.