Merge pull request #30291 from DanielYankura/overwriting_existing_nodeset

Changes several tests so that they still pass with new libmesh feature

Author: GiudGiud

framework/src/meshgenerators/AdvancedExtruderGenerator.C

@@ -491,10 +491,12 @@ AdvancedExtruderGenerator::generate()
           boundary_info.add_node(new_node, ids_to_copy);
         else
           for (const auto & id_to_copy : ids_to_copy)
+          {
             boundary_info.add_node(new_node,
                                    _boundary_swap_pairs[e].count(id_to_copy)
                                        ? _boundary_swap_pairs[e][id_to_copy]
                                        : id_to_copy);
+          }
 
         old_distance = current_distance;
         current_node_layer++;

test/tests/mesh/named_entities/named_entities.e

@@ -126,7 +126,7 @@ left_side	 top_right	 right_side
 23 5 3 
 2	 # nodeset id to name map
 2	 # vector length
-1	 2	 	 
+1	 3	 	 
 2	 # vector length
 left_nodes	 right_nodes	 	 
 18	 # number of nodesets
@@ -139,12 +139,12 @@ left_nodes	 right_nodes
 12 1 
 13 1 
 16 1 
-54 2 
-55 2 
-56 2 
-57 2 
-58 2 
-59 2 
-60 2 
-61 2 
-62 2 
+54 3 
+55 3 
+56 3 
+57 3 
+58 3 
+59 3 
+60 3 
+61 3 
+62 3 

test/tests/mesh/named_entities/named_entities.xda

@@ -120,6 +120,7 @@
     requirement = 'The system shall be able to retain and swap boundary ids during extrusion.'
     design = 'AdvancedExtruderGenerator.md'
     issues = '#21852'
+    skip = 'Pending libmesh PR #4121 being merged'
   []
   [helicoidal]
     type = Exodiff