Add typing to polkadot queries and events #1001
Conversation
| async function getSubIdentityName(helper: UniqueHelper, address: string) { | ||
| return ((await helper.getApi().query.identity.superOf(address)).toHuman() as any); | ||
| function instanceOfCodec(obj: any): obj is Codec { | ||
| return typeof obj == 'object' && 'registry' in obj; |
| } | ||
|
|
||
| describe('Integration Test: Identities Manipulation', () => { | ||
| let superuser: IKeyringPair; | ||
|
|
||
| before(async function() { | ||
| if(!process.env.RUN_COLLATOR_TESTS) this.skip(); | ||
| function equal(this: any, _super: any) { |
There was a problem hiding this comment.
Неплохо, но надо бы это для всех тестов включить (глобальный before/вообще без обёрток)
tests/package.json
Outdated
| @@ -140,6 +140,7 @@ | |||
| "@polkadot/wasm-crypto-wasm": "^7.2.1", | |||
| "@rmrk-team/evm-contracts": "^1.2.1", | |||
| "@typechain/web3-v1": "^6.0.3", | |||
| "@unique-nft": "workspaces^", | |||
There was a problem hiding this comment.
| "@unique-nft": "workspaces^", | |
| "@unique-nft/opal-testnet-types": "workspaces^", |
| @@ -152,6 +153,9 @@ | |||
| "resolutions": { | |||
| "decode-uri-component": "^0.2.1" | |||
| }, | |||
| "workspaces": [ | |||
| "src/interfaces" | |||
There was a problem hiding this comment.
В интерфейсы нужно добавить package.json с name: @unique-nft/opal-testnet-types
fairax
force-pushed
the
fix/ts-tests-linter-warnings
branch
2 times, most recently
from
1f54fc9 to
625bc84
Compare
| @@ -123,7 +123,7 @@ describe('Integration Test: Identities Manipulation', () => { | |||
| const oldIdentities = await getIdentityAccounts(helper); | |||
|
|
|||
| // delete a couple, check that they are no longer there | |||
| const registry = helper.api!.registry; | |||
| const registry = helper.getApi().registry; | |||
There was a problem hiding this comment.
Почему бы не переименовать поле api в _api/#api, и не создать getter?
There was a problem hiding this comment.
В плане,
#api?: ApiPromise;
get api(): ApiPromise {
if(this.#api) return this.#api;
this.#api = ApiPromise.create(...);
return this.#api;
}
Чтобы не писать везде getApi(), это некрасиво)
There was a problem hiding this comment.
Там в текущей реализации бросает эксцепшен, если коннекта нет, это как раз то поведение, которое хочется сохранить. Через геттеры вроде не очень эксцепшенами бросаться
There was a problem hiding this comment.
Какая разница, эксепшон оно кинет, или руганётся на undefined в случае с .api!
У нас коннекшон есть в 99% случаев, я не думаю что ради этого стоит весь код в таком стиле писать
Из геттеров можно бросаться эксепшонами, в стектрейсе они тоже видны
There was a problem hiding this comment.
У нас - да, однако в скриптах хелпер инициализируется самостоятельно, без using.
Это оттуда пошло и опять же хочу сохранить для пакета.
There was a problem hiding this comment.
Потом в скриптах у меня было часто
const api = helper.getApi();
Что не так красиво, обращаясь к его свойству
There was a problem hiding this comment.
Ну вот да, с get методом напрашивается сделать один раз получение
А если геттером сделать - то можно код как есть оставить
fairax
force-pushed
the
fix/ts-tests-linter-warnings
branch
from
6d7420f to
742522c
Compare
No description provided.