freerange · floehopper · Jan 9, 2024 · Jan 7, 2024 · Jan 7, 2024 · Jan 7, 2024
diff --git a/Gemfile b/Gemfile
@@ -26,6 +26,7 @@ gem 'rails_autolink', '~> 1.1'
 gem 'redcarpet', '~> 3.6'
 gem 'redis', '~> 5.0'
 gem 'rollbar'
+gem 'rss'
 gem 'rubyzip', '~> 2.3'
 gem 'sidekiq'
 gem 'sprockets-rails'

diff --git a/Gemfile.lock b/Gemfile.lock
@@ -288,6 +288,8 @@ GEM
       io-console (~> 0.5)
     rexml (3.2.6)
     rollbar (3.4.2)
+    rss (0.3.0)
+      rexml
     rubocop (1.59.0)
       json (~> 2.3)
       language_server-protocol (>= 3.17.0)
@@ -391,6 +393,7 @@ DEPENDENCIES
   redcarpet (~> 3.6)
   redis (~> 5.0)
   rollbar
+  rss
   rubocop-capybara
   rubocop-factory_bot
   rubocop-rails

diff --git a/app/views/artists/index.atom.builder b/app/views/artists/index.atom.builder
@@ -0,0 +1,17 @@
+# frozen_string_literal: true
+
+artists = @artists.sort_by(&:first_listed_on).reverse
+
+atom_feed(root_url: artists_url, language: 'en-GB', schema_date: 2024) do |f|
+  f.title 'Artists on jam.coop'
+  f.updated artists.first.updated_at
+  artists.each do |artist|
+    id = "tag:#{request.host},2024:#{artist_path(artist)}"
+    published = artist.first_listed_on
+    f.entry(artist, id:, published:) do |e|
+      e.title artist.name
+      e.author { |a| a.name 'jam.coop' }
+      e.content artist.description, type: 'html'
+    end
+  end
+end
diff --git a/app/views/artists/index.html.erb b/app/views/artists/index.html.erb
@@ -4,6 +4,10 @@
   )
 %>
 
+<% content_for(:auto_discovery) do %>
+  <%= auto_discovery_link_tag(:atom, { format: :atom }, { title: 'Artists on jam.coop' }) %>
+<% end %>
+
 <%= render('shared/page_header', text: 'Artists') %>
 
 <div class="pt-4 grid grid-cols-3 gap-4">

diff --git a/app/views/artists/show.atom.builder b/app/views/artists/show.atom.builder
@@ -0,0 +1,18 @@
+# frozen_string_literal: true
+
+albums = @albums.in_release_order
+
+atom_feed(language: 'en-GB', schema_date: 2024) do |f|
+  f.title "#{@artist.name} albums on jam.coop"
+  f.updated albums.first.released_on
+  albums.each do |album|
+    id = "tag:#{request.host},2024:#{artist_album_path(@artist, album)}"
+    url = artist_album_url(@artist, album)
+    published = album.released_on
+    f.entry(album, id:, url:, published:) do |e|
+      e.title album.title
+      e.author { |a| a.name @artist.name }
+      e.content format_metadata(album.about), type: 'html'
+    end
+  end
+end
diff --git a/app/views/artists/show.html.erb b/app/views/artists/show.html.erb
@@ -5,6 +5,10 @@
   )
 %>
 
+<% content_for(:auto_discovery) do %>
+  <%= auto_discovery_link_tag(:atom, { format: :atom }, { title: "#{@artist.name} albums on jam.coop" }) %>
+<% end %>
+
 <div class="flex flex-row border-b-2 border-b-slate-100 pb-6">
   <div class="basis-1/4 mr-4 m:mr-0">
     <div class="aspect-w-1 aspect-h-1">

diff --git a/app/views/layouts/application.html.erb b/app/views/layouts/application.html.erb
@@ -10,6 +10,7 @@
     <%= stylesheet_link_tag "application", "data-turbo-track": "reload" %>
     <%= javascript_importmap_tags %>
     <%= hotwire_livereload_tags if Rails.env.development? %>
+    <%= content_for(:auto_discovery) %>
   </head>
 
   <body class="bg-slate-100">

diff --git a/test/controllers/artists_controller_test.rb b/test/controllers/artists_controller_test.rb
@@ -142,6 +142,31 @@ class ArtistsControllerTestSignedOut < ActionDispatch::IntegrationTest
     assert_select 'p', { text: @artist.name }
   end
 
+  test '#index includes auto-discovery link for atom feed' do
+    get artists_url
+
+    url = artists_url(format: :atom)
+    assert_select "head link[rel='alternate'][type='application/atom+xml'][href='#{url}']"
+  end
+
+  test '#index with atom format should render atom feed' do
+    @artist.update!(name: 'Older Artist')
+    @artist.albums << create(:album, publication_status: :published)
+
+    another_artist = create(:artist, name: 'Newer Artist')
+    another_artist.albums << create(:album, publication_status: :published)
+
+    create(:artist, name: 'Unlisted Artist')
+
+    get artists_url(format: :atom)
+
+    feed = RSS::Parser.parse(response.body)
+    assert_equal 'Artists on jam.coop', feed.title.content
+    assert_equal 'Newer Artist', feed.entries.first.title.content
+    assert_equal 'Older Artist', feed.entries.last.title.content
+    assert_not_includes feed.entries.map(&:title).map(&:content), 'Unlisted Artist'
+  end
+
   test '#show should include published albums' do
     @artist.albums << create(:album, title: 'Album Title', publication_status: :published)
 
@@ -166,6 +191,29 @@ class ArtistsControllerTestSignedOut < ActionDispatch::IntegrationTest
     assert_select 'p', { text: 'Album Title (pending)', count: 0 }
   end
 
+  test '#show includes auto-discovery link for atom feed' do
+    get artist_url(@artist)
+
+    url = artist_url(@artist, format: :atom)
+    assert_select "head link[rel='alternate'][type='application/atom+xml'][href='#{url}']"
+  end
+
+  test '#show with atom format should render atom feed' do
+    @artist.albums << create(:album, title: 'Older', publication_status: :published, released_on: 2.days.ago)
+    @artist.albums << create(:album, title: 'Newer', publication_status: :published, released_on: 1.day.ago)
+    @artist.albums << create(:album, title: 'Pending', publication_status: :pending, released_on: 0.days.ago)
+    @artist.albums << create(:album, title: 'Unpublished', publication_status: :unpublished, released_on: 0.days.ago)
+
+    get artist_url(@artist, format: :atom)
+
+    feed = RSS::Parser.parse(response.body)
+    assert_equal "#{@artist.name} albums on jam.coop", feed.title.content
+    assert_equal 'Newer', feed.entries.first.title.content
+    assert_equal 'Older', feed.entries.last.title.content
+    assert_not_includes feed.entries.map(&:title).map(&:content), 'Pending'
+    assert_not_includes feed.entries.map(&:title).map(&:content), 'Unpublished'
+  end
+
   test '#new' do
     get new_artist_url
     assert_redirected_to log_in_path