some stuff

Author: technicallyty

activities/loadtest/loadtest.go

@@ -69,6 +69,8 @@ type PetriChain interface {
 	GetNodes() []types.NodeI
 }
 
+// TODO: this function is kind of confusing because we give it a half-baked loadtest spec, and then fill in the rest.
+// I'm not sure if there is a better/more clearer way to go about this, but it seems confusing as is.
 func generateLoadTestSpec(ctx context.Context, logger *zap.Logger, chain PetriChain, chainID string,
 	loadTestSpec ctltypes.LoadTestSpec, baseMnemonic string, numWallets int,
 ) ([]byte, error) {

workflows/testnet/testnet.go

@@ -297,7 +297,6 @@ func startWorkflow(ctx workflow.Context, req messages.TestnetWorkflowRequest, ru
 
 	shutdownSelector := workflow.NewSelector(ctx)
 	// 1. load test selector
-	// TODO: do catalyst refactor here.
 	loadTestFuture, err := runLoadTest(ctx, req, chainState, providerState, shutdownSelector)
 	if err != nil {
 		workflow.GetLogger(ctx).Error("load test initiation failed", zap.Error(err))