Merge pull request #388 from devsapp/fix/fc-stress-run-time

fix: dstress runtime set error
devsapp · Oct 27, 2021 · 61af2c5 · 61af2c5
2 parents d96ea6f + 81c6983
commit 61af2c5
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/src/index.ts b/src/index.ts
@@ -469,6 +469,9 @@ export default class FcBaseComponent extends BaseComponent {
       };
       this.logger.debug(`Using event options: \n${yaml.dump(eventTypeOpts)}`);
     } else if (stressOpts?.functionType === 'http') {
+      if (!argsData?.url) {
+        this.logger.error('Function type is http, please specify --url');
+      }
       httpTypeOpts = {
         url: argsData?.url,
         method: argsData?.method,

diff --git a/src/lib/component/fc-stress.ts b/src/lib/component/fc-stress.ts
@@ -28,7 +28,7 @@ export default class FcStress {
       args += ` --spawn-rate ${this.stressOpts?.spawnRate}`;
     }
     if (this.stressOpts?.runningTime) {
-      args += ` --run-time ${this.stressOpts?.spawnRate}`;
+      args += ` --run-time ${this.stressOpts?.runningTime}`;
     }
     if (this.isEventFunctionType()) {
       args += ` --service-name ${this.eventTypeOpts?.serviceName} --function-name ${this.eventTypeOpts?.functionName}`;