revert: align three files to main after merge

test_no_flask_import, the test_miniofs comment, and the groups.yaml Fernet
comment were intentionally dropped or shortened in #2115; keep main's version
rather than the e2e branch's re-additions.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_017UVfw7aAocC3KCJaEyZ5GU

Author: chandrasekharan-zipstack

tests/groups.yaml

@@ -66,8 +66,7 @@ groups:
       # base.py resolves these at import time with no default; supply test-safe
       # values here.
       DJANGO_SECRET_KEY: test-secret-key-not-for-production
-      # All-zero Fernet key: valid format, zero entropy so it reads as the
-      # obvious test placeholder it is (not a real secret).
+      # All-zero Fernet key: valid format, obviously a placeholder.
       ENCRYPTION_KEY: AAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAA=
       CELERY_BROKER_BASE_URL: redis://localhost:6379
       CELERY_BROKER_USER: guest

unstract/connectors/tests/filesystems/test_miniofs.py

@@ -23,10 +23,8 @@ class TestMinoFS(unittest.TestCase):
         "Integration test requires a live MinIO and MINIO_ACCESS_KEY_ID + MINIO_SECRET_ACCESS_KEY",
     )
     def test_minio(self) -> None:
-        # Endpoint comes from the rig (testcontainers MinIO) via
-        # MINIO_ENDPOINT_URL; falls back to the local run-platform MinIO so a
-        # developer can run this by hand. Real round-trip: create a bucket and
-        # prove it shows up through the access-filtered listing.
+        # Endpoint from the rig's testcontainers MinIO via MINIO_ENDPOINT_URL;
+        # falls back to the local platform MinIO for manual runs.
         self.assertEqual(MinioFS.requires_oauth(), False)
         fs = MinioFS(
             {

workers/tests/test_legacy_executor_scaffold.py

@@ -249,25 +249,6 @@ def test_extraction_error_has_code_and_message(self):
         assert err.message == "extraction failed"
         assert err.code == 500
 
-    def test_no_flask_import(self):
-        """Verify exceptions module does NOT (transitively) import Flask.
-
-        Runs in a fresh interpreter: an in-process check would be polluted
-        by whatever sibling tests already imported, and reloading the
-        module in place would swap class identities under other tests.
-        """
-        import os
-        import subprocess
-        import sys
-
-        code = (
-            "import sys; import executor.executors.exceptions; "
-            "flask = [m for m in sys.modules if m.startswith('flask')]; "
-            "assert not flask, f'Flask modules imported: {flask}'"
-        )
-        env = {**os.environ, "PYTHONPATH": os.pathsep.join(sys.path)}
-        subprocess.run([sys.executable, "-c", code], check=True, env=env)
-
     def test_custom_data_error_signature(self):
         from executor.executors.exceptions import CustomDataError