Make headline images explicit, add asset credits

- related_assets gains a `headline` boolean (DB-enforced: at most one
  per page), replacing "first image by position" as the headline
  rule. A rake task backfills the current first image on every live
  head/draft, so nothing changes visually until an editor changes it.
- The image picker sidebar gets a star toggle reflecting the flag;
  the TinyMCE inline-image picker's badge now reads it too, instead
  of assuming position 0.
- No headline chosen (or none attached) now falls back to the
  gallery-count caption itself becoming the lightbox trigger, instead
  of the gallery being unreachable.
- Assets gain creator, source_url, and license_key (against a new
  config/asset_licenses.yml dictionary). asset_credit renders a
  degrading attribution line, reused as a hidden per-image glightbox
  caption so credit is one click away for every image, not only the
  headline's always-visible one.
- Fixed: asset thumbnails rendered unconditionally regardless of
  whether a real variant exists on disk. Asset#has_variant? checks
  file existence, not content type -- some legacy PDFs have real
  pre-rewrite thumbnails a content-type check would have hidden.
- assets#new/edit rebuilt onto the same node_description/node_content
  layout as assets#show, picking up the three new fields in the
  process.

1 files changed, 76 insertions, 2 deletions

diff --git a/test/models/helpers/content_helper_test.rb b/test/models/helpers/content_helper_test.rb
index a7ed478d..d6c7b438 100644
--- a/test/models/helpers/content_helper_test.rb
+++ b/test/models/helpers/content_helper_test.rb
@@ -2,7 +2,81 @@ require 'test_helper'
 
 class ContentHelperTest < ActionView::TestCase
   test "weekday_abbr delegates through the current I18n locale" do
-    I18n.locale = :de
-    assert_equal "Mo", weekday_abbr(Time.parse("2026-07-06"))
+    I18n.with_locale(:de) do
+      assert_equal "Mo", weekday_abbr(Time.parse("2026-07-06"))
+    end
+  end
+
+  test "asset_credit returns nil for a nil asset" do
+    assert_nil asset_credit(nil)
+  end
+
+  test "asset_credit returns nil when creator, source, and license are all blank" do
+    assert_nil asset_credit(Asset.new(:name => "blank"))
+  end
+
+  test "asset_credit renders creator, source link, and license link when all are present" do
+    asset = Asset.new(:name => "demo", :creator => "Jane Doe",
+                       :source_url => "https://example.org/photo", :license_key => "cc_by_4")
+
+    result = I18n.with_locale(:en) { asset_credit(asset) }
+
+    assert_match %r{<a href="https://example.org/photo">Photo demo</a>}, result
+    assert_match "by Jane Doe", result
+    assert_match %r{<a href="https://creativecommons.org/licenses/by/4.0/">Licensed under CC BY 4.0</a>}, result
+  end
+
+  test "asset_credit falls back to plain text when source_url is blank" do
+    asset = Asset.new(:name => "demo", :creator => "Jane Doe", :license_key => "cc_by_4")
+    assert_no_match %r{<a href="[^"]*">Photo demo</a>}, I18n.with_locale(:en) { asset_credit(asset) }
+  end
+
+  test "asset_credit omits the 'by' clause when creator is blank" do
+    asset = Asset.new(:name => "demo", :source_url => "https://example.org/photo", :license_key => "cc_by_4")
+    assert_no_match "by ", I18n.with_locale(:en) { asset_credit(asset) }
+  end
+
+  test "a note-style license renders with no 'Licensed under' prefix and no link" do
+    asset = Asset.new(:name => "demo", :creator => "CCC", :license_key => "own_work")
+    result = I18n.with_locale(:en) { asset_credit(asset) }
+
+    assert_match "Own work", result
+    assert_no_match "Licensed under", result
+    assert_no_match "<a", result
+  end
+
+  test "asset_credit degrades gracefully when the license_key is no longer in the dictionary" do
+    asset = Asset.new(:name => "demo", :creator => "Jane Doe")
+    asset.license_key = "a_retired_key"
+
+    result = I18n.with_locale(:en) { asset_credit(asset) }
+
+    assert_match "Photo demo by Jane Doe", result
+    assert_no_match "Licensed under", result
+  end
+
+  test "headline_image renders nothing when no images are attached" do
+    node = Node.root.children.create!(:slug => "headline_image_empty_test")
+    @page = node.draft
+    assert_nil headline_image
+  end
+
+  test "headline_image renders the flagged headline asset" do
+    node = Node.root.children.create!(:slug => "headline_image_flagged_test")
+    asset = Asset.create!(:name => "flagged", :upload_content_type => "image/png")
+    node.draft.assets << asset
+    node.draft.related_assets.find_by(:asset_id => asset.id).update!(:headline => true)
+    @page = node.draft
+
+    assert_match "glightbox", headline_image
+  end
+
+  test "headline_image falls back to a gallery-open caption when no headline is flagged" do
+    node = Node.root.children.create!(:slug => "headline_image_no_flag_test")
+    asset = Asset.create!(:name => "unflagged", :upload_content_type => "image/png")
+    node.draft.assets << asset
+    @page = node.draft
+
+    I18n.with_locale(:en) { assert_match t(:open_gallery), headline_image }
   end
 end