wip

Author: benesjan

yarn-project/end-to-end/src/bench/client_flows/amm.test.ts

@@ -220,15 +220,15 @@ describe('AMM benchmark', () => {
                 );
               }
 
-              // The following check serves as a regression test. If you get a failure and it's expected then just go
-              // ahead and update the EXPECTED_ROUND_TRIPS constants. If the failure is unexpected this needs to be
-              // investigated. To investigate set the DEBUG_ROUND_TRIPS environment variable to true on the `next`
-              // branch and here and compare the outputs (what round trips are newly appeared or disappeared?).
+              // The following check serves as a regression test. If you get a failure and it is expected, just update
+              // the EXPECTED_ROUND_TRIPS constants. If the failure is unexpected, it needs to be investigated. To
+              // investigate, set the DEBUG_ROUND_TRIPS environment variable to true on both the `next` branch and
+              // here, then compare the outputs (look for any newly appearing or missing round trips).
               //
-              // Note that unfortunately the round trip values depend on this test suite being run in full and in the
-              // expected sequence.
+              // Note that the round trip values depend on this test suite being run in its entirety and in the correct
+              // order.
               //
-              // If you got this failure in CI and you have no idea what this is about ping @benesjan.
+              // If you encounter this failure in CI and are unsure of the cause, contact @benesjan.
               expect(actualRoundTrips).toBe(expectedRoundTrips);
             }
           });