New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

minimal stateful keygen happy path #1

Merged

ggutoski merged 53 commits into master from abstract-stateful

Jan 12, 2021

Contributor

ggutoski commented Dec 31, 2020

No description provided.

ggutoski added 13 commits

December 28, 2020 12:10


          rename protocols -> protocol

b69c9b8


          move keygen tests to a separate file

df70e4d


          WIP: object-oriented design for keygen round 1

cf31503


          WIP: stateful keygen round 2

aeef238


          move stateless keygen into its own module

1dd7f7b


          test stubs

3fa627e


          make generic ID type for Protocol, State

576370e


          begin mock test suite

708833e


          move mock to tests/ subfolders, mark as abandoned

b02241f


          finish minimal stateful keygen happy path


          add test that everyone got the right pubkey

9ae2013


          WIP: begin test for stateful keygen happy path, add get_id() method t…

0cd25fd

…o State trait


          minimal passing test for stateful keygen happy path

2d7b9e8

ggutoski requested a review from sergeynog

December 31, 2020 16:52

ggutoski self-assigned this

sergeynog reviewed

View reviewed changes

src/protocol.rs Outdated Show resolved Hide resolved

sergeynog reviewed

View reviewed changes

src/protocol/gg20/keygen.rs Outdated Show resolved Hide resolved

sergeynog reviewed

View reviewed changes

src/protocol/gg20/keygen.rs Outdated Show resolved Hide resolved

sergeynog reviewed

View reviewed changes

src/protocol.rs Outdated Show resolved Hide resolved

sergeynog reviewed

View reviewed changes

src/protocol/gg20/keygen/tests.rs Outdated Show resolved Hide resolved

sergeynog reviewed

View reviewed changes

sergeynog left a comment

a few comments.

fish-sammy reviewed

View reviewed changes

src/protocol.rs Outdated Show resolved Hide resolved

src/protocol.rs Outdated Show resolved Hide resolved

ggutoski added 2 commits

January 4, 2021 16:54


          replace generic ID type with concrete String

ae5905e


          satisfy clippy

4e77459

ggutoski mentioned this pull request

error handling for add_message_in #2

Closed

ggutoski and others added 5 commits

January 5, 2021 17:31


          extensive clean up of stateless keygen happy path: store all intermed…

5cb9e5a

…iate state, consistent naming


          prototype a generic FillMap struct for storing expected messages

4c63ec2


          begin using the new FillMap in keygen

2b0e487


          trivial cleanup

40f8394


          devops/github actions for running cargo test, clippy and fmt

960d26c

ggutoski added 27 commits

January 12, 2021 15:38


          add to README: install gmp

8d8e83c


          rename protocols -> protocol

e757a97


          move keygen tests to a separate file

e223a5c


          WIP: object-oriented design for keygen round 1

11d1707


          WIP: stateful keygen round 2

3cb4dbd


          move stateless keygen into its own module


          test stubs

01645cf


          make generic ID type for Protocol, State

2dc1210


          begin mock test suite

699fee7


          move mock to tests/ subfolders, mark as abandoned

f251fb7


          finish minimal stateful keygen happy path

02d423a


          add test that everyone got the right pubkey

ba0c3b2


          WIP: begin test for stateful keygen happy path, add get_id() method t…

d3a98e4

…o State trait


          minimal passing test for stateful keygen happy path

5c36b87


          replace generic ID type with concrete String

59c8703


          satisfy clippy

612dc0e


          extensive clean up of stateless keygen happy path: store all intermed…

117d48b

…iate state, consistent naming


          prototype a generic FillMap struct for storing expected messages

df0c38f


          begin using the new FillMap in keygen

b24db53


          trivial cleanup

7f2ba93


          add Result associated type to State trait

eee832b


          clean up tests a bit


          satisfy clippy under rust toolchain 1.49

067336d


          run cargo fmt

786f179


          move test cases higher up the mod tree

d93d20d


          add to README: install gmp

ee5cb51


          Merge branch 'abstract-stateful' of github.com:axelarnetwork/thrush i…

ea66c96

…nto abstract-stateful

ggutoski mentioned this pull request

Clearer design #5

Closed

ggutoski merged commit db321cb into master

ggutoski deleted the abstract-stateful branch

January 12, 2021 21:23

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet