git.delta.rocks / unique-network / refs/commits / 96d265cead38

difftreelog

fix scheduler doesn't change nonce

Daniel Shiposha2022-11-08parent: #a4f0bd9.patch.diff
in: master

2 files changed

modifiedruntime/common/scheduler.rsdiffbeforeafterboth
3232
33/// The SignedExtension to the basic transaction logic.33/// The SignedExtension to the basic transaction logic.
34pub type SignedExtraScheduler = (34pub type SignedExtraScheduler = (
35 frame_system::CheckSpecVersion<Runtime>,
36 frame_system::CheckGenesis<Runtime>,
37 frame_system::CheckEra<Runtime>,
38 frame_system::CheckNonce<Runtime>,
39 frame_system::CheckWeight<Runtime>,35 frame_system::CheckWeight<Runtime>,
40 maintenance::CheckMaintenance,36 maintenance::CheckMaintenance,
41 ChargeTransactionPayment<Runtime>,37 ChargeTransactionPayment<Runtime>,
42);38);
4339
44fn get_signed_extras(from: <Runtime as frame_system::Config>::AccountId) -> SignedExtraScheduler {40fn get_signed_extras(from: <Runtime as frame_system::Config>::AccountId) -> SignedExtraScheduler {
45 (41 (
46 frame_system::CheckSpecVersion::<Runtime>::new(),
47 frame_system::CheckGenesis::<Runtime>::new(),
48 frame_system::CheckEra::<Runtime>::from(Era::Immortal),
49 frame_system::CheckNonce::<Runtime>::from(frame_system::Pallet::<Runtime>::account_nonce(
50 from,
51 )),
52 frame_system::CheckWeight::<Runtime>::new(),42 frame_system::CheckWeight::<Runtime>::new(),
53 maintenance::CheckMaintenance,43 maintenance::CheckMaintenance,
54 ChargeTransactionPayment::<Runtime>::from(0),44 ChargeTransactionPayment::<Runtime>::from(0),
modifiedtests/src/scheduler.seqtest.tsdiffbeforeafterboth
468 expect(await helper.testUtils.testValue()).to.be.equal(secondExecTestVal);468 expect(await helper.testUtils.testValue()).to.be.equal(secondExecTestVal);
469 });469 });
470
471 itSub('scheduled operations does not change nonce', async ({helper}) => {
472 const scheduledId = await helper.arrange.makeScheduledId();
473 const blocksBeforeExecution = 4;
474
475 await helper.scheduler
476 .scheduleAfter<DevUniqueHelper>(scheduledId, blocksBeforeExecution)
477 .balance.transferToSubstrate(alice, bob.address, 1n);
478
479 const initNonce = await helper.chain.getNonce(alice.address);
480
481 await helper.wait.newBlocks(blocksBeforeExecution + 1);
482
483 const finalNonce = await helper.chain.getNonce(alice.address);
484
485 expect(initNonce).to.be.equal(finalNonce);
486 });
470});487});
471488
472describe('Negative Test: Scheduling', () => {489describe('Negative Test: Scheduling', () => {