בעיה עם הרצת טסטים שעברו merge

adi1 · 1 בפברואר,‏ 2021,‏ 9:29am

היי,
אני עובדת על קובץ dayview.html של שגיא.
רציתי להריץ את קובץ הבדיקות ששגיא יצר test_dayview.py, אך אני מקבלת את הודעת השגיאה הבאה:
נראה שלא נוצרים לי כל הפיצ’רים אשר מדמים יוזר ואירועים.
עשיתי pull מ develop (דאגתי למחוק לפני את הdb הקודם שהיה)
התקנתי requirements
העתקתי לconfig את התוכן של config.example.
בעת הרצת uvicorn נוצר לי dev.db
ובעת הרצת pytest נוצר לי test.db
הייתי שמחה לעזרה כיצד לפתור את הבעיה.
קישור לקובץ הבדיקות בגיט:

github.com

PythonFreeCourse/calendar/blob/develop/tests/test_dayview.py

from datetime import datetime, timedelta

from bs4 import BeautifulSoup
import pytest

from app.database.models import Event, User
from app.routers.dayview import DivAttributes


# TODO add user session login
@pytest.fixture
def user():
    return User(username='test1', email='user@email.com',
                password='1a2b3c4e5f', full_name='test me')


@pytest.fixture
def event1():
    start = datetime(year=2021, month=2, day=1, hour=7, minute=5)
    end = datetime(year=2021, month=2, day=1, hour=9, minute=15)

This file has been truncated. show original

תודה!

itzik245 · 1 בפברואר,‏ 2021,‏ 4:32pm

לדעתי זה בגלל שה- fixture user קיים כבר בקוד.
תנסי או להשתמש במה שקיים או לשנות את השם של הfixture שלך

itzik245 · 1 בפברואר,‏ 2021,‏ 4:35pm

וגם אולי כדי להפריד את הפיקצרים לקובץ אחר.
תראי בקוד איך עושים את זה, זה יהיה יותר מסודר🙂

adi1 · 1 בפברואר,‏ 2021,‏ 7:02pm

תודה, ניסיתי את מה שהצעת והופיעו אותן שגיאות.
בנוסף, זהו קוד שעבר merge ולאחר שיחה עם שגיא , הבנתי כי הקוד אצלו רץ תקין, ולכן הבעיה צריכה להיות אצלי, אך לצערי אני לא מצליחה לפתור אותה.

Yam · 2 בפברואר,‏ 2021,‏ 3:09am

משונה, טכנית זה באמת אמור לעבוד אם זה עבר merge.

זה נשמע שלא חוזר שום user מהשורה הראשונה ב־dayview. בדקת מה יש בטבלת users?

adi1 · 2 בפברואר,‏ 2021,‏ 7:08am

תודה על העזרה,
בdev.dB קיימת טבלת users עם השדות שצריכים להיות אבל היא ריקה מנתונים.

test.dB נוצרה אך לא נוצרו בה טבלאות כלל.

מה ניתן עוד לעשות?

Yam · 2 בפברואר,‏ 2021,‏ 2:26pm

ב־testdb הנתונים נמחקים אחרי שהטסטים רצים. ניסית למחוק את השורה שמוחקת את הנתונים?

adi1 · 2 בפברואר,‏ 2021,‏ 3:02pm

האם התכוונת לשורה:

Base.metadata.drop_all(bind=sqlite_test_engine)

אם כן, מחקתי והרצתי ועדיין מתקבלות אותן הודעות שגיאה.
אם התכוונת לשורה אחרת , אשמח להסבר נוסף

yuval.cagan · 2 בפברואר,‏ 2021,‏ 3:14pm

נראה לי שהבעיה היא ב-client שמועבר לטסט.
בקובץ client_fixture.py יש מספר קליינטים שונים, כשכל אחד נועד להחליף את ה-dependency שניגש ל-DB.
למעשה הבדיקה מוספיה את היוזר והאיוונטים אל test.db, אך הפונקציה שניבדקת ניגשת ל-dev.db ולכן לא מוצאת את היוזר (שימי לב שהבדיקה נכשלת בעקבות שגיאה בפונקציה המקורית ולא שגיאת assert).
מה יש לעשות אם כך זה ליצור fixture נוסף dayview_test_client לפי השבלונה של שאר הקליינטים עם הקובץ המתאים כפרמטר לפונקציה.
מקווה שזה יפתור את הבעיה

efratush · 2 בפברואר,‏ 2021,‏ 3:18pm

נדחפת לפוסט,
האם בטסט, כשאני מעבירה כפרמטר את client, הוא כולל בתוכו session?
ניסיתי לצור מצב מדומה שאין חיבור לdb. במקום להשתמש ב file_test_client, השתמשתי בclient וחזר לי בהצלחה פעולה שמחייבת שימוש בsession.

Yam · 3 בפברואר,‏ 2021,‏ 3:24am

מצאנו את הבעיה אצל עדי בסשן זום שהתקיים היום עם צוות הפרונטאנד. הבעיה הייתה שינוי לוקאלי באחד הקבצים.
עשינו git stash לשינויים שלה, בדקנו בעזרת git log וראינו שהיו commit¯ים. שמרנו את מספר הקומיט הנוכחי ועשינו checkout ל¯upstream/develop. כשהרצנו שם את הטסטים הכל עבד.
אם יש למישהו/י בעיות נוספות אפשר לפתוח אשכול חדש