Handle special characters in "target" completion

Hyphens should be suppressed and underscores are to be taken as pairs
which contain the real value to be picked.

Signed-off-by: Miquel Sabaté Solà <msabate@suse.com>

1 files changed, 25 insertions, 3 deletions

diff --git a/test/system/things_test.rb b/test/system/things_test.rb
index dafe45d..bb9d4ef 100644
--- a/test/system/things_test.rb
+++ b/test/system/things_test.rb
@@ -121,7 +121,7 @@ class ThingsTest < ApplicationSystemTestCase
     assert_text I18n.t('activerecord.attributes.thing.access')
   end
 
-  test '"target" gets generated from last name' do
+  test 'js: "target" gets generated from last name' do
     visit new_thing_url
 
     fill_in I18n.t('activerecord.attributes.thing.authors'), with: 'Name LastName'
@@ -131,7 +131,7 @@ class ThingsTest < ApplicationSystemTestCase
     assert_equal 'LastName', text
   end
 
-  test '"target" gets generated from last name and year' do
+  test 'js: "target" gets generated from last name and year' do
     visit new_thing_url
 
     fill_in I18n.t('activerecord.attributes.thing.authors'), with: 'Name LastName'
@@ -142,7 +142,7 @@ class ThingsTest < ApplicationSystemTestCase
     assert_equal 'LastName2024', text
   end
 
-  test '"target" gets generated from multiple names and year' do
+  test 'js: "target" gets generated from multiple names and year' do
     visit new_thing_url
 
     fill_in I18n.t('activerecord.attributes.thing.authors'), with: 'Name LastName, John Smith'
@@ -152,4 +152,26 @@ class ThingsTest < ApplicationSystemTestCase
 
     assert_equal 'LastNameSmith2024', text
   end
+
+  test 'js: hyphenated names are treated together on "target" automation' do
+    visit new_thing_url
+
+    fill_in I18n.t('activerecord.attributes.thing.authors'), with: 'Name Last-Other, John Smith'
+    fill_in I18n.t('activerecord.attributes.thing.year'), with: '2024'
+
+    text = find_field(I18n.t('activerecord.attributes.thing.target')).value
+
+    assert_equal 'LastOtherSmith2024', text
+  end
+
+  test 'js: names surrounded by underscores are treated together on "target" automation' do
+    visit new_thing_url
+
+    fill_in I18n.t('activerecord.attributes.thing.authors'), with: 'Name _Last Other_, John Smith'
+    fill_in I18n.t('activerecord.attributes.thing.year'), with: '2024'
+
+    text = find_field(I18n.t('activerecord.attributes.thing.target')).value
+
+    assert_equal 'LastOtherSmith2024', text
+  end
 end