Merge pull request #12 from orijtech/ui-fixes

orijtech · Dec 15, 2022 · cf9cd20 · cf9cd20
2 parents 1d3c5ba + c31aef3
commit cf9cd20
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 10 deletions.
diff --git a/ui/src/components/load-tester/Fields.tsx b/ui/src/components/load-tester/Fields.tsx
@@ -57,14 +57,14 @@ export const fields: FormField[] = [
         name: 'maxEndpointCount',
         label: 'Max endpoint count',
         info: 'The maximum number of endpoints to use for testing, where 0 means unlimited. Maps to --max-endpoints in tm-load-test.',
-        default: 1,
+        default: 10,
         fieldType: FieldType.VALUE_BASED,
     },
     {
         name: 'minPeerConnectivityCount',
         label: 'Min peer connectivity count',
         info: 'The minimum number of peers to which each peer must be connected before starting the load test. Maps to --min-peer-connectvity in tm-load-test.',
-        default: 1,
+        default: 0,
         fieldType: FieldType.VALUE_BASED,
     },
     {
@@ -78,35 +78,35 @@ export const fields: FormField[] = [
         name: 'sendPeriod',
         label: 'Send period',
         info: 'The period (in seconds) at which to send batches of transactions. Maps to --send-period in tm-load-test.',
-        default: 20,
+        default: 1,
         fieldType: FieldType.TIME_BASED,
     },
     {
         name: 'connectionCount',
         label: 'Connection Count',
-        info: 'The identifier of the client factory to use for generating load testing transactions. Maps to --client-factory in tm-load-test.',
+        info: 'The number of connections to open to each endpoint simultaneously. Maps to --connections in tm-load-test.',
         default: 1,
         fieldType: FieldType.VALUE_BASED,
     },
     {
         name: 'statsOutputFilePath',
         label: 'Stats Output File Path',
         info: 'Where to store aggregate statistics (in CSV format) for the load test. Maps to --stats-output in tm-load-test.',
-        default: 1,
+        default: "",
         fieldType: FieldType.VALUE_BASED,
     },
     {
         name: 'expectPeersCount',
         label: 'Expect peers count',
         info: 'The minimum number of peers to expect when crawling the P2P network from the specified endpoint(s) prior to waiting for workers to connect. Maps to --expect-peers in tm-load-test.',
-        default: 1,
+        default: 0,
         fieldType: FieldType.VALUE_BASED,
     },
     {
         name: 'transactionSizeBytes',
         label: 'Transaction size bytes',
         info: 'The size of each transaction, in bytes - must be greater than 40. Maps to --size in tm-load-test.',
-        default: 40,
+        default: 250,
         fieldType: FieldType.VALUE_BASED,
     },
     {
@@ -115,5 +115,12 @@ export const fields: FormField[] = [
         info: 'The number of transactions to generate each second on each connection, to each endpoint. Maps to --rate in tm-load-test.',
         default: 1000,
         fieldType: FieldType.VALUE_BASED,
-    }
+    },
+    {
+        name: 'transactionCount',
+        label: 'Transaction count',
+        info: 'The maximum number of transactions to send - set to -1 to turn off this limit. Maps to --count in tm-load-test.',
+        default: -1,
+        fieldType: FieldType.VALUE_BASED,
+    },
 ];
diff --git a/ui/src/components/load-tester/LoadTester.tsx b/ui/src/components/load-tester/LoadTester.tsx
@@ -50,8 +50,8 @@ export default function LoadTester() {
                 .setStatsOutputFilePath(data.statsOutputFilePath)
                 .setTransactionCount(data.transactionCount)
                 .setTransactionSizeBytes(data.transactionSizeBytes)
-                .setTransactionsPerSecond(data.transactionsPerSecond);
-
+                .setTransactionsPerSecond(data.transactionsPerSecond)
+                .setConnectionCount(data.connectionCount);
             const result = await service.runLoadtest(request, null);
             setData(result.toObject());
             console.log(result.toObject());