comment

src/simulation/test/LoadGeneratorTests.cpp

@@ -144,22 +144,12 @@ TEST_CASE("handle soroban loadgen mode misconfiguration gracefully",
     auto& loadGen = app.getLoadGenerator();
 
     // Users are required to run SOROBAN_INVOKE_SETUP_LOAD before running
-    // SOROBAN_INVOKE_LOAD.
-    // This is an example call.
-    // loadGen.generateLoad(GeneratedLoadConfig::createSorobanInvokeSetupLoad(
-    ///* nAccounts */ nAccounts, numInstances,
-    ///* txRate */ 1));
+    // SOROBAN_INVOKE_LOAD. Running a SOROBAN_INVOKE_LOAD without a prior
+    // SOROBAN_INVOKE_SETUP_LOAD should throw a helpful exception explaining 
+    // the misconfiguration.
     auto invokeLoadCfg = GeneratedLoadConfig::txLoad(
         LoadGenMode::SOROBAN_INVOKE, /* nAccounts*/ 1, /* numSorobanTxs */ 1,
         /* txRate */ 1);
-
-    // Plan: See how this acts when run without changes.
-    // Then, add a bit of state if needed to load genreator to
-    // store whether an account has called invoke setup
-    // and if not, print a warning/error or throw an exception
-    // probably thats best since its fatal anyway but its confusing at the
-    // moment.
-
     REQUIRE_THROWS_WITH(
         loadGen.generateLoad(invokeLoadCfg),
         "Before running MODE::SOROBAN_INVOKE, please run "