(CONT-904) - Removal of tech debt

Fixes a bug where non-array values passed to 'object_name' did not work
and returned unknown variable "$_togrant_object_only".

The correcting of this bug means the temporary fix to disable strict
checking can now be removed.

Author: jordanbreen28

manifests/server/grant.pp

@@ -432,6 +432,9 @@
     default: {
       $_granted_object = $_object_name
       $_togrant_object = $_object_name
+      # if $_togrant_object_only not set, set it to a default value $_togrant_object
+      # allows an Array or String to be passed as $_object_name i.e. [$schema, $table] or $table
+      $_togrant_object_only = $_togrant_object
     }
   }
 

spec/spec_helper_acceptance_local.rb

@@ -16,17 +16,9 @@ def unindent
 RSpec.configure do |c|
   c.before :suite do
     install_dependencies
-    turn_off_strict_error
   end
 end
 
-def turn_off_strict_error
-  # this is only temporary fix until we finish adding support for puppet 8
-  # strict=warning is defaulted on puppet versions below 8, whereas puppet 8 has strict=error by default
-  # error is caused by 'onlyif_exists' in spec/acceptance/server/grant_spec.rb L78 (works when removed, but not what we want to test)
-  LitmusHelper.instance.run_shell("echo 'strict=warning\nstrict_variables=false' >> /etc/puppetlabs/puppet/puppet.conf")
-end
-
 def export_locales(locale)
   LitmusHelper.instance.run_shell('echo export PATH="/opt/puppetlabs/bin:$PATH" > ~/.bashrc')
   LitmusHelper.instance.run_shell('echo export LC_ALL="C" > /etc/profile.d/my-custom.lang.sh')