Skip to content
This repository has been archived by the owner on Aug 2, 2021. It is now read-only.

swap: fix and rename Peer.getLastSentCumulativePayout #1769

Merged
merged 2 commits into from
Sep 19, 2019

Conversation

ralph-pichler
Copy link
Member

  • cumulative amount of the last sent cheque (previously it was the last received one) is used as a basis for the next one.
  • getLastCumulativePayout was renamed to getLastSentCumulativePayout

Copy link
Contributor

@mortelli mortelli left a comment

Choose a reason for hiding this comment

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

LGTM.

@ralph-pichler did our test suite catch this? if not: is it possible to add tests that do?

@ralph-pichler
Copy link
Member Author

@mortelli fabios not yet merged simulation tests did. But i can add a unit test as well.

@ralph-pichler ralph-pichler force-pushed the swap_correct_cumulative_payout branch from 00ff820 to e393949 Compare September 18, 2019 21:02
Copy link
Contributor

@Eknir Eknir left a comment

Choose a reason for hiding this comment

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

LGTM!

@ralph-pichler ralph-pichler merged commit bf33304 into master Sep 19, 2019
@ralph-pichler ralph-pichler deleted the swap_correct_cumulative_payout branch September 19, 2019 07:57
@skylenet skylenet added this to the 0.5.0 milestone Sep 19, 2019
chadsr added a commit to chadsr/swarm that referenced this pull request Sep 23, 2019
* 'master' of github.com:ethersphere/swarm: (32 commits)
  network/stream: refactor cursors tests (ethersphere#1786)
  network: Add capabilities if peer from store does not have it (ethersphere#1791)
  Swap logger (ethersphere#1754)
  network: Add capability filtered depth calculation (ethersphere#1787)
  travis: remove go1.12 job (ethersphere#1784)
  cmd/swarm: correct bzznetworkid flag description (ethersphere#1761)
  network, pss: Capability in pss (ethersphere#1764)
  network/stream: handle nil peer in TestNodesExchangeCorrectBinIndexes (ethersphere#1779)
  protocols, retrieval: swap-enabled messages implement Price (ethersphere#1771)
  cmd/swarm-smoke: fix waitToPushSynced connection closing (ethersphere#1781)
  cmd/swarm: simplify testCluster.StartNewNodes (ethersphere#1777)
  build: increase golangci-lint deadline (ethersphere#1778)
  docker: ignore build/bin when copying files (ethersphere#1780)
  swap: fix and rename Peer.getLastSentCumulativePayout (ethersphere#1769)
  network/stream: more resilient TestNodesCorrectBinsDynamic (ethersphere#1776)
  network: Add Capabilities to Kademlia database (ethersphere#1713)
  network: add own address to KademliaInfo (ethersphere#1775)
  pss: Refactor. Step 2. Refactor forward cache (ethersphere#1742)
  all: configurable payment/disconnect thresholds (ethersphere#1729)
  network/stream/v2: more resilient TestNodesExchangeCorrectBinIndexes (ethersphere#1760)
  ...
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants