Postにメディア情報を紐づける

[sushichan044]

closes #71

やりたいこと

API側の GET /posts において、media クエリを用いて必要なら画像や動画などのメディア情報を取得できるようにする

実装詳細

• 既に仮置きされていた MediaDetails を変更した
  • list[HttpUrl] | None -> list[XMedia]
  • Noneを除去しないと SQL Alchemy側でリレーションが組めない
• DB的には Post.id と Media.media_key の N:N relationとして格納している
  • 命名を media に統一するか xmediaに統一するかは意見をいただきたいです

Warning

API側で、 media=false のテストを書いた際に、 pydantic のモデル周りの罠を踏んでおりテストが壊れています
#113 をMergeすると治ります

todo

• common側のテスト追加
• api側のテスト追加
• 発行されるSQLの調査 → 完了

[sushichan044]

9666eb6 時点で手元の tox は通っているが CI が落ちている

[yu23ki14]

命名を media に統一するか xmediaに統一するかは意見をいただきたいです

mediaに統一でお願いします！

[sushichan044]

mediaに統一でお願いします！

承知です！ありがとうございます

[sushichan044]

memo Post.media_details にアクセスしたときに PostMedia中間テーブルにどういうSQLが飛んでいるか確認しておく。
理想は関連するMediaRecordまで一気にJOINされている状態だが、N回クエリを飛ばしていそうなら対応する

[sushichan044]

memo
test_posts_get_with_media_false のみテストが落ちる。
pydanticのmodel copyをした際に参照が変わるが、その際にlinkが変わっている？

Full diff:
    {
        'data': [
            {
                'createdAt': 1153921700000,
                'impressionCount': 30,
                'likeCount': 10,
  -             'link': 'http://localhost/',
  +             'link': 'https://x.com/User1/status/2234567890123456791',
                'mediaDetails': [],
                'postId': '2234567890123456791',
                'repostCount': 20,
                'text': 'このブログ記事、めちゃくちゃ参考になった！🔥 チェックしてみて！\n'
                '\n'
                'https://t.co/yyyyyyyyyyy/ #学び #自己啓発',
                'xUser': {
                    'followersCount': 100,
                    'followingCount': 200,
                    'name': 'User1',
                    'profileImage': 'http://localhost/',
                    'userId': '1234567890123456781',
                },
                'xUserId': '1234567890123456781',
            },
        ],
        'meta': {
            'next': None,
            'prev': None,
        },
    }

[sushichan044]

そもそもlinkは XUser.id と post_id から計算できるので、 Computed Fieldにした方が良さそう
ref: https://docs.pydantic.dev/latest/concepts/fields/#the-computed_field-decorator

=> 修正PR作成済み #114

[sushichan044]

Post.media_details にアクセスした際に発生するクエリは以下のよう。
おそらく意図通りアクセスした時点で関連するメディアを1回のクエリで取得できている。
N+1 にはなっていないとみられる。

INFO:sqlalchemy.engine.Engine:SELECT post_media.post_id AS post_media_post_id, post_media.media_key AS post_media_media_key, media_1.media_key AS media_1_media_key, media_1.type AS media_1_type, media_1.url AS media_1_url, media_1.width AS media_1_width, media_1.height AS media_1_height 
FROM post_media LEFT OUTER JOIN media AS media_1 ON media_1.media_key = post_media.media_key 
WHERE %(param_1)s = post_media.post_id
INFO:sqlalchemy.engine.Engine:[cached since 0.001606s ago] {'param_1': '2234567890123456801'}