Skip to content

Commit

Permalink
Add test for out of context shares using share ranges
Browse files Browse the repository at this point in the history
  • Loading branch information
Manav-Aggarwal committed May 18, 2023
1 parent b802fa0 commit 6f8697b
Showing 1 changed file with 36 additions and 3 deletions.
39 changes: 36 additions & 3 deletions pkg/shares/compact_shares_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -112,20 +112,53 @@ func Test_processCompactShares(t *testing.T) {
}
}

func TestParseOutOfContextShares(t *testing.T) {
func TestAlSplit(t *testing.T) {
txs := testfactory.GenerateRandomlySizedTxs(1000, 150)
txShares, _, _, err := SplitTxs(txs)
require.NoError(t, err)
resTxs, err := ParseTxs(txShares)
require.NoError(t, err)
assert.Equal(t, resTxs, txs)

}

func TestParseRandomOutOfContextShares(t *testing.T) {
txs := testfactory.GenerateRandomlySizedTxs(1000, 150)
txShares, _, _, err := SplitTxs(txs)
require.NoError(t, err)

for i := 0; i < 1000; i++ {
start, length := testfactory.GetRandomSubSlice(len(txShares))

resTxs, err := ParseTxs(txShares[start : start+length])
randomRange := NewRange(start, start+length)
resTxs, err := ParseTxs(txShares[randomRange.Start:randomRange.End])
require.NoError(t, err)
assert.True(t, testfactory.CheckSubArray(txs, resTxs))
}
}

func TestParseOutOfContextSharesUsingShareRanges(t *testing.T) {
txs := testfactory.GenerateRandomlySizedTxs(1000, 150)
txShares, _, shareRanges, err := SplitTxs(txs)
require.NoError(t, err)

resTxs, err := ParseTxs(txShares)
require.NoError(t, err)
assert.Equal(t, resTxs, txs)

for key, r := range shareRanges {
resTxs, err := ParseTxs(txShares[r.Start:r.End])
require.NoError(t, err)
has := false
for _, tx := range resTxs {
if tx.Key() == key {
has = true
break
}
}
assert.True(t, has)
}
}

func TestCompactShareContainsInfoByte(t *testing.T) {
css := NewCompactShareSplitter(appns.TxNamespace, appconsts.ShareVersionZero)
txs := testfactory.GenerateRandomTxs(1, appconsts.ContinuationCompactShareContentSize/4)
Expand Down

0 comments on commit 6f8697b

Please sign in to comment.