Skip to content
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.

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

Egoistic Funding Test #52

Merged
merged 5 commits into from
Apr 9, 2024

Conversation

sophia1ch
Copy link
Contributor

This PR includes a test case that:

  • tests correctness of SetEgoisticPart
  • tests correctness of SetEgoisticChains
  • tests funding when one participant is set to be egoistic (funds last)

@sophia1ch sophia1ch marked this pull request as ready for review March 26, 2024 13:24
@sophia1ch sophia1ch requested a review from iljabvh March 26, 2024 13:25
@sophia1ch sophia1ch marked this pull request as draft March 26, 2024 15:03
@sophia1ch sophia1ch marked this pull request as ready for review April 2, 2024 13:43
iljabvh
iljabvh previously approved these changes Apr 3, 2024
Copy link
Contributor

@iljabvh iljabvh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good work, marking an important addition to the funding functionality. This breaks the symmetry between funding agents and increases security for some applications.

@@ -0,0 +1,68 @@
// Copyright 2020 - See NOTICE file for copyright holders.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Minor: The dates of this and the other file should be updated :)

iljabvh and others added 5 commits April 9, 2024 11:45
…ctness of egoisticPart and EgoisticChains.

Signed-off-by: Sophia Koehler <[email protected]>
… list of times, each funder took to finish.

feat(TestEgoisticParticipantFunding): Add Test that sets one participant as egoistic, funds and checks if egoistic funder funds last.

Signed-off-by: Sophia Koehler <[email protected]>

refactor(multiledger): Add comment, rename function.

Signed-off-by: Sophia Koehler <[email protected]>

refactor(multiledger): Add var for participants length.

Signed-off-by: Sophia Koehler <[email protected]>
Signed-off-by: Sophia Koehler <[email protected]>

refactor(fund): Add t.Helper()

Signed-off-by: Sophia Koehler <[email protected]>

refactor(fund): Rearrange arguments in fundAll.

Signed-off-by: Sophia Koehler <[email protected]>

refactor(fund): Add comment for FundAll.

Signed-off-by: Sophia Koehler <[email protected]>

refactor(fund): Add const for waiting time.

Signed-off-by: Sophia Koehler <[email protected]>

refactor(fund): Change to loop.

Signed-off-by: Sophia Koehler <[email protected]>

Revert "refactor(fund): change to loop."

This reverts commit a567031.

Signed-off-by: Sophia Koehler <[email protected]>

refactor(fund): change to loop.

Signed-off-by: Sophia Koehler <[email protected]>

refactor(fund): remove loop, add 2 second wait.

Signed-off-by: Sophia Koehler <[email protected]>

refactor(funder): rearrange imports.

Signed-off-by: Sophia Koehler <[email protected]>

refactor(fund): Add waitgroup.

Signed-off-by: Sophia Koehler <[email protected]>

refactor(funder_test)
refactor(fund): Add concurrent testing.

Signed-off-by: Sophia Koehler <[email protected]>

refactor(fund): Add comments, delete unnecessary function.

Signed-off-by: Sophia Koehler <[email protected]>
@sophia1ch sophia1ch merged commit 3c47eab into hyperledger-labs:main Apr 9, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants