Mark comments as FIXMEs

Author: reivilibre

tests/replication/tcp/streams/test_typing.py

@@ -100,8 +100,11 @@ def test_reset(self) -> None:
         This is emulated by jumping the stream ahead, then reconnecting (which
         sends the proper position and RDATA).
         """
-        # A huge RDATA log line is triggered in this test, which breaks trial
+        # FIXME: Because huge RDATA log line is triggered in this test,
+        # trial breaks, sometimes (flakily) failing the test run.
         # ref: https://github.com/twisted/twisted/issues/12482
+        # To remove this, we would need to fix the above issue and
+        # update, including in olddeps (so several years' wait).
         server_logger = logging.getLogger("tests.server")
         server_logger_was_disabled = server_logger.disabled
         server_logger.disabled = True

tests/rest/client/test_profile.py

@@ -649,9 +649,12 @@ def test_set_custom_field_profile_too_long(self) -> None:
         """
         Attempts to set a custom field that would push the overall profile too large.
         """
-        # Because we emit huge SQL log lines and trial sometimes crashes on these,
-        # disable SQL logging for this test
+        # FIXME: Because we emit huge SQL log lines and trial can't handle these,
+        # sometimes (flakily) failing the test run,
+        # disable SQL logging for this test.
         # ref: https://github.com/twisted/twisted/issues/12482
+        # To remove this, we would need to fix the above issue and
+        # update, including in olddeps (so several years' wait).
         sql_logger = logging.getLogger("synapse.storage.SQL")
         sql_logger_was_disabled = sql_logger.disabled
         sql_logger.disabled = True