Skip to content

Commit

Permalink
clarifies the result of verification
Browse files Browse the repository at this point in the history
  • Loading branch information
staheri14 committed Jun 29, 2023
1 parent 7eb4aef commit d54fdfc
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions proof_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -821,13 +821,13 @@ func TestVerifyNamespace_ShortAbsenceProof_Invalid(t *testing.T) {
Node_0_4, err := tree.computeRoot(0, 4)

Check warning on line 821 in proof_test.go

View workflow job for this annotation

GitHub Actions / golangci-lint

var-naming: don't use underscores in Go names; var Node_0_4 should be Node0_4 (revive)
assert.NoError(t, err)

// nodes needed for the short absence proof of qNS; the proof of inclusion of the parent of Node_4_5;
// this case should not work since the namespace range o Node_4_6, the parent, has overlap with the qNS i.e., 7
// nodes needed for the short absence proof of qNS; the proof of inclusion of the parent of Node_5_6;
// the verification should fail since the namespace range o Node_4_6, the parent, has overlap with the qNS i.e., 7
Node_4_6, err := tree.computeRoot(4, 6)
assert.NoError(t, err)

// nodes needed for another short absence parent of qNS; the proof of inclusion of the grandparent of Node_4_5
// this case should not work since the namespace range of Node_4_8, the grandparent, has overlap with the qNS i.e., 7
// nodes needed for another short absence parent of qNS; the proof of inclusion of the grandparent of Node_5_6
// the verification should fail since the namespace range of Node_4_8, the grandparent, has overlap with the qNS i.e., 7
Node_4_8, err := tree.computeRoot(4, 8)
assert.NoError(t, err)

Expand Down

0 comments on commit d54fdfc

Please sign in to comment.