Some conversions for Mock failed

[yito88]

Crate

IBC

Summary of Bug

I found two bugs in using mock.

• ClientType::Mock couldn't be converted with from_str() when using ClientType::as_string()
  • Looks inconsistent between as_string() and from_str()
• When converting MockHeader to RowMockHeader, stack overflow happened
  • It seems that the From implementation causes this bug

Version

0.5.0

Steps to Reproduce

• Try to get the client type from the string

let client_type = ClientType::Mock;
let type_string = client_type.as_string();
let client_type_from_str = ClientType::from_str(type_string).expect("Failed")

• Try to encode MockHeader wrapped by AnyHeader

let height = Height::new(1, 10);
let header = MockHeader::new(height);
// `wrap_any()` isn't implemented for now
let any_header = AnyHeader::from(header);
let bytes = header.encode_vec().expect("Stack overflow happened");

Acceptance Criteria

• ClientType::Mock can be converted from a proper string
• MockHeader can be encoded

---

For Admin Use

• Not duplicate issue
• Appropriate labels applied
• Appropriate milestone (priority) applied
• Appropriate contributors tagged
• Contributor assigned/self-assigned