drjones/AetherForge
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix erasure Params normalize test to match defaulting behavior.
Some checks failed
CI Docker Mining Proof / Linux agent hashrate proof (push) Has been cancelled
Details

Browse Source 
Non-positive shard counts normalize to 4+2 defaults rather than erroring; align TestParamsNormalizeDefaultsAndLimits with codec.Normalize semantics.
This commit is contained in:
AetherForge
2026-06-07 06:49:10 -07:00
parent 821a0e2cd7
commit f5e9b765fe
1 changed files with 7 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
server/internal/erasure/codec_test.go
View File

@@ -78,12 +78,14 @@ func TestEncodeRejectsEmptyPayload(t *testing.T) {
}
}
func TestParamsNormalizeInvalid(t *testing.T) {
if _, err := (Params{DataShards: 0, ParityShards: 0}).Normalize(); err != nil {
t.Fatalf("zero values should normalize: %v", err)
func TestParamsNormalizeDefaultsAndLimits(t *testing.T) {
p, err := (Params{DataShards: 0, ParityShards: 0}).Normalize()
if err != nil || p.DataShards != DefaultDataShards || p.ParityShards != DefaultParityShards {
t.Fatalf("zero values should normalize to defaults: %+v err=%v", p, err)
}
if _, err := (Params{DataShards: -1, ParityShards: 1}).Normalize(); err == nil {
t.Fatal("expected error for negative data shards")
p, err = (Params{DataShards: -1, ParityShards: 1}).Normalize()
if err != nil || p.DataShards != DefaultDataShards {
t.Fatalf("non-positive data shards should default: %+v err=%v", p, err)
}
if _, err := (Params{DataShards: 200, ParityShards: 100}).Normalize(); err == nil {
t.Fatal("expected error for too many shards")