fix(simulate): fix the bug that the multi pub config does not load co…

…rrectly
emqx · May 9, 2023 · 7c68fa0 · 7c68fa0
1 parent 082edd1
commit 7c68fa0
Showing 1 changed file with 5 additions and 17 deletions.
diff --git a/cli/src/lib/pub.ts b/cli/src/lib/pub.ts
@@ -126,19 +126,7 @@ const multiPub = async (
   commandType: CommandType,
   message?: string | Buffer,
 ) => {
-  const {
-    save,
-    config,
-    count,
-    interval,
-    messageInterval,
-    hostname,
-    port,
-    topic,
-    clientId,
-    verbose,
-    maximumReconnectTimes,
-  } = options
+  const { save, config } = options
 
   let simulator: Simulator = {} as Simulator
   if (commandType === 'simulate') {
@@ -148,15 +136,15 @@ const multiPub = async (
     const simulateOptions = options as SimulatePubOptions
     checkScenarioExists(simulateOptions.scenario, simulateOptions.file)
     simulator = loadSimulator(simulateOptions.scenario, simulateOptions.file)
-
-    checkTopicExists(topic, 'simulate')
   } else {
     options = config ? loadConfig('benchPub', config) : options
     save && saveConfig('benchPub', options)
-
-    checkTopicExists(topic, 'benchPub')
   }
 
+  const { count, interval, messageInterval, hostname, port, topic, clientId, verbose, maximumReconnectTimes } = options
+
+  checkTopicExists(topic, commandType)
+
   const connOpts = parseConnectOptions(options, 'pub')
 
   const pubOpts = parsePublishOptions(options)