From d7123fb153d6fca3f5d792c299c971eb91c1cc4d Mon Sep 17 00:00:00 2001
From: Julia Lange <public@julialange.org>
Date: Wed, 21 Jan 2026 13:04:51 -0800
Subject: [PATCH] Finalize feed, tests which don't need channel creation

---
 koucha/src/feed.rs | 78 ++++++++++++++++++++++++++++++++++++++++------
 1 file changed, 68 insertions(+), 10 deletions(-)

diff --git a/koucha/src/feed.rs b/koucha/src/feed.rs
index 1a38d91..56097fb 100644
--- a/koucha/src/feed.rs
+++ b/koucha/src/feed.rs
@@ -36,35 +36,58 @@ impl Feed {
   pub fn id(&self) -> FeedId { self.id }
   pub fn title(&self) -> &str { &self.title }
 
-  pub async fn create(
-    pool: &SqlitePool, id: UserId, name: &str
+  pub async fn get(
+    pool: &SqlitePool, id: FeedId
   ) -> Result<Self> {
-    let int_id = i64::from(id);
+    let feed = sqlx::query_as!(
+      UnparsedFeed,
+      "SELECT id, title FROM feeds WHERE id = ?",
+      id.0
+    ).fetch_one(pool).await?.parse();
+
+    feed
+  }
+
+  pub async fn create(
+    pool: &SqlitePool, user_id: UserId, title: &str
+  ) -> Result<Self> {
+    let int_id = i64::from(user_id);
     let new_feed = sqlx::query_as!(
       UnparsedFeed,
       "INSERT INTO feeds (user_id, title) 
        VALUES (?, ?)
        RETURNING id as `id!`, title",
-      int_id, name
+      int_id, title
     ).fetch_one(pool).await?.parse();
 
     new_feed
   }
 
+  pub async fn update_title(
+    pool: &SqlitePool, id: FeedId, new_title: &str
+  ) -> Result<()> {
+    sqlx::query!(
+      "UPDATE feeds SET title = ? WHERE id = ?",
+      new_title, id.0
+    ).execute(pool).await?;
+
+    Ok(())
+  }
+
   pub async fn add_channel(
-    pool: &SqlitePool, id: FeedId, channel_id: ChannelId
+    &self, pool: &SqlitePool, channel_id: ChannelId
   ) -> Result<()> {
     sqlx::query!(
       "INSERT INTO feed_channels (feed_id, channel_id)
        VALUES (?, ?)",
-      id.0, channel_id.0
+      self.id.0, channel_id.0
     ).execute(pool).await?;
 
     Ok(())
   }
 
   pub async fn get_items(
-    pool: &SqlitePool, id: FeedId, limit: u8, offset: u32
+    &self, pool: &SqlitePool, limit: u8, offset: u32
   ) -> Result<Vec<Item>> {
     let items = sqlx::query_as!(
       Item,
@@ -72,14 +95,14 @@ impl Feed {
        WHERE feed_id = ? AND archived = FALSE
        ORDER BY score DESC
        LIMIT ? OFFSET ?",
-      id.0, limit, offset
+      self.id.0, limit, offset
     ).fetch_all(pool).await?;
 
     Ok(items)
   }
 
   pub async fn get_channels(
-    pool: &SqlitePool, id: FeedId
+    &self, pool: &SqlitePool
   ) -> Result<Vec<Channel>> {
     let channels: Result<Vec<Channel>> = sqlx::query_as!(
       UnparsedChannel,
@@ -87,7 +110,7 @@ impl Feed {
        FROM channels c
        JOIN feed_channels fc on c.id = fc.channel_id
        WHERE fc.feed_id = ?",
-      id.0
+      self.id.0
     ).fetch_all(pool).await?.into_iter()
       .map(UnparsedChannel::parse).collect();
 
@@ -95,3 +118,38 @@ impl Feed {
   }
 }
 
+#[cfg(test)]
+mod tests {
+  use super::*;
+  use crate::User;
+  use sqlx::SqlitePool;
+
+  async fn setup_test_db() -> SqlitePool {
+    let pool = SqlitePool::connect("sqlite::memory:").await.unwrap();
+    sqlx::migrate!().run(&pool).await.unwrap();
+    pool
+  }
+
+  #[tokio::test]
+  async fn create_feed() {
+    let pool = setup_test_db().await;
+    let user = User::create(&pool, "Alice").await.unwrap();
+
+    let feed = Feed::create(&pool, user.id(), "Tech News").await.unwrap();
+
+    assert_eq!(feed.title(), "Tech News");
+    assert!(feed.id().0 > 0);
+  }
+
+  #[tokio::test]
+  async fn test_update_title() {
+    let pool = setup_test_db().await;
+    let user = User::create(&pool, "Alice").await.unwrap();
+    let feed = Feed::create(&pool, user.id(), "Tech News").await.unwrap();
+
+    Feed::update_title(&pool, feed.id(), "Technology").await.unwrap();
+
+    let updated = Feed::get(&pool, feed.id()).await.unwrap();
+    assert_eq!(updated.title(), "Technology");
+  }
+}