fix option (#619)

AcalaNetwork · Jan 4, 2024 · e26e1cc · e26e1cc
1 parent fd1d5de
commit e26e1cc
Show file tree

Hide file tree

Showing 4 changed files with 16 additions and 8 deletions.
diff --git a/packages/chopsticks/src/plugins/dry-run/cli.ts b/packages/chopsticks/src/plugins/dry-run/cli.ts
@@ -6,9 +6,11 @@ import type { Argv } from 'yargs'
 
 const schema = z.object({
   ...configSchema.shape,
-  extrinsic: z.string({
-    description: 'Extrinsic or call to dry run. If you pass call here then address is required to fake signature',
-  }),
+  extrinsic: z
+    .string({
+      description: 'Extrinsic or call to dry run. If you pass call here then address is required to fake signature',
+    })
+    .optional(),
   address: z
     .string({
       description: 'Address to fake sign extrinsic',

diff --git a/packages/chopsticks/src/plugins/dry-run/dry-run-extrinsic.ts b/packages/chopsticks/src/plugins/dry-run/dry-run-extrinsic.ts
@@ -11,6 +11,10 @@ import { setupContext } from '../../context.js'
 export const dryRunExtrinsic = async (argv: DryRunSchemaType) => {
   const context = await setupContext(argv)
 
+  if (!argv.extrinsic) {
+    throw new Error('Extrinsic is required')
+  }
+
   const input = argv['address']
     ? { call: argv['extrinsic'] as HexString, address: argv['address'] }
     : (argv['extrinsic'] as HexString)

diff --git a/packages/chopsticks/src/plugins/dry-run/index.ts b/packages/chopsticks/src/plugins/dry-run/index.ts
@@ -3,9 +3,11 @@ import { z } from 'zod'
 
 export const dryRunSchema = z.object({
   ...configSchema.shape,
-  extrinsic: z.string({
-    description: 'Extrinsic or call to dry run. If you pass call here then address is required to fake signature',
-  }),
+  extrinsic: z
+    .string({
+      description: 'Extrinsic or call to dry run. If you pass call here then address is required to fake signature',
+    })
+    .optional(),
   address: z
     .string({
       description: 'Address to fake sign extrinsic',

diff --git a/packages/e2e/src/block.test.ts b/packages/e2e/src/block.test.ts
@@ -14,12 +14,12 @@ describe('block', async () => {
     const promises: Promise<any>[] = []
     expect(await chain.upcomingBlocks()).toEqual(0)
     promises.push(dev.newBlock())
-    await delay(10)
+    await delay(20)
     expect(await chain.upcomingBlocks()).toEqual(1)
 
     promises.push(dev.newBlock())
     promises.push(dev.newBlock())
-    await delay(10)
+    await delay(20)
     expect(await chain.upcomingBlocks()).toEqual(2)
 
     await Promise.all(promises)