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

pss: fixed flaky test TestForwardBasic #1702

Merged
merged 1 commit into from
Aug 30, 2019

Conversation

kortatu
Copy link
Contributor

@kortatu kortatu commented Aug 30, 2019

This test was using a global variable testCases to append test cases to run. When executed several times, newest test cases were appended at the end instead of creating a new slice of test cases.
Previously, executing

$> go test -count 2 -run TestForwardBasic

...always failed. Now, no matter how many times this test is repeated, it always passes.

fixes #1545

Copy link
Member

@acud acud left a comment

Choose a reason for hiding this comment

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

Very nice catch!

@acud acud merged commit 2d377ce into ethersphere:master Aug 30, 2019
@kortatu kortatu deleted the issue-1545 branch August 30, 2019 08:41
@skylenet skylenet added this to the 0.5.0 milestone Sep 17, 2019
chadsr added a commit to chadsr/swarm that referenced this pull request Sep 23, 2019
* 'master' of github.com:ethersphere/swarm:
  pss: Modularize crypto and remove Whisper. Step 1 - isolate whisper code (ethersphere#1698)
  pss: Improve pressure backstop queue handling - no mutex (ethersphere#1695)
  cmd/swarm-snapshot: if 2 nodes to create snapshot use connectChain (ethersphere#1709)
  network: Add API for Capabilities (ethersphere#1675)
  pss: fixed flaky test that was using a global variable instead of a local one (ethersphere#1702)
  pss: Port tests to `network/simulation` (ethersphere#1682)
  storage: fix hasherstore seen check to happen when error is nil (ethersphere#1700)
  vendor: upgrade go-ethereum to 1.9.2 (ethersphere#1689)
  bzzeth: initial support for bzz-eth protocol (ethersphere#1571)
  network/stream: terminate runUpdateSyncing on peer quit (ethersphere#1696)
  all: first working SWAP version (ethersphere#1554)
  version: update to v0.5.0 unstable (ethersphere#1694)
  chunk, storage: storage with multi chunk Set method (ethersphere#1684)
  chunk, storage: add HasMulti to chunk.Store (ethersphere#1686)
  chunk, shed, storage: chunk.Store GetMulti method (ethersphere#1691)
  api, chunk: progress bar support (ethersphere#1649)
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.

TestForwardBasic is flaky
4 participants