Refactor ContractData vs copied ContractInfoResponse

[ueco-jb]

During this implementation CosmWasm/cw-plus#554
by ommision we included new structure ContractInfoResponse, which is basically copy of already existing Contract Data.
https://github.com/CosmWasm/cw-plus/blob/main/packages/multi-test/src/wasm.rs#L63-L77

ContractData should be a superset of ContractInfoResponse.
Add pinned and ibc_port to the ContractData, originally set to false and None.

[webmaster128]

Seems like label is currently missing here. created is something that is not supposed to be exported if I understand this comment correctly, so we should not add it:

  // Created Tx position when the contract was instantiated.
  // This data should kept internal and not be exposed via query results. Just
  // use for sorting
  AbsoluteTxPosition created = 5;

ContractData should be a superset of ContractInfoResponse.

Doesn't that mean you need more fields in ContractData? Is there anything that should be done in this repo?

[ueco-jb]

Doesn't that mean you need more fields in ContractData?

*ContractInfoResposne

Yeah, you're correct. At that time I though that both structures were not needed, since both were representing basically same data.
And since label could/should be added in comswasm_std, I think there's actually nothing to do here.

Thank you for pointing that out.