Remove pub from MockConsensusState::root field

ibc-core/ics03-connection/types/src/connection.rs

@@ -2,7 +2,6 @@
 
 use core::fmt::{Display, Error as FmtError, Formatter};
 use core::time::Duration;
-use core::u64;
 
 use ibc_core_client_types::error::ClientError;
 use ibc_core_commitment_types::commitment::CommitmentPrefix;

ibc-testkit/src/testapp/ibc/clients/mock/consensus_state.rs

@@ -9,11 +9,17 @@ use crate::testapp::ibc::clients::mock::header::MockHeader;
 use crate::testapp::ibc::clients::mock::proto::ConsensusState as RawMockConsensusState;
 pub const MOCK_CONSENSUS_STATE_TYPE_URL: &str = "/ibc.mock.ConsensusState";
 
+// Note: This type differs from the `mock::ConsensusState` type exposed by
+// ibc-proto in a few ways:
+// - ibc-proto's `mock::ConsensusState`'s `header` field takes the form of an
+//   `Option<Header>`, while the `header` field here is not optional.
+// - ibc-proto's `mock::ConsensusState` does not have a `root` field for holding
+//   the `CommitmentRoot`; this field exists here because ...
 #[cfg_attr(feature = "serde", derive(serde::Serialize, serde::Deserialize))]
 #[derive(Clone, Debug, PartialEq, Eq)]
 pub struct MockConsensusState {
     pub header: MockHeader,
-    pub root: CommitmentRoot,
+    root: CommitmentRoot,
 }
 
 impl MockConsensusState {