Skip to content

Commit

Permalink
swarm, swarm/storage: lower constants for faster tests (ethereum#17876)
Browse files Browse the repository at this point in the history
* swarm/storage: lower constants for faster tests

* swarm: reduce test size for TestLocalStoreAndRetrieve

* swarm: reduce nodes for dec_inc_node_count
  • Loading branch information
nonsense authored Oct 9, 2018
1 parent d5c7a60 commit 0fe9a37
Show file tree
Hide file tree
Showing 6 changed files with 18 additions and 21 deletions.
6 changes: 3 additions & 3 deletions swarm/network_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -151,13 +151,13 @@ func TestSwarmNetwork(t *testing.T) {
name: "dec_inc_node_count",
steps: []testSwarmNetworkStep{
{
nodeCount: 5,
nodeCount: 3,
},
{
nodeCount: 3,
nodeCount: 1,
},
{
nodeCount: 10,
nodeCount: 5,
},
},
options: &testSwarmNetworkOptions{
Expand Down
3 changes: 2 additions & 1 deletion swarm/storage/chunker_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -232,7 +232,8 @@ func TestDataAppend(t *testing.T) {
func TestRandomData(t *testing.T) {
// This test can validate files up to a relatively short length, as tree chunker slows down drastically.
// Validation of longer files is done by TestLocalStoreAndRetrieve in swarm package.
sizes := []int{1, 60, 83, 179, 253, 1024, 4095, 4096, 4097, 8191, 8192, 8193, 12287, 12288, 12289, 524288, 524288 + 1, 524288 + 4097, 7 * 524288, 7*524288 + 1, 7*524288 + 4097}
//sizes := []int{1, 60, 83, 179, 253, 1024, 4095, 4096, 4097, 8191, 8192, 8193, 12287, 12288, 12289, 524288, 524288 + 1, 524288 + 4097, 7 * 524288, 7*524288 + 1, 7*524288 + 4097}
sizes := []int{1, 60, 83, 179, 253, 1024, 4095, 4097, 8191, 8192, 12288, 12289, 524288}
tester := &chunkerTester{t: t}

for _, s := range sizes {
Expand Down
2 changes: 1 addition & 1 deletion swarm/storage/filestore_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@ import (
"testing"
)

const testDataSize = 0x1000000
const testDataSize = 0x0001000

func TestFileStorerandom(t *testing.T) {
testFileStoreRandom(false, t)
Expand Down
16 changes: 8 additions & 8 deletions swarm/storage/ldbstore_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -111,12 +111,12 @@ func TestDbStoreCorrect_1(t *testing.T) {
testDbStoreCorrect(1, 4096, false, t)
}

func TestDbStoreRandom_5k(t *testing.T) {
testDbStoreRandom(5000, 0, false, t)
func TestDbStoreRandom_1k(t *testing.T) {
testDbStoreRandom(1000, 0, false, t)
}

func TestDbStoreCorrect_5k(t *testing.T) {
testDbStoreCorrect(5000, 4096, false, t)
func TestDbStoreCorrect_1k(t *testing.T) {
testDbStoreCorrect(1000, 4096, false, t)
}

func TestMockDbStoreRandom_1(t *testing.T) {
Expand All @@ -127,12 +127,12 @@ func TestMockDbStoreCorrect_1(t *testing.T) {
testDbStoreCorrect(1, 4096, true, t)
}

func TestMockDbStoreRandom_5k(t *testing.T) {
testDbStoreRandom(5000, 0, true, t)
func TestMockDbStoreRandom_1k(t *testing.T) {
testDbStoreRandom(1000, 0, true, t)
}

func TestMockDbStoreCorrect_5k(t *testing.T) {
testDbStoreCorrect(5000, 4096, true, t)
func TestMockDbStoreCorrect_1k(t *testing.T) {
testDbStoreCorrect(1000, 4096, true, t)
}

func testDbStoreNotFound(t *testing.T, mock bool) {
Expand Down
8 changes: 2 additions & 6 deletions swarm/storage/memstore_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -111,19 +111,15 @@ func TestMemStoreAndLDBStore(t *testing.T) {
chunkSize: 4096,
},
{
n: 201,
n: 101,
chunkSize: 4096,
},
{
n: 501,
chunkSize: 4096,
},
{
n: 3100,
chunkSize: 4096,
},
{
n: 100,
n: 1100,
chunkSize: 4096,
},
}
Expand Down
4 changes: 2 additions & 2 deletions swarm/swarm_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -316,11 +316,11 @@ func TestLocalStoreAndRetrieve(t *testing.T) {
}

// by default, test only the lonely chunk cases
sizes := []int{1, 60, 4097, 524288 + 1, 7*524288 + 1, 128*524288 + 1}
sizes := []int{1, 60, 4097, 524288 + 1, 7*524288 + 1}

if *longrunning {
// test broader set of cases if -longruning flag is set
sizes = append(sizes, 83, 179, 253, 1024, 4095, 4096, 8191, 8192, 8193, 12287, 12288, 12289, 123456, 2345678, 67298391, 524288, 524288+4096, 524288+4097, 7*524288, 7*524288+4096, 7*524288+4097, 128*524288, 128*524288+4096, 128*524288+4097, 816778334)
sizes = append(sizes, 83, 179, 253, 1024, 4095, 4096, 8191, 8192, 8193, 12287, 12288, 12289, 123456, 2345678, 67298391, 524288, 524288+4096, 524288+4097, 7*524288, 7*524288+4096, 7*524288+4097, 128*524288+1, 128*524288, 128*524288+4096, 128*524288+4097, 816778334)
}
for _, n := range sizes {
testLocalStoreAndRetrieve(t, swarm, n, true)
Expand Down

0 comments on commit 0fe9a37

Please sign in to comment.