Fix queries and tx integrations

[ical10]

Issues

We are trying to integrate this package into our project ink!athon, but ran into some issues (scio-labs/inkathon#64).

The issues are:

• Incorrect type generation: some unescaped control characters show up in the generated stringified JSON
• Typed transaction doesn't work: undefined error when getting event identifier

Expected Results

It should work seamlessly without any issues. This PR aims to solve it.

In addition, I added a sample contract in our boilerplate as an example, created some tests for it, and checked that other tests still pass.

cc @forgetso @wottpal

[forgetso]

@ical10 thanks for the PR! Is the format below (ContractName::MethodName::EventName) specified anywhere like in /node_modules/@polkadot/types-support/metadata/v15/substrate-types.json ?

		// extract event identifier (ContractName::MethodName::EventName -> EventName)
		const length = event.identifier.toString().split("::").length;
		const name = event.identifier.toString().split("::")[length - 1] ?? event.identifier.toString();

How does polkadotJs apps pluck the event name from the DecodedEvent?

[ical10]

Hi @forgetso

Is the format below (ContractName::MethodName::EventName) specified anywhere like in /node_modules/@polkadot/types-support/metadata/v15/substrate-types.json ?

This isn't defined anywhere in the metadata, but this is coming from the new format inside events field in contract's ABI (contract.json) generated by ink v5.0.

  "events": [
      {
        "args": [ ... ],
        "docs": [ ... ],
        "label": "...",
+       "module_path": "erc20::erc20",
+       "signature_topic": "0xb5b61a3e6a21a16be4f044b517c28ac692492f73c5bfd3f60178ad98c767f4cb"
      },      
      ...
  ]

Events 2.0

How does polkadotJs apps pluck the event name from the DecodedEvent?

Not sure about how it exactly works. In any case, in order to get the typesafe tx call working, we need to "grab" only label from the DecodedEvent (which is currently prepended with module_path).

[forgetso]

@ical10 thanks for the explanation. I just want to make sure that the string format is always going to be the same so the code always works. Having read it a bit more closely, I can see the code works if there are no :: separators.

You'll need to get the tests working in order to merge.

Please also make this a variable instead of writing it twice:

event.identifier.toString().split("::")

Otherwise, looks good! :)

[ical10]

@forgetso

Sorry for the delay. I've added your feedbacks, also ran the tests and all passed.