docs: improve docs on tests

test/__fixtures/BaseTest.sol

@@ -138,16 +138,10 @@ abstract contract BaseTest is Test {
         vm.record();
         aPair.observation(aIndex);
 
-        // TODO: investigate this retarded error
-        // it's supposed to be 1 access but somehow it's giving 5. To see on foundry github if someone is complaining about this
-        // already did foundryup
-        // maybe I need to upgrade the forge-std lib too???
-
-        // okay upgraded forge-std too
-        // no mentions on github
-        // ask on telegram?
-
         (bytes32[] memory lAccesses,) = vm.accesses(address(aPair));
+
+        // for coverage, due to the optimizer being turned off, the accesses will be >1 instead of 1, due to the struct
+        // members
         // solhint-disable-next-line no-console
         if (lAccesses.length != 1) console2.log ("warn: invalid number of accesses");