[FOLD] Review feedback from @gregtatcam:

** Do not include this message in the squashed commit message
* Move one of the tests and update a comment.

src/test/app/RCLValidations_test.cpp

@@ -106,9 +106,9 @@ class RCLValidations_test : public beast::unit_test::suite
                 *prev, env.app().timeKeeper().closeTime());
             // Force a different hash on the first iteration
             next->updateSkipList();
+            BEAST_EXPECT(next->read(keylet::fees()));
             if (forceHash)
             {
-                BEAST_EXPECT(next->read(keylet::fees()));
                 next->setImmutable();
                 forceHash = false;
             }

src/test/app/TxQ_test.cpp

@@ -2939,9 +2939,9 @@ class TxQ1_test : public beast::unit_test::suite
         // Verify that nothing can be added now that the gap is filled.
         env(noop(alice), seq(aliceSeq + 20), ter(telCAN_NOT_QUEUE_FULL));
 
-        // Close ledger 6.  That removes 6 of alice's transactions,
-        // but alice adds one more transaction at seq(aliceSeq + 20) so
-        // we only see a reduction by 5.
+        // Close ledger 6.  That removes some of alice's transactions,
+        // but alice adds some more transaction(s) so expectedCount
+        // may not reduce to 8.
         env.close();
         checkMetrics(__LINE__, env, 9, 50, 6, 5, 256);
         BEAST_EXPECT(env.seq(alice) == aliceSeq + 13);