test: add tests for article archiving and blog ordering

Author: k0va1

test/controllers/admin/articles_controller_test.rb

@@ -78,4 +78,24 @@ class Admin::ArticlesControllerTest < ActionDispatch::IntegrationTest
     end
     assert_redirected_to admin_articles_path
   end
+
+  test "archive" do
+    assert_nil @article.archived_at
+    patch archive_admin_article_path(@article)
+    assert_redirected_to admin_article_path(@article)
+    assert_not_nil @article.reload.archived_at
+  end
+
+  test "unarchive" do
+    @article.update!(archived_at: Time.current)
+    patch unarchive_admin_article_path(@article)
+    assert_redirected_to admin_article_path(@article)
+    assert_nil @article.reload.archived_at
+  end
+
+  test "index shows archived articles at the end" do
+    articles(:martians_article).update!(archived_at: Time.current)
+    get admin_articles_path
+    assert_response :success
+  end
 end

test/controllers/admin/blogs_controller_test.rb

@@ -86,4 +86,10 @@ class Admin::BlogsControllerTest < ActionDispatch::IntegrationTest
     blog = Blog.last
     assert_equal ["ruby", "rails", "web"], blog.tags
   end
+
+  test "index excludes archived articles from blog ordering" do
+    articles(:martians_article).update!(archived_at: Time.current)
+    get admin_blogs_path
+    assert_response :success
+  end
 end

test/models/article_test.rb

@@ -51,4 +51,53 @@ class ArticleTest < ActiveSupport::TestCase
     assert_includes featured, articles(:featured_article)
     assert_not_includes featured, articles(:rails_performance)
   end
+
+  test "archived? returns true when archived_at is set" do
+    article = articles(:rails_performance)
+    assert_not article.archived?
+
+    article.archive!
+    assert article.archived?
+  end
+
+  test "archive! sets archived_at" do
+    article = articles(:rails_performance)
+    assert_nil article.archived_at
+
+    article.archive!
+    assert_not_nil article.reload.archived_at
+  end
+
+  test "unarchive! clears archived_at" do
+    article = articles(:rails_performance)
+    article.update!(archived_at: Time.current)
+
+    article.unarchive!
+    assert_nil article.reload.archived_at
+  end
+
+  test "archived scope returns only archived articles" do
+    article = articles(:rails_performance)
+    article.update!(archived_at: Time.current)
+
+    assert_includes Article.archived, article
+    assert_not_includes Article.not_archived, article
+  end
+
+  test "not_archived scope excludes archived articles" do
+    article = articles(:rails_performance)
+    assert_includes Article.not_archived, article
+
+    article.update!(archived_at: Time.current)
+    assert_not_includes Article.not_archived, article
+  end
+
+  test "archived_last scope puts archived articles at the end" do
+    recent = articles(:martians_article)
+    older = articles(:rails_performance)
+    recent.update!(archived_at: Time.current)
+
+    results = Article.archived_last.by_publish_date.to_a
+    assert results.index(older) < results.index(recent)
+  end
 end