Add a shard:group# tag to tests for slow runners

Adds a shard:group[1-4] tag to each test, placing them into four
distinct groups so that they can be run in parallel on separate gha
instances when we're running slowly under qemu for arm64 testing.

The tagging was done by a script just striping the alphabetic list of
test file names. Four groupings was based on their being 225 tests, and
the fastest runner in gha getting to ~70 of them before getting killed
by the six hour run limit.

Signed-off-by: Josh Partlow <jpartlow@glatisant.org>

Author: jpartlow

acceptance/tests/agent/agent_disable_lockfile.rb

@@ -3,7 +3,8 @@
                          # The validation of the `vardir` at the OS level
                          # should be accomplished in another test.
     'audit:high',
-    'audit:refactor'     # This test should not require a master. Remove the use of `with_puppet_running_on`.
+    'audit:refactor',    # This test should not require a master. Remove the use of `with_puppet_running_on`.
+    'shard:group1' # For splitting out groups of tests for slow test runners
 
 #
 # This test is intended to ensure that puppet agent --enable/--disable

acceptance/tests/agent/agent_fails_with_unknown_resource.rb

@@ -1,6 +1,7 @@
 test_name "agent run should fail if it finds an unknown resource type" do
   tag 'audit:high',
-      'audit:integration'
+      'audit:integration',
+      'shard:group2' # For splitting out groups of tests for slow test runners
 
   require 'puppet/acceptance/common_utils'
 

acceptance/tests/agent/agent_parses_json_catalog.rb

@@ -4,7 +4,8 @@
     'audit:high',        # tests defined catalog format
     'audit:integration', # There is no OS specific risk here.
     'server',
-    'catalog:json'
+    'catalog:json',
+    'shard:group3' # For splitting out groups of tests for slow test runners
 
 require 'puppet/acceptance/common_utils'
 require 'json'

acceptance/tests/agent/fallback_to_cached_catalog.rb

@@ -2,7 +2,8 @@
 
 tag 'audit:high',
     'audit:integration', # This test is not OS sensitive.
-    'audit:refactor'     # A catalog fixture can be used for this test. Remove the usage of `with_puppet_running_on`.
+    'audit:refactor',    # A catalog fixture can be used for this test. Remove the usage of `with_puppet_running_on`.
+    'shard:group4' # For splitting out groups of tests for slow test runners
 
 step "run agents once to cache the catalog" do
   with_puppet_running_on master, {} do

acceptance/tests/agent/last_run_summary_report.rb

@@ -1,5 +1,6 @@
 test_name "The 'last_run_summary.yaml' report has the right location and permissions" do
-  tag 'audit:high'
+  tag 'audit:high',
+      'shard:group1' # For splitting out groups of tests for slow test runners
 
   require 'puppet/acceptance/temp_file_utils'
   extend Puppet::Acceptance::TempFileUtils

acceptance/tests/aix/aix_package_provider.rb

@@ -1,7 +1,8 @@
 test_name "aix package provider should work correctly" do
 
   tag 'audit:high',
-      'audit:acceptance'  # OS specific by definition.
+      'audit:acceptance', # OS specific by definition.
+      'shard:group2' # For splitting out groups of tests for slow test runners
 
   confine :to, :platform => /aix/
 

acceptance/tests/aix/nim_package_provider.rb

@@ -1,7 +1,8 @@
 test_name "NIM package provider should work correctly"
 
 tag 'audit:high',
-    'audit:acceptance'  # OS specific by definition
+    'audit:acceptance', # OS specific by definition
+    'shard:group3' # For splitting out groups of tests for slow test runners
 
 # nim test is slow, confine to only aix 7.2 and recent puppet versions
 confine :to, :platform => "aix" do |aix|

acceptance/tests/allow_arbitrary_node_name_fact_for_agent.rb

@@ -3,7 +3,8 @@
 tag 'audit:high',
     'audit:integration',  # Tests that the server properly overrides certname with node_name fact.
                           # Testing of passenger master is no longer needed.
-    'server'
+    'server',
+    'shard:group4' # For splitting out groups of tests for slow test runners
 
 success_message = "node_name_fact setting was correctly used to determine the node name"
 

acceptance/tests/allow_arbitrary_node_name_for_agent.rb

@@ -3,7 +3,8 @@
 tag 'audit:high',
     'audit:integration',  # Tests that the server properly overrides certname with node_name fact.
                           # Testing of passenger master is no longer needed.
-    'server'
+    'server',
+    'shard:group1' # For splitting out groups of tests for slow test runners
 
 success_message = "node_name_value setting was correctly used as the node name"
 testdir = master.tmpdir('nodenamevalue')

acceptance/tests/apply/classes/parameterized_classes.rb

@@ -1,7 +1,8 @@
 test_name "parametrized classes"
 
 tag 'audit:high',
-    'audit:unit'   # This should be covered at the unit layer.
+    'audit:unit',  # This should be covered at the unit layer.
+    'shard:group2' # For splitting out groups of tests for slow test runners
 
 ########################################################################
 step "should allow param classes"