Skip to content

Commit

Permalink
Merge pull request #295 from pulibrary/267-ellipsis
Browse files Browse the repository at this point in the history
Add ellipses character to list of removed punctuation
  • Loading branch information
tpendragon authored Jan 13, 2025
2 parents 6251a4b + f051436 commit 1426ce0
Show file tree
Hide file tree
Showing 2 changed files with 14 additions and 1 deletion.
2 changes: 1 addition & 1 deletion solr/conf/slug.js

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

13 changes: 13 additions & 0 deletions test/dpul_collections/solr_test.exs
Original file line number Diff line number Diff line change
Expand Up @@ -190,6 +190,19 @@ defmodule DpulCollections.SolrTest do
"ديوان-القاضي-ناصح-الدين-ابي"
end

test "slug generation with ellipsis character in title" do
doc = %{
"id" => "3cb7627b-defc-401b-9959-42ebc4488f74",
"title_txtm" => ["Паук семейства СОИ…"]
}

Solr.add([doc], active_collection())
Solr.commit(active_collection())

assert Solr.find_by_id("3cb7627b-defc-401b-9959-42ebc4488f74")["slug_s"] ==
"паук-семейства-сои"
end

test "slug generation with Spanish title" do
doc = %{
"id" => "3cb7627b-defc-401b-9959-42ebc4488f74",
Expand Down

0 comments on commit 1426ce0

Please sign in to comment.