Karavel/fixin.me
0
0
Fork 0
forked from fixin.me/fixin.me
Code Pull Requests Activity

Equally sample add unit/add subunit/edit links for test

Browse Source
This commit is contained in:
cryptogopher 2024-12-12 00:35:11 +01:00
parent bb4fbb3adc
commit 3a25c1dbd0
1 changed files with 25 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

35
test/system/units_test.rb
View File

@ -55,24 +55,39 @@ class UnitsTest < ApplicationSystemTestCase
test "add and edit disallow opening multiple forms" do test "add and edit disallow opening multiple forms" do
# Once new/edit form is open, attempt to open another one will close it # Once new/edit form is open, attempt to open another one will close it
links = {} links = {}
# Define tr count change depending on link clicked targets = {}
link_labels = {1 => ADD_UNIT_LABELS, 0 => units.map(&:symbol)} {
link_labels.each_pair do |row_change, labels| add_unit: t('units.index.add_unit'),
all(:link_or_button, exact_text: Regexp.union(labels)).map { |l| links[l] = row_change } add_subunit: t('units.unit.add_subunit'),
edit: Regexp.union(units.map(&:symbol))
}.each_pair do |type, labels|
links[type] = all(:link_or_button, exact_text: labels).to_a
targets[type] = links[type].sample
end end
# Define tr count change depending on link clicked
row_change = {add_unit: 1, add_subunit: 1, edit: 0}
link, rows = links.assoc(links.keys.sample).tap { |l, _| links.delete(l) } type, link = targets.assoc(targets.keys.sample).tap { |t, _| targets.delete(t) }
rows = row_change[type]
assert_difference ->{ all('tbody tr').count }, rows do assert_difference ->{ all('tbody tr').count }, rows do
link.click link.click
end end
find('tbody tr:has(input[type=text])').assert_selector ':focus' within('tbody tr:has(input[type=text])') { assert_selector ':focus' }
assert !link.visible? || link[:disabled] if type == :edit
assert !link.visible?
[:add_subunit, :edit].each do |t|
assert_difference(->{ links[t].length }, -1) { links[t].select!(&:visible?) }
end
else
assert link[:disabled]
end
link = links.keys.select(&:visible?).sample targets.merge([:add_subunit, :edit].map { |t| [t, links[t].sample] }.to_h)
assert_difference ->{ all('tbody tr').count }, links[link] - rows do type, link = targets.assoc(targets.keys.sample)
assert_difference ->{ all('tbody tr').count }, row_change[type] - rows do
link.click link.click
end end
find('tbody tr:has(input[type=text])').assert_selector ':focus' within('tbody tr:has(input[type=text])') { assert_selector ':focus' }
end end
# NOTE: extend with any add/edit link # NOTE: extend with any add/edit link