Re-add asset identifier to be picked up in tests

seek4science · Sep 27, 2023 · fdb9319 · fdb9319
1 parent f85083d
commit fdb9319
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/app/views/assets/batch_selection/_asset_row.html.erb b/app/views/assets/batch_selection/_asset_row.html.erb
@@ -34,7 +34,7 @@
   end
 -%>
 <div class="batch-selection-asset batch-selection-scope">
-  <div class="isa-tree batch-selection-asset-row <%= html_classes -%> <%= tree_class %>">
+  <div class="isa-tree batch-selection-asset-row <%= html_classes -%> <%= tree_class %>" data-asset-id="<%= item_id -%>">
     <%= batch_selection_collapse_toggle if children.any? %>
 
     <%= render partial: 'assets/batch_selection/checkbox',

diff --git a/test/functional/publishing/single_publishing_test.rb b/test/functional/publishing/single_publishing_test.rb
@@ -162,15 +162,15 @@ def setup
 
     # split-button dropdown menu shown for tree branches
     should_have_dropdown.each do |asset|
-      assert_select ".isa-tree.#{asset.class.name}_#{asset.id}", count: 1 do
+      assert_select '.isa-tree[data-asset-id=?]', "#{asset.class.name}_#{asset.id}", count: 1 do
         assert_select '.parent-btn-dropdown', count: 1
         assert_select '.dropdown-menu', count: 1 do
           assert_select 'li', count: 2 do
-            assert_select 'a.selectChildren', text: /Select this item and all of its sub-items./, count: 1 do
+            assert_select 'a.batch-selection-select-children', text: /Select this item and all of its sub-items./, count: 1 do
               assert_select 'img[src=?]', '/assets/checkbox_select_all.svg'
             end
 
-            assert_select 'a.deselectChildren', text: /Deselect this item and all of its sub-items./, count: 1 do
+            assert_select 'a.batch-selection-deselect-children', text: /Deselect this item and all of its sub-items./, count: 1 do
               assert_select 'img[src=?]', '/assets/checkbox_deselect_all.svg'
             end
           end
@@ -179,7 +179,7 @@ def setup
     end
     # split-button dropdown menu not shown for tree leafs
     should_not_have_dropdown.each do |asset|
-      assert_select ".isa-tree.#{asset.class.name}_#{asset.id}", count: 1 do
+      assert_select '.isa-tree[data-asset-id=?]', "#{asset.class.name}_#{asset.id}", count: 1 do
         assert_select '.parent-btn-dropdown', count: 0
         assert_select '.dropdown-menu', count: 0
       end