CI: שירותי mongo ו-redis אינם נגישים מג'וב Unit Tests

[amirbiron]

התסמין

mongodb:27017: [Errno -3] Temporary failure in name resolution

המארח mongodb אינו נפתר מהג'וב unit-tests. אותו דבר חל על redis.

השורש

הג'וב הוא runs-on: ubuntu-latest בלי container:, ושני השירותים מוגדרים בלי ports: — עם הערה שאומרת "Avoid host port binding to prevent conflicts; job connects via service hostname".

תיעוד GitHub Actions:

"The service running in the Docker container does not expose its ports to the job on the runner by default. You need to map ports on the service container to the Docker host."

"You can access service containers using localhost:<port> or 127.0.0.1:<port>."

גישה לפי שם השירות עובדת רק כשהג'וב עצמו רץ בקונטיינר. כאן הוא לא. כלומר ההערה בקובץ מתארת תצורה שאינה קיימת, והשירותים מעולם לא היו נגישים מהג'וב הזה.

מה זה עלה בפועל

MONGODB_URL ו-REDIS_URL מוגדרים בג'וב אבל מצביעים לשום מקום. כל בדיקה שניסתה להתחבר באמת דילגה או נכשלה — ולכן איש לא שם לב.

docs/testing.rst טען ש-tests/test_note_boards_mongo.py "רצות באמת ב-CI". 33 הבדיקות שם מדלגות בשקט מאז ומעולם. המשפט תוקן ב-5cbfcc0, יחד עם דילוג-על-שרת-לא-נגיש בפיקסצ'ר wired_mongo שמונע נפילת חבילה.

התיקון

יש תקדים נכון באותו קובץ — ג'וב alembic-migrations (ci.yml:454-455):

    services:
      postgres:
        image: postgres:16
        ports:
          - 5432:5432

הצעד:

1. להוסיף ports: לשירותי mongodb ו-redis בג'וב unit-tests
2. לשנות את MONGODB_URL, REDIS_URL ו-NOTE_FONTS_TEST_MONGO_URI מ-mongodb/redis ל-127.0.0.1
3. לתקן את ההערה בקובץ, שאומרת היום את ההפך מהמצב
4. לעדכן את האזהרה ב-docs/testing.rst — היא כתובה כך שתיהפך לנכונה אחרי התיקון

NOTE_FONTS_TEST_MONGO_URI כבר מוגדר בג'וב, עם הערה שמסבירה שהוא ממתין לתיקון הזה. ברגע שהכתובת תשתנה הבדיקות יתחילו לרוץ בלי לזכור דבר.

⚠️ למה זה PR נפרד

התיקון יעיר 33 בדיקות ב-tests/test_note_boards_mongo.py שמדלגות היום, ואי אפשר לדעת מראש אילו מהן עוברות — הן לא רצו מעולם ב-CI. ייתכן גם שבדיקות נוספות שגודרו על MONGODB_URL יתעוררו.

זה רצוי — כיסוי אמיתי במקום דיווח שווא — אבל זה סבב משלו, ואין לערבב אותו עם PR של פיצ'ר.

בדיקה שהתיקון עבד

לא מספיק ש-CI ירוק: ירוק הוא גם מה שרואים היום, כשהכול מדלג. הראיה היא שמספר ה-skipped יורד ומספר ה-passed עולה — ושהבדיקות ב-test_note_boards_mongo.py מופיעות כ-PASSED ולא כ-SKIPPED.

[amirbiron]

חוב:

Check if this issue is valid — if so, understand the root cause and fix it. At webapp/app.py, line 15127:

<comment>חומרה: בינונית (4/10). כשלקוח שולח רק `If-Modified-Since`, `md_preview` מחזיר כעת 200 בכל בקשה אף שהוא ממשיך לשלוח `Last-Modified`. השאר בדיקת `If-Modified-Since`, או הפסק לשלוח `Last-Modified` במסלול הזה עד שיהיה תאריך שמייצג גם שינויי העדפות.</comment>

<file context>
@@ -15100,18 +15124,23 @@ def md_preview(file_id):
-            resp.headers['ETag'] = etag
-            resp.headers['Last-Modified'] = last_modified_str
-            return resp
+    # **``If-Modified-Since`` אינו מכריע כאן, לפי התקן עצמו.**
+    #
+    # RFC 9110 §13.1.3: *"A recipient MUST ignore the If-Modified-Since
</file context>

[amirbiron]

.

נקודה אחת שכדאי להכין אליה מראש ב-PR:
כעת, כשהבדיקות באמת ינסו להתחבר ל-127.0.0.1, ייתכן שתיתקל בבעיות תזמון (Race Conditions). לעיתים קרובות ב-CI, הקונטיינר של MongoDB או Redis מוגדר כ-Running, אבל ה-Service בתוכו עדיין עושה Boot ולא מוכן לקבל חיבורים כשהטסט הראשון מזנק.
אם פתאום תראה נפילות בגלל Connection Refused ספורדי, כדאי לשקול להוסיף מנגנון healthcheck תחת הגדרות ה-options של השירותים ב-YAML, כך שהג'וב ימתין לוודא שהדאטה-בייס באמת למעלה לפני שהוא מעביר את השרביט לריצת הטסטים. לדוגמה:

    services:
      mongodb:
        image: mongo
        ports:
          - 27017:27017
        options: >-
          --health-cmd "echo 'db.runCommand("ping").ok' | mongosh localhost:27017/test --quiet"
          --health-interval 10s
          --health-timeout 5s
          --health-retries 5

[amirbiron]

תוספת מ-PR #3502: עוד קובץ שמדלג היום ב-CI ויתעורר עם התיקון הזה, tests/test_mcp_content_sha256_real_mongo.py. ב-#3502 הוא גדל ל-43 בדיקות: 12 המקרים הקשים של content_sha256, עכשיו גם דרך upload_id, בדיקת האינדקסים של mcp_uploads, ושער המחיקה מול delete_one אמיתי. הן רצו מקומית, 43/43 מול mongod 8.0.15 ב-30.09, ולא ב-CI.

למי שמריץ אותן מקומית עד אז: צריך גם MONGODB_URL שמצביע לאותו mongod, ולא רק NOTE_FONTS_TEST_MONGO_URI. הסיבה היא #3486, והמדידה שם.

---

Generated by Claude Code