clean up testing

Author: gjermundgaraba

modules/light-clients/10-attestations/README.md

@@ -0,0 +1,31 @@
+# 10-attestations Light Client
+
+An attestor-based IBC light client that verifies state using quorum-signed ECDSA attestations from a fixed set of trusted signers.
+
+## Trust Model
+
+- A fixed set of ECDSA attestors (EOA addresses) is configured at client creation
+- Updates require `minRequiredSigs` unique signatures from the attestor set
+
+## State
+
+**Client State**
+- `attestorAddresses` — trusted attestor set
+- `minRequiredSigs` — quorum threshold
+- `latestHeight` — highest trusted height
+- `isFrozen` — halts all operations when true
+
+**Consensus State** (per height)
+- `timestamp` — trusted UNIX timestamp in nanoseconds
+
+## Proofs
+
+All proofs contain an `AttestationProof` with:
+- `attestationData` — the attested payload (encoded `StateAttestation` or `PacketAttestation`)
+- `signatures` — 65-byte ECDSA signatures over `sha256(attestationData)`
+
+## Limitations
+
+- No client recovery supported
+- No client upgrades supported
+- No attestor updates or rotation supported

modules/light-clients/10-attestations/attestation_proof_test.go

@@ -0,0 +1,58 @@
+package attestations_test
+
+import (
+	attestations "github.com/cosmos/ibc-go/v10/modules/light-clients/10-attestations"
+)
+
+func (s *AttestationsTestSuite) TestAttestationProofValidateBasic() {
+	testCases := []struct {
+		name             string
+		attestationProof attestations.AttestationProof
+		expErr           string
+	}{
+		{
+			"valid proof",
+			attestations.AttestationProof{
+				AttestationData: []byte("valid data"),
+				Signatures:      [][]byte{make([]byte, 65)},
+			},
+			"",
+		},
+		{
+			"empty attestation data",
+			attestations.AttestationProof{
+				AttestationData: []byte{},
+				Signatures:      [][]byte{make([]byte, 65)},
+			},
+			"attestation data cannot be empty",
+		},
+		{
+			"empty signatures",
+			attestations.AttestationProof{
+				AttestationData: []byte("valid data"),
+				Signatures:      [][]byte{},
+			},
+			"signatures cannot be empty",
+		},
+		{
+			"invalid signature length",
+			attestations.AttestationProof{
+				AttestationData: []byte("valid data"),
+				Signatures:      [][]byte{make([]byte, 64)},
+			},
+			"signature 0 has invalid length",
+		},
+	}
+
+	for _, tc := range testCases {
+		s.Run(tc.name, func() {
+			err := tc.attestationProof.ValidateBasic()
+			if tc.expErr != "" {
+				s.Require().Error(err)
+				s.Require().ErrorContains(err, tc.expErr)
+			} else {
+				s.Require().NoError(err)
+			}
+		})
+	}
+}