diff options
| author | erdgeist <erdgeist@erdgeist.org> | 2026-07-20 19:45:43 +0200 |
|---|---|---|
| committer | erdgeist <erdgeist@erdgeist.org> | 2026-07-20 19:45:43 +0200 |
| commit | 836308471f8d31ccdcdd3a5bd88bc76cc1c0831b (patch) | |
| tree | 821a526d4d10870654a38d07a4eb53a2180ba313 /test/models/asset_license_test.rb | |
| parent | d89e2de48a5e16ff01ec627e79c76b833e4618cf (diff) | |
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.
Diffstat (limited to 'test/models/asset_license_test.rb')
| -rw-r--r-- | test/models/asset_license_test.rb | 31 |
1 files changed, 31 insertions, 0 deletions
diff --git a/test/models/asset_license_test.rb b/test/models/asset_license_test.rb new file mode 100644 index 00000000..9f2d04f8 --- /dev/null +++ b/test/models/asset_license_test.rb | |||
| @@ -0,0 +1,31 @@ | |||
| 1 | require 'test_helper' | ||
| 2 | |||
| 3 | class AssetLicenseTest < ActiveSupport::TestCase | ||
| 4 | test "find returns a populated entry for a known key" do | ||
| 5 | entry = AssetLicense.find("cc_by_4") | ||
| 6 | |||
| 7 | assert_equal "cc_by_4", entry.key | ||
| 8 | assert_equal "https://creativecommons.org/licenses/by/4.0/", entry.url | ||
| 9 | assert entry.requires_attribution | ||
| 10 | assert_equal "license", entry.style | ||
| 11 | end | ||
| 12 | |||
| 13 | test "find returns nil for an unknown or blank key" do | ||
| 14 | assert_nil AssetLicense.find("not_a_real_license") | ||
| 15 | assert_nil AssetLicense.find(nil) | ||
| 16 | assert_nil AssetLicense.find("") | ||
| 17 | end | ||
| 18 | |||
| 19 | test "keys includes every dictionary entry" do | ||
| 20 | assert_includes AssetLicense.keys, "cc0" | ||
| 21 | assert_includes AssetLicense.keys, "own_work" | ||
| 22 | end | ||
| 23 | |||
| 24 | test "a note-style entry has no attribution requirement and no url" do | ||
| 25 | entry = AssetLicense.find("own_work") | ||
| 26 | |||
| 27 | assert_not entry.requires_attribution | ||
| 28 | assert_nil entry.url | ||
| 29 | assert_equal "note", entry.style | ||
| 30 | end | ||
| 31 | end | ||
