Create unit-test for parse public key and assert correct behavior

[iramiller]

Summary of Bug

Parse public key appears to incorrectly handle public keys (observed behavior: empty addresses) when used by memorialize contract

Version

v1.0.0-HEAD

Steps to Reproduce/Remediation

During testing of P8e client integration using Memorialize contract it seems that the parse public key method is not behaving as expected. Create a unit-test for this method and ensure it correctly handles both 33 byte (compressed) and 64 byte (uncompressed) secp256k1 keys.

provenance/x/metadata/types/p8e.go

Lines 381 to 391 in 4192fd4

	func parsePublicKey(data []byte) (tmcrypt.PubKey, sdk.AccAddress, error) {
	// Parse the secp256k1 public key.
	pk, err := btcec.ParsePubKey(data, btcec.S256())
	if err != nil {
	return nil, nil, err
	}
	// Create tendermint public key type and return with address.
	tmKey := tmcurve.PubKey{} // PubKeySecp256k1{}
	copy(tmKey[:], pk.SerializeCompressed())
	return tmKey, tmKey.Address().Bytes(), nil
	}

Appropriate test case public key values:

// TODO @piercetrey-figure will supply these test case values
comp := ''"
uncomp := ""

---

For Admin Use

• Not duplicate issue
• Appropriate labels applied
• Appropriate contributors tagged
• Contributor assigned/self-assigned

[piercetrey-figure]

here are the uncompressed/compressed base64-encoded forms of a secp256k1 public key

uncompressed: BGxX6eJRAdXlU64APi95Al44m1FJVgfHlrTpXAqUAB+8JNhM0HgIGWElKbgD6K0KOX9HTJZdlX0z3WTmQrdW+8Q
compressed: AmxX6eJRAdXlU64APi95Al44m1FJVgfHlrTpXAqUAB+8