Add standardrb and fix issues in embeded ruby file

- ruby 2.6: add ignore for Style/ArgumentsForwarding

Author: aramprice

.github/workflows/ruby.yml

@@ -0,0 +1,19 @@
+name: Run Specs
+on: [push, pull_request]
+
+jobs:
+  test_embedded_ruby:
+    strategy:
+      matrix:
+        os: [ubuntu-latest, macos-latest]
+        ruby: [ '2.6', '2.7', '3.0', '3.1', '3.2', '3.3', '3.4', head]
+    runs-on: ${{ matrix.os }}
+    steps:
+      - uses: actions/checkout@v6
+      - uses: ruby/setup-ruby@v1
+        with:
+          ruby-version: ${{ matrix.ruby }}
+      - run: bundle install
+        working-directory: templatescompiler/erbrenderer/
+      - run: bundle exec rake
+        working-directory: templatescompiler/erbrenderer/

templatescompiler/erbrenderer/.gitignore

@@ -0,0 +1 @@
+Gemfile.lock

templatescompiler/erbrenderer/Gemfile

@@ -0,0 +1,6 @@
+source "https://rubygems.org"
+
+group :test do
+  gem "rake"
+  gem "standardrb"
+end

templatescompiler/erbrenderer/Rakefile

@@ -0,0 +1,3 @@
+require "standard/rake"
+
+task default: [:standard]

templatescompiler/erbrenderer/erb_renderer.rb

@@ -7,8 +7,8 @@
 
 class Hash
   def recursive_merge!(other)
-    self.merge!(other) do |_, old_value, new_value|
-      if old_value.class == Hash && new_value.class == Hash
+    merge!(other) do |_, old_value, new_value|
+      if old_value.class == Hash && new_value.class == Hash # rubocop:disable Style/ClassEqualityComparison
         old_value.recursive_merge!(new_value)
       else
         new_value
@@ -27,14 +27,14 @@ def initialize(spec)
     @name = spec["job"]["name"] if spec["job"].is_a?(Hash)
     @index = spec["index"]
 
-    if !spec['job_properties'].nil?
-      properties1 = spec['job_properties']
+    properties1 = if !spec["job_properties"].nil?
+      spec["job_properties"]
     else
-      properties1 = spec['global_properties'].recursive_merge!(spec['cluster_properties'])
+      spec["global_properties"].recursive_merge!(spec["cluster_properties"])
     end
 
     properties = {}
-    spec['default_properties'].each do |name, value|
+    spec["default_properties"].each do |name, value|
       copy_property(properties, properties1, name, value)
     end
 
@@ -66,7 +66,7 @@ def if_p(*names)
       value
     end
 
-    yield *values
+    yield(*values)
     InactiveElseBlock.new
   end
 
@@ -97,13 +97,15 @@ def copy_property(dst, src, name, default = nil)
 
   def openstruct(object)
     case object
-      when Hash
-        mapped = object.inject({}) { |h, (k,v)| h[k] = openstruct(v); h }
-        OpenStruct.new(mapped)
-      when Array
-        object.map { |item| openstruct(item) }
-      else
-        object
+    when Hash
+      mapped = object.each_with_object({}) { |(k, v), h|
+        h[k] = openstruct(v)
+      }
+      OpenStruct.new(mapped)
+    when Array
+      object.map { |item| openstruct(item) }
+    else
+      object
     end
   end
 
@@ -137,13 +139,14 @@ def else
       yield
     end
 
-    def else_if_p(*names, &block)
-      @context.if_p(*names, &block)
+    def else_if_p(*names, &block) # rubocop:disable Style/ArgumentsForwarding
+      @context.if_p(*names, &block) # rubocop:disable Style/ArgumentsForwarding
     end
   end
 
   class InactiveElseBlock
-    def else; end
+    def else
+    end
 
     def else_if_p(*names)
       InactiveElseBlock.new
@@ -153,7 +156,7 @@ def else_if_p(*names)
 
 # todo do not use JSON in releases
 class << JSON
-  alias dump_array_or_hash dump
+  alias_method :dump_array_or_hash, :dump
 
   def dump(*args)
     arg = args[0]
@@ -174,18 +177,15 @@ def render(src_path, dst_path)
     erb = ERB.new(File.read(src_path), trim_mode: "-")
     erb.filename = src_path
 
-    context_hash = JSON.load(File.read(@json_context_path))
+    context_hash = JSON.load_file(@json_context_path)
     template_evaluation_context = TemplateEvaluationContext.new(context_hash)
 
-    File.open(dst_path, "w") do |f|
-      f.write(erb.result(template_evaluation_context.get_binding))
-    end
-
-  rescue Exception => e
+    File.write(dst_path, erb.result(template_evaluation_context.get_binding))
+  rescue Exception => e # rubocop:disable Lint/RescueException
     name = "#{template_evaluation_context&.name}/#{template_evaluation_context&.index}"
 
-    line_i = e.backtrace.index { |l| l.include?("#{erb&.filename}") }
-    line_num = line_i ? e.backtrace[line_i].split(':')[1] : "unknown"
+    line_i = e.backtrace.index { |l| l.include?(erb&.filename.to_s) }
+    line_num = line_i ? e.backtrace[line_i].split(":")[1] : "unknown"
     location = "(line #{line_num}: #{e.inspect})"
 
     raise("Error filling in template '#{src_path}' for #{name} #{location}")