fix: remove unnecessary height assertion (#116)

* fix: remove unnecessary assertion

* imp: added exec report

* fix: e2e

contracts/src/SP1ICS07Tendermint.sol

@@ -182,15 +182,6 @@ contract SP1ICS07Tendermint is
         private
         returns (uint256)
     {
-        if (proofHeight.revisionNumber != clientState.latestHeight.revisionNumber) {
-            revert ProofHeightMismatch(
-                proofHeight.revisionNumber,
-                proofHeight.revisionHeight,
-                clientState.latestHeight.revisionNumber,
-                clientState.latestHeight.revisionHeight
-            );
-        }
-
         SP1MembershipProof memory proof = abi.decode(proofBytes, (SP1MembershipProof));
         if (proof.sp1Proof.vKey != MEMBERSHIP_PROGRAM_VKEY) {
             revert VerificationKeyMismatch(MEMBERSHIP_PROGRAM_VKEY, proof.sp1Proof.vKey);

e2e/interchaintestv8/operator/operator.go

@@ -1,11 +1,13 @@
 package operator
 
 import (
+	"bytes"
 	"encoding/base64"
 	"encoding/hex"
 	"encoding/json"
 	"errors"
 	"fmt"
+	"io"
 	"os"
 	"os/exec"
 	"strconv"
@@ -63,7 +65,7 @@ func StartOperator(args ...string) error {
 func UpdateClientAndMembershipProof(trusted_height, target_height uint64, paths string, args ...string) (*sp1ics07tendermint.IICS02ClientMsgsHeight, []byte, error) {
 	args = append([]string{"fixtures", "update-client-and-membership", "--trusted-block", strconv.FormatUint(trusted_height, 10), "--target-block", strconv.FormatUint(target_height, 10), "--key-paths", paths}, args...)
 
-	output, err := exec.Command("target/release/operator", args...).CombinedOutput()
+	output, err := execOperatorCommand(exec.Command("target/release/operator", args...))
 	if err != nil {
 		return nil, nil, err
 	}
@@ -133,9 +135,9 @@ func MisbehaviourProof(cdc codec.Codec, misbehaviour tmclient.Misbehaviour, writ
 	defer os.Remove(misbehaviourFileName)
 
 	args = append([]string{"fixtures", "misbehaviour", "--misbehaviour-path", misbehaviourFileName}, args...)
-	output, err := exec.Command("target/release/operator", args...).CombinedOutput()
+	output, err := execOperatorCommand(exec.Command("target/release/operator", args...))
 	if err != nil {
-		return nil, fmt.Errorf("operator misbehaviour failed: %w, output: %s", err, output)
+		return nil, err
 	}
 
 	// eliminate non-json characters
@@ -306,3 +308,21 @@ func marshalMisbehaviour(cdc codec.Codec, misbehaviour tmclient.Misbehaviour) ([
 
 	return json.Marshal(jsonIntermediary)
 }
+
+func execOperatorCommand(c *exec.Cmd) ([]byte, error) {
+	var outBuf bytes.Buffer
+
+	// Create a MultiWriter to write to both os.Stdout and the buffer
+	multiWriter := io.MultiWriter(os.Stdout, &outBuf)
+
+	// Set the command's stdout and stderror to the MultiWriter
+	c.Stdout = multiWriter
+	c.Stderr = multiWriter
+
+	// Run the command
+	if err := c.Run(); err != nil {
+		return nil, fmt.Errorf("operator command '%s' failed: %s", strings.Join(c.Args, " "), outBuf.String())
+	}
+
+	return outBuf.Bytes(), nil
+}