Add lock events to tokens pallet

[sea212]

Fundamental issue: Block indexers and processors like Subsquid have a really hard time to track locks.

This PR mirrors updates within pallet-balances: paritytech/substrate#12287

Difference in respect to paritytech/substrate#12287:

• A currency_id field was added to the events
• The lock reason was removed from the events (no differentiation between MISC and FEE reasons)

The tests cases were adjusted to the previously mentioned differences:

Locks = []                          --> [10/DOT]                    // emits Locked(currency_id: DOT, ..., amount=10)
Locks = [10/DOT]                    --> [15/DOT]                    // emits Locked(currency_id: DOT, ..., amount=5)
Locks = [15/DOT]                    --> [15/DOT, 20/BTC]            // emits Locked(currency_id: BTC, ..., amount=20)
Locks = [15/DOT, 20/BTC]            --> [15/DOT, 20/BTC, 10/DOT]    // emits nothing
Locks = [15/DOT, 20/BTC, 10/DOT]    --> [15/DOT, 20/BTC, 12/DOT]    // emits nothing
Locks = [15/DOT, 20/BTC, 12/DOT]    --> [15/DOT, 20/BTC, 10/DOT]    // emits nothing
Locks = [15/DOT, 20/BTC, 10/DOT]    --> [15/DOT, 20/BTC, 20/DOT]    // emits Locked(currency_id: DOT, ..., amount=5)
Locks = [15/DOT, 20/BTC, 20/DOT]    --> [15/DOT, 20/BTC, 16/DOT]    // emits Unlocked(currency_id: DOT, ..., amount = 4)
Locks = [15/DOT, 20/BTC, 16/DOT]    --> [15/DOT, 12/BTC, 16/DOT]    // emits Unlocked(currency_id: BTC, ..., amount = 8)
Locks = [15/DOT, 12/BTC, 16/DOT]    --> [15/DOT, 12/BTC]            // emits Unlocked(currency_id: DOT, ..., amount = 1)

Bonus change: Replace deprecated clippy rule name.

[sea212]

Does anyone know why those two pipelines fail?
https://github.com/open-web3-stack/open-runtime-module-library/actions/runs/4018262443/jobs/6903666151
https://github.com/open-web3-stack/open-runtime-module-library/actions/runs/4018262445/jobs/6903669260

I am unable to reproduce the error locally. Regarding the second workflow, I executed cargo test --all --features=runtime-benchmarks just as the pipeline does, however in my local instance it works perfectly fine. Regarding the first workflow, I executed the exact command used in the GitHub workflow (for CodeCov) and also used same cargo-tarpaulin and rustc version. Can't reproduce this either, works locally perfectly fine 😕

[zjb0807]

Does anyone know why those two pipelines fail? https://github.com/open-web3-stack/open-runtime-module-library/actions/runs/4018262443/jobs/6903666151 https://github.com/open-web3-stack/open-runtime-module-library/actions/runs/4018262445/jobs/6903669260

I am unable to reproduce the error locally. Regarding the second workflow, I executed cargo test --all --features=runtime-benchmarks just as the pipeline does, however in my local instance it works perfectly fine. Regarding the first workflow, I executed the exact command used in the GitHub workflow (for CodeCov) and also used same cargo-tarpaulin and rustc version. Can't reproduce this either, works locally perfectly fine 😕

please merge origin/master. I reproduce the error.

[sea212]

please merge origin/master. I reproduce the error.

Thanks fellow enthusiast!
Seems it was too late yesterday, didn't realize that I was on PureStake's fork 🤦‍♂️

[zjb0807]

LGTM