origen_testers 0.52.15 → 0.53.0

data/lib/origen_testers/igxl_based_tester/j750/patsubr_pattern.rb

@@ -7,8 +7,7 @@ module OrigenTesters
         PATSUBR_ATTRS = %w(file_name comment)
 
         # Pattern subroutine defaults
-        PATSUBR_DEFAULTS = {
-        }
+        PATSUBR_DEFAULTS = {}
 
         # Generate the instance method definitions based on the above
         define

data/lib/origen_testers/igxl_based_tester/j750/test_instance.rb

@@ -19,8 +19,7 @@ module OrigenTesters
           timeset:          :time_sets,
           timesets:         :time_sets,
 
-          other:            {
-          },
+          other:            {},
 
           empty:            {
             start_func:           :arg0,
@@ -634,10 +633,10 @@ module OrigenTesters
         #   instance.set_wait_flags(:a)
         #   instance.set_wait_flags(:a, :c)
         def set_wait_flags(*flags)
-          a = (flags.include?(:a) || flags.include?(:a)) ? '1' : 'X'
-          b = (flags.include?(:b) || flags.include?(:b)) ? '1' : 'X'
-          c = (flags.include?(:c) || flags.include?(:c)) ? '1' : 'X'
-          d = (flags.include?(:d) || flags.include?(:d)) ? '1' : 'X'
+          a = (flags.include?(:a) || flags.include?(:a)) ? '1' : 'X' # rubocop:disable Lint/BinaryOperatorWithIdenticalOperands
+          b = (flags.include?(:b) || flags.include?(:b)) ? '1' : 'X' # rubocop:disable Lint/BinaryOperatorWithIdenticalOperands
+          c = (flags.include?(:c) || flags.include?(:c)) ? '1' : 'X' # rubocop:disable Lint/BinaryOperatorWithIdenticalOperands
+          d = (flags.include?(:d) || flags.include?(:d)) ? '1' : 'X' # rubocop:disable Lint/BinaryOperatorWithIdenticalOperands
           self.wait_flags = d + c + b + a
           self
         end
@@ -658,9 +657,9 @@ module OrigenTesters
         # intended to be used in documentation
         def to_meta
           return @meta if @meta
+
           m = { 'Test' => name,
-                'Type' => type
-          }
+                'Type' => type }
           if type == :functional
             m['Pattern'] = pattern
           elsif type == :board_pmu || type == :pin_pmu

data/lib/origen_testers/igxl_based_tester/j750.rb

@@ -12,7 +12,7 @@ module OrigenTesters
     # Also note that this class inherits from the base Tester class and so all methods
     # described there are also available.
     class J750 < Base
-      require 'origen_testers/igxl_based_tester/j750/generator.rb'
+      require 'origen_testers/igxl_based_tester/j750/generator'
 
       attr_accessor :use_hv_pin
       attr_accessor :software_version
@@ -20,6 +20,7 @@ module OrigenTesters
       def self.hpt_mode
         @@hpt_mode
       end
+
       def self.hpt_mode?
         @@hpt_mode
       end
@@ -245,7 +246,7 @@ module OrigenTesters
       def handshake(options = {})
         options = {
           readcode:    false,
-          manual_stop: false,    # set a 2nd CPU flag in case 1st flag is automatically cleared
+          manual_stop: false    # set a 2nd CPU flag in case 1st flag is automatically cleared
         }.merge(options)
         if options[:readcode]
           set_code(options[:readcode])

data/lib/origen_testers/igxl_based_tester/j750_hpt/test_instance.rb

@@ -19,8 +19,7 @@ module OrigenTesters
           timeset:          :time_sets,
           timesets:         :time_sets,
 
-          other:            {
-          },
+          other:            {},
 
           empty:            {
             start_func:           :arg0,
@@ -586,10 +585,10 @@ module OrigenTesters
         #   instance.set_wait_flags(:a)
         #   instance.set_wait_flags(:a, :c)
         def set_wait_flags(*flags)
-          a = (flags.include?(:a) || flags.include?(:a)) ? '1' : 'X'
-          b = (flags.include?(:b) || flags.include?(:b)) ? '1' : 'X'
-          c = (flags.include?(:c) || flags.include?(:c)) ? '1' : 'X'
-          d = (flags.include?(:d) || flags.include?(:d)) ? '1' : 'X'
+          a = (flags.include?(:a) || flags.include?(:a)) ? '1' : 'X' # rubocop:disable Lint/BinaryOperatorWithIdenticalOperands
+          b = (flags.include?(:b) || flags.include?(:b)) ? '1' : 'X' # rubocop:disable Lint/BinaryOperatorWithIdenticalOperands
+          c = (flags.include?(:c) || flags.include?(:c)) ? '1' : 'X' # rubocop:disable Lint/BinaryOperatorWithIdenticalOperands
+          d = (flags.include?(:d) || flags.include?(:d)) ? '1' : 'X' # rubocop:disable Lint/BinaryOperatorWithIdenticalOperands
           self.wait_flags = d + c + b + a
           self
         end

data/lib/origen_testers/igxl_based_tester/j750_hpt.rb

@@ -12,7 +12,7 @@ module OrigenTesters
     # *Also note that this class inherits from the base Tester class and so all methods
     # described there are also available*
     class J750_HPT < J750
-      require 'origen_testers/igxl_based_tester/j750_hpt/generator.rb'
+      require 'origen_testers/igxl_based_tester/j750_hpt/generator'
 
       def initialize(options = {})
         super(options)

data/lib/origen_testers/igxl_based_tester/parser/descriptions.rb

@@ -13,6 +13,7 @@ module OrigenTesters
         # All descriptions are stored in this lookup table
         def lookup
           return @lookup if @lookup
+
           # Use the one from the interface if present, program generation will
           # automatically push descriptions in here
           if Origen.interface_present?
@@ -26,7 +27,6 @@ module OrigenTesters
           @parser = options[:parser]
           FileUtils.rm_rf(SCRATCH_DIR) if File.exist?(SCRATCH_DIR)
           parse_program
-          true
         end
 
         # Returns the description for the given flow
@@ -57,11 +57,11 @@ module OrigenTesters
           end
         end
 
-        def source_directories
+        def source_directories # rubocop:disable Lint/DuplicateMethods
           [@source_directories, Origen.config.test_program_source_directory].compact.flatten
         end
 
-        def template_directories
+        def template_directories # rubocop:disable Lint/DuplicateMethods
           [@template_directories, Origen.config.test_program_template_directory].compact.flatten
         end
 
@@ -151,8 +151,8 @@ module OrigenTesters
               markup_source_file_comments
               # Compile the flow file, with Ruby comments now preserved and marked up
               desc = Origen.app.runner.generate(program: true, patterns: ungenerated_dir, output: generated_dir,
-                                              check_for_changes: false, collect_stats: false, quiet: true,
-                                              collect_descriptions: true)
+                                                check_for_changes: false, collect_stats: false, quiet: true,
+                                                collect_descriptions: true)
               Origen.log.info ''
             end
             @program_generated = true
@@ -172,7 +172,7 @@ module OrigenTesters
               markup_template_comments
               # Compile the flow file, with Ruby comments now preserved and marked up
               Origen.app.runner.generate(compile: true, patterns: uncompiled_dir, output: compiled_dir,
-                                       check_for_changes: false, collect_stats: false, quiet: true)
+                                         check_for_changes: false, collect_stats: false, quiet: true)
               Origen.log.info ''
             end
             @program_compiled = true
@@ -251,8 +251,7 @@ module OrigenTesters
         # templates to
         def flow_or_instance_file?(file, options = {})
           options = { flow:     true,
-                      instance: true
-          }.merge(options)
+                      instance: true }.merge(options)
           if options[:flow] && options[:instance]
             match = 'Flow|Instances'
           elsif options[:flow]
@@ -263,19 +262,18 @@ module OrigenTesters
           # Not sure the best way to determine the file type of a partial, just
           # return true for now to play it safe
           return true if file.basename.to_s =~ /^_/
+
           File.readlines(file).each do |line|
-            begin
-              unless line =~ /^%/ || line =~ /^\s*<comment>/
-                return !!(line =~ /#{match}/)
-              end
-            rescue Exception => e
-              if e.is_a?(ArgumentError) && e.message =~ /invalid byte sequence/
-                return false
-              else
-                puts e.message
-                puts e.backtrace
-                exit 1
-              end
+            unless line =~ /^%/ || line =~ /^\s*<comment>/
+              return !!(line =~ /#{match}/)
+            end
+          rescue Exception => e
+            if e.is_a?(ArgumentError) && e.message =~ /invalid byte sequence/
+              return false
+            else
+              puts e.message
+              puts e.backtrace
+              exit 1
             end
           end
         end

data/lib/origen_testers/igxl_based_tester/parser/test_instance.rb

@@ -392,10 +392,10 @@ module OrigenTesters
 
         # Returns an array of all pattern names referenced in this test instance
         def patterns
-          if self.respond_to?(:pattern)
+          if respond_to?(:pattern)
             pattern.split(',').map do |pat|
               extract_pattern_from_patset(pat)
-            end.flatten.map { |pat| pat.gsub(/.*[\\\/]/, '').gsub(/\..*/, '') }
+            end.flatten.map { |pat| pat.gsub(/.*[\\\/]/, '').gsub(/\..*/, '') } # rubocop:disable Style/MultilineBlockChain
           end
         end
 

data/lib/origen_testers/igxl_based_tester/parser.rb

@@ -1,18 +1,18 @@
 module OrigenTesters
   class IGXLBasedTester
     class Parser
-      autoload :Flows,   'origen_testers/igxl_based_tester/parser/flows'
-      autoload :Flow,   'origen_testers/igxl_based_tester/parser/flow'
-      autoload :FlowLine,   'origen_testers/igxl_based_tester/parser/flow_line'
-      autoload :TestInstances,   'origen_testers/igxl_based_tester/parser/test_instances'
-      autoload :TestInstance,   'origen_testers/igxl_based_tester/parser/test_instance'
-      autoload :PatternSets,   'origen_testers/igxl_based_tester/parser/pattern_sets'
-      autoload :PatternSet,   'origen_testers/igxl_based_tester/parser/pattern_set'
-      autoload :DCSpecs,   'origen_testers/igxl_based_tester/parser/dc_specs'
-      autoload :DCSpec,   'origen_testers/igxl_based_tester/parser/dc_spec'
-      autoload :ACSpecs,   'origen_testers/igxl_based_tester/parser/ac_specs'
-      autoload :ACSpec,   'origen_testers/igxl_based_tester/parser/ac_spec'
-      autoload :Descriptions,   'origen_testers/igxl_based_tester/parser/descriptions'
+      autoload :Flows, 'origen_testers/igxl_based_tester/parser/flows'
+      autoload :Flow, 'origen_testers/igxl_based_tester/parser/flow'
+      autoload :FlowLine, 'origen_testers/igxl_based_tester/parser/flow_line'
+      autoload :TestInstances, 'origen_testers/igxl_based_tester/parser/test_instances'
+      autoload :TestInstance, 'origen_testers/igxl_based_tester/parser/test_instance'
+      autoload :PatternSets, 'origen_testers/igxl_based_tester/parser/pattern_sets'
+      autoload :PatternSet, 'origen_testers/igxl_based_tester/parser/pattern_set'
+      autoload :DCSpecs, 'origen_testers/igxl_based_tester/parser/dc_specs'
+      autoload :DCSpec, 'origen_testers/igxl_based_tester/parser/dc_spec'
+      autoload :ACSpecs, 'origen_testers/igxl_based_tester/parser/ac_specs'
+      autoload :ACSpec, 'origen_testers/igxl_based_tester/parser/ac_spec'
+      autoload :Descriptions, 'origen_testers/igxl_based_tester/parser/descriptions'
 
       def reset
         @flows = nil

data/lib/origen_testers/igxl_based_tester/ultraflex/ate_hardware.rb

@@ -18,7 +18,7 @@ module OrigenTesters
                        forcei_50mA__lte_20mA:      (-1.V..6.V),
                        forcei_Non50mA__half_scale: (-1..6.V),
                        forcei_Non50mA__full_scale: (-1.V..5.5.V) }
-            measi = [2.uA,  20.uA, 200.uA,  2.mA,  50.mA]
+            measi = [2.uA, 20.uA, 200.uA, 2.mA, 50.mA]
             measv =  { measi_50mA__gt_20mA:         (-0.1.V..4.5.V),
                        measi_50mA__lte_20mA:        (-1.V..6.V),
                        measi_20mA__0p5mA_to_20mA:   (-1.V..5.5.V),
@@ -40,7 +40,7 @@ module OrigenTesters
                        forcei_50mA__n35mA_to_n20mA: (0.5.V..5.1.V),
                        forcei_50mA__n50mA_to_n35mA: (1.1.V..4.5.V),
                        forcei_Non50mA:              (-1.V..6.V) }
-            measi = [2.uA,  20.uA, 200.uA,  2.mA,  50.mA]
+            measi = [2.uA, 20.uA, 200.uA, 2.mA, 50.mA]
             measv = { measi_50mA:              (-1.V..6.V),
                       measi_200uA:             (-1.V..6.V),
                       measi_20uA:              (-1.V..6.V),
@@ -51,7 +51,7 @@ module OrigenTesters
             ppmu.new(forcei, forcev, measi, measv, vclamp)
           elsif @instrument == 'HSD-4G'
             forcei = [200.uA, 2.mA, 30.mA]
-            forcev = { measi_30mA:  [min: '1.5V, 1.5V - 50mV/mA * Idut', max: '0V, -50mV/mA * Idut'],
+            forcev = { measi_30mA:  [{ min: '1.5V, 1.5V - 50mV/mA * Idut', max: '0V, -50mV/mA * Idut' }],
                        measi_2mA:   (-1.V..1.5.V),
                        measi_200uA: (-1.V..1.5.V) }
             measi = [200.uA, 2.mA, 30.mA]
@@ -72,8 +72,8 @@ module OrigenTesters
             vclamp = (-1.3.V..3.9.V)
             ppmu.new(forcei, forcev, measi, measv, vclamp)
           else
-            puts "please enter an instrument type: e.g. $tester.ate_hardware(\"HSD-M\").ppmu"
-            puts "Instrument type available: \"HSD-M\", \"HSD-U\", \"HSD-4G\", and \"HSS-6G\" "
+            puts 'please enter an instrument type: e.g. $tester.ate_hardware("HSD-M").ppmu'
+            puts 'Instrument type available: "HSD-M", "HSD-U", "HSD-4G", and "HSS-6G" '
             puts 'HSD-U is also known as Ultrapin1600.  HSS-6G is also known as SB6G.'
           end
         end
@@ -82,7 +82,7 @@ module OrigenTesters
           supply = Struct.new(:forcev, :irange, :source_overload_i, :source_overload_t, :source_fold_i,
                               :source_fold_t, :sink_overload_i, :sink_overload_t, :sink_fold_i,
                               :sink_fold_t, :meter_irange, :meter_vrange, :tdelay, :accuracy,
-                              :filter, :bandwidth)
+                              :filter, :bandwidth) # rubocop:disable Lint/StructNewOverride
           if @instrument == 'VSM'
             forcev = (0.V..4.V)
             irange = [1.A, 11.A, 21.A, 51.A, 81.A]
@@ -98,17 +98,17 @@ module OrigenTesters
                               irange_51A: (2.55.A..52.53.A),
                               irange_81A: (4.05.A..83.43.A) }
             source_fold_t = (10.uS..8.S)
-            sink_overload_i = { irange_1A:  [max: 78.mA],    # ?????    Not programmable?
-                                irange_11A: [max: 858.mA],    # ?????    Not programmable?
-                                irange_21A: [max: 1.64.A],    # ?????    Not programmable?
-                                irange_51A: [max: 3.98.A],    # ?????    Not programmable?
-                                irange_81A: [max: 6.32.A] }    # ?????    Not programmable?
+            sink_overload_i = { irange_1A:  [{ max: 78.mA }],    # ?????    Not programmable?
+                                irange_11A: [{ max: 858.mA }],    # ?????    Not programmable?
+                                irange_21A: [{ max: 1.64.A }],    # ?????    Not programmable?
+                                irange_51A: [{ max: 3.98.A }],    # ?????    Not programmable?
+                                irange_81A: [{ max: 6.32.A }] }    # ?????    Not programmable?
             sink_overload_t = 0                  # ?????    Not programmable?
-            sink_fold_i = { irange_1A:  [max: 78.mA],      # ?????    Not programmable?
-                            irange_11A: [max: 858.mA],     # ?????    Not programmable?
-                            irange_21A: [max: 1.64.A],     # ?????    Not programmable?
-                            irange_51A: [max: 3.98.A],    # ?????    Not programmable?
-                            irange_81A: [max: 6.32.A] }    # ?????    Not programmable?
+            sink_fold_i = { irange_1A:  [{ max: 78.mA }],      # ?????    Not programmable?
+                            irange_11A: [{ max: 858.mA }],     # ?????    Not programmable?
+                            irange_21A: [{ max: 1.64.A }],     # ?????    Not programmable?
+                            irange_51A: [{ max: 3.98.A }],    # ?????    Not programmable?
+                            irange_81A: [{ max: 6.32.A }] }    # ?????    Not programmable?
             sink_fold_t = (10.uS..8.S)
             meter_irange = { irange_1A:  [1.25.A, 2.5.A],
                              irange_11A: [13.75.A, 27.5.A],
@@ -144,23 +144,23 @@ module OrigenTesters
                               irange_102A: (5.1.A..105.06.A),
                               irange_162A: (8.1.A..166.86.A) }
             source_fold_t = (10.uS..8.S)
-            sink_overload_i = { irange_1A:   [max: 78.mA],     # ?????    Not programmable?
-                                irange_2A:   [max: 156.mA],     # ?????    Not programmable?
-                                irange_11A:  [max: 858.mA],     # ?????    Not programmable?
-                                irange_21A:  [max: 1.64.A],     # ?????    Not programmable?
-                                irange_51A:  [max: 3.98.A],     # ?????    Not programmable?
-                                irange_81A:  [max: 6.32.A],     # ?????    Not programmable?
-                                irange_102A: [max: 7.96.A],     # ?????    Not programmable?
-                                irange_162A: [max: 12.64.A] }     # ?????    Not programmable?
+            sink_overload_i = { irange_1A:   [{ max: 78.mA }],     # ?????    Not programmable?
+                                irange_2A:   [{ max: 156.mA }],     # ?????    Not programmable?
+                                irange_11A:  [{ max: 858.mA }],     # ?????    Not programmable?
+                                irange_21A:  [{ max: 1.64.A }],     # ?????    Not programmable?
+                                irange_51A:  [{ max: 3.98.A }],     # ?????    Not programmable?
+                                irange_81A:  [{ max: 6.32.A }],     # ?????    Not programmable?
+                                irange_102A: [{ max: 7.96.A }],     # ?????    Not programmable?
+                                irange_162A: [{ max: 12.64.A }] }     # ?????    Not programmable?
             sink_overload_t = 0                  # ?????    Not programmable?
-            sink_fold_i = { irange_1A:   [max: 78.mA],     # ?????    Not programmable?
-                            irange_2A:   [max: 156.mA],     # ?????    Not programmable?
-                            irange_11A:  [max: 858.mA],     # ?????    Not programmable?
-                            irange_21A:  [max: 1.64.A],     # ?????    Not programmable?
-                            irange_51A:  [max: 3.98.A],     # ?????    Not programmable?
-                            irange_81A:  [max: 6.32.A],     # ?????    Not programmable?
-                            irange_102A: [max: 7.96.A],     # ?????    Not programmable?
-                            irange_162A: [max: 12.64.A] }     # ?????    Not programmable?
+            sink_fold_i = { irange_1A:   [{ max: 78.mA }],     # ?????    Not programmable?
+                            irange_2A:   [{ max: 156.mA }],     # ?????    Not programmable?
+                            irange_11A:  [{ max: 858.mA }],     # ?????    Not programmable?
+                            irange_21A:  [{ max: 1.64.A }],     # ?????    Not programmable?
+                            irange_51A:  [{ max: 3.98.A }],     # ?????    Not programmable?
+                            irange_81A:  [{ max: 6.32.A }],     # ?????    Not programmable?
+                            irange_102A: [{ max: 7.96.A }],     # ?????    Not programmable?
+                            irange_162A: [{ max: 12.64.A }] }     # ?????    Not programmable?
             meter_irange = { irange_1A:   [1.25.A, 2.5.A],
                              irange_2A:   [2.5.A, 5.A],
                              irange_11A:  [13.75.A, 27.5.A],
@@ -213,8 +213,8 @@ module OrigenTesters
             sink_fold_i = { irange_15A: (1.A..2.A),
                             irange_30A: (2.A..4.A) }
             sink_fold_t = (102.4.uS..5.S)
-            meter_irange =  { irange_15A: [10.mA, 100.mA, 1.A, 15.A],
-                              irange_30A: [30.A] } # This is verified to be correct on tester.
+            meter_irange = { irange_15A: [10.mA, 100.mA, 1.A, 15.A],
+                             irange_30A: [30.A] } # This is verified to be correct on tester.
             meter_vrange = [4.V, 8.V]
             tdelay = 0                     # default tdelay
             accuracy = { neg: -7.mV, pos: 7.mV }
@@ -238,8 +238,8 @@ module OrigenTesters
             sink_fold_i = { irange_15A: (1.A..2.A),
                             irange_60A: (4.A..8.A) }
             sink_fold_t = (102.4.uS..5.S)
-            meter_irange =  { irange_15A: [10.mA, 100.mA, 1.A, 15.A],
-                              irange_60A: [60.A] } # This is verified to be correct on tester.
+            meter_irange = { irange_15A: [10.mA, 100.mA, 1.A, 15.A],
+                             irange_60A: [60.A] } # This is verified to be correct on tester.
             meter_vrange = [4.V, 8.V]
             tdelay = 0                     # default tdelay
             accuracy = { neg: -7.mV, pos: 7.mV }
@@ -263,8 +263,8 @@ module OrigenTesters
             sink_fold_i = { irange_15A: (1.A..2.A),
                             irange_90A: (6.A..12.A) }
             sink_fold_t = (102.4.uS..5.S)
-            meter_irange =  { irange_15A: [10.mA, 100.mA, 1.A, 15.A],
-                              irange_90A: [90.A] } # This is verified to be correct on tester.
+            meter_irange = { irange_15A: [10.mA, 100.mA, 1.A, 15.A],
+                             irange_90A: [90.A] } # This is verified to be correct on tester.
             meter_vrange = [4.V, 8.V]
             tdelay = 0                     # default tdelay
             accuracy = { neg: -7.mV, pos: 7.mV }
@@ -308,8 +308,8 @@ module OrigenTesters
             sink_fold_i = { irange_15A: (1.A..2.A),
                             irange_30A: (2.A..4.A) }
             sink_fold_t = (102.4.uS..5.S)
-            meter_irange =  { irange_15A: [10.mA, 100.mA, 1.A, 15.A],
-                              irange_30A: [30.A] } # This is verified to be correct on tester.
+            meter_irange = { irange_15A: [10.mA, 100.mA, 1.A, 15.A],
+                             irange_30A: [30.A] } # This is verified to be correct on tester.
             meter_vrange = [4.V, 8.V]
             tdelay = 0                     # default tdelay
             accuracy = { neg: -2.mV, pos: 2.mV }
@@ -333,8 +333,8 @@ module OrigenTesters
             sink_fold_i = { irange_15A: (1.A..2.A),
                             irange_60A: (4.A..8.A) }
             sink_fold_t = (102.4.uS..5.S)
-            meter_irange =  { irange_15A: [10.mA, 100.mA, 1.A, 15.A],
-                              irange_60A: [60.A] } # This is verified to be correct on tester.
+            meter_irange = { irange_15A: [10.mA, 100.mA, 1.A, 15.A],
+                             irange_60A: [60.A] } # This is verified to be correct on tester.
             meter_vrange = [4.V, 8.V]
             tdelay = 0                     # default tdelay
             accuracy = { neg: -2.mV, pos: 2.mV }
@@ -358,8 +358,8 @@ module OrigenTesters
             sink_fold_i = { irange_15A: (1.A..2.A),
                             irange_90A: (6.A..12.A) }
             sink_fold_t = (102.4.uS..5.S)
-            meter_irange =  { irange_15A: [10.mA, 100.mA, 1.A, 15.A],
-                              irange_90A: [90.A] } # This is verified to be correct on tester.
+            meter_irange = { irange_15A: [10.mA, 100.mA, 1.A, 15.A],
+                             irange_90A: [90.A] } # This is verified to be correct on tester.
             meter_vrange = [4.V, 8.V]
             tdelay = 0                     # default tdelay
             accuracy = { neg: -2.mV, pos: 2.mV }
@@ -379,7 +379,7 @@ module OrigenTesters
             sink_overload_t = 'n/a'
             sink_fold_i = (5.mA..200.mA)
             sink_fold_t = (0.S..167.77.mS)
-            meter_irange =  [10.uA, 100.uA, 1.mA, 10.mA, 100.mA, 1.A]
+            meter_irange = [10.uA, 100.uA, 1.mA, 10.mA, 100.mA, 1.A]
             meter_vrange = 7.V
             tdelay = 0                     # default tdelay
             accuracy = { neg: '-0.001xSUPPLY-10.mV', pos: '0.001xSUPPLY+10.mV' }
@@ -402,7 +402,7 @@ module OrigenTesters
             sink_overload_t = 'n/a'
             sink_fold_i = (10.mA..400.mA)
             sink_fold_t = (0.S..167.77.mS)
-            meter_irange =  [20.uA, 200.uA, 2.mA, 20.mA, 200.mA, 2.A]
+            meter_irange = [20.uA, 200.uA, 2.mA, 20.mA, 200.mA, 2.A]
             meter_vrange = 7.V
             tdelay = 0                     # default tdelay
             accuracy = { neg: '-0.001xSUPPLY-10.mV', pos: '0.001xSUPPLY+10.mV' }
@@ -425,7 +425,7 @@ module OrigenTesters
             sink_overload_t = 'n/a'
             sink_fold_i = (20.mA..800.mA)
             sink_fold_t = (0.S..167.77.mS)
-            meter_irange =  [40.uA, 400.uA, 4.mA, 40.mA, 400.mA, 4.A]
+            meter_irange = [40.uA, 400.uA, 4.mA, 40.mA, 400.mA, 4.A]
             meter_vrange = 7.V
             tdelay = 0                     # default tdelay
             accuracy = { neg: '-0.001xSUPPLY-10.mV', pos: '0.001xSUPPLY+10.mV' }
@@ -930,14 +930,14 @@ module OrigenTesters
                        source_fold_t, sink_overload_i, sink_overload_t, sink_fold_i,
                        sink_fold_t, meter_irange, meter_vrange, tdelay, accuracy, filter, bandwidth)
           else
-            puts "please enter an instrument type: e.g. $tester.ate_hardware(\"VSM\").supply"
-            puts "Instrument type available: \"VSM\", \"VSMx2\", \"VSMx4\","
-            puts "\"HexVS\", \"HexVSx2\", \"HexVSx4\", \"HexVSx6\","
-            puts "\"HexVS+\", \"HexVS+x2\", \"HexVS+x4\", \"HexVS+x6\", \"HDVS1\","
-            puts "\"HDVS1x2\", \"HDVS1x4\", \"VHDVS\", \"VHDVS_HC\", \"VHDVSx2\","
-            puts "\"VHDVS_HCx2\", \"VHDVS_HCx4\", and \"VHDVS_HCx8\"."
-            puts "\"VHDVS+\", \"VHDVS_HC+\", \"VHDVS+x2\","
-            puts "\"VHDVS_HC+x2\", \"VHDVS_HC+x4\", and \"VHDVS_HC+x8\"."
+            puts 'please enter an instrument type: e.g. $tester.ate_hardware("VSM").supply'
+            puts 'Instrument type available: "VSM", "VSMx2", "VSMx4",'
+            puts '"HexVS", "HexVSx2", "HexVSx4", "HexVSx6",'
+            puts '"HexVS+", "HexVS+x2", "HexVS+x4", "HexVS+x6", "HDVS1",'
+            puts '"HDVS1x2", "HDVS1x4", "VHDVS", "VHDVS_HC", "VHDVSx2",'
+            puts '"VHDVS_HCx2", "VHDVS_HCx4", and "VHDVS_HCx8".'
+            puts '"VHDVS+", "VHDVS_HC+", "VHDVS+x2",'
+            puts '"VHDVS_HC+x2", "VHDVS_HC+x4", and "VHDVS_HC+x8".'
             puts 'HDVS1 is also known as HDVS.  VHDVS is also known as UVS256.'
             puts 'x2 is Merged2, x4 is Merged4, x6 is Merged6.  _HC is High-Current.'
             puts '+ is High-Accuracy.'

data/lib/origen_testers/igxl_based_tester/ultraflex/flow_line.rb

@@ -8,8 +8,7 @@ module OrigenTesters
                                    bin_pass bin_fail sort_pass sort_fail result flag_pass flag_fail state
                                    group_specifier group_sense group_condition group_name
                                    device_sense device_condition device_name
-                                   debug_assume debug_sites comment
-                                )
+                                   debug_assume debug_sites comment)
 
         # Generate the instance method definitions based on the above
         define

data/lib/origen_testers/igxl_based_tester/ultraflex/patsubr_pattern.rb

@@ -7,8 +7,7 @@ module OrigenTesters
         PATSUBR_ATTRS = %w(file_name comment)
 
         # Pattern subroutine defaults
-        PATSUBR_DEFAULTS = {
-        }
+        PATSUBR_DEFAULTS = {}
 
         # Generate the instance method definitions based on the above
         define

data/lib/origen_testers/igxl_based_tester/ultraflex/test_instance.rb

@@ -23,8 +23,7 @@ module OrigenTesters
           timeset:          :time_sets,
           timesets:         :time_sets,
 
-          other:            {
-          },
+          other:            {},
 
           empty:            {
             arg_list:             :arg0,
@@ -310,7 +309,7 @@ module OrigenTesters
             wait_flag1:     -2, # waitoff
             wait_flag2:     -2, # waitoff
             wait_flag3:     -2, # waitoff
-            wait_flag4:     -2, # waitoff
+            wait_flag4:     -2 # waitoff
           },
           dcvi_powersupply: {
             arg_list:       'PreconditionPat,StartOfBodyF,PrePatF,PreTestF,PostTestF,PostPatF,EndOfBodyF,HoldStatePat,DriveLoPins,DriveHiPins,DriveZPins,FloatPins,SamplingTime,SampleSize,SettlingTime,MainVoltage,AltVoltage,PowerPins,DisablePins,VoltageOutput,PcpStartLabel,PcpStopLabel,StartOfBodyFArgs,PrePatFArgs,PreTestFArgs,PostTestFArgs,PostPatFArgs,EndOfBodyFArgs,HspStartLabel,HspStopLabel,PcpCheckPatGen,CurrentClamp,HspCheckPatGen,HspResumePat,RelayMode,Util1Pins,Util0Pins,TestControl,SerializeMeas,MeasF,MeasFArgs,WaitFlagA,WaitFlagB,WaitFlagC,WaitFlagD,Validating_,Irange,PatternTimeout,PcpDisableAlarm,HspDisableAlarm',
@@ -320,7 +319,7 @@ module OrigenTesters
             wait_flag1:     -2, # waitoff
             wait_flag2:     -2, # waitoff
             wait_flag3:     -2, # waitoff
-            wait_flag4:     -2, # waitoff
+            wait_flag4:     -2 # waitoff
           }
 
         }
@@ -335,10 +334,10 @@ module OrigenTesters
         # assumes clr flag means to set it off (waitoff = -2)
         # does not yet support waitlo = 0
         def set_wait_flags(*flags)
-          a = (flags.include?(:a) || flags.include?(:a)) ? '-1' : '-2'
-          b = (flags.include?(:b) || flags.include?(:b)) ? '-1' : '-2'
-          c = (flags.include?(:c) || flags.include?(:c)) ? '-1' : '-2'
-          d = (flags.include?(:d) || flags.include?(:d)) ? '-1' : '-2'
+          a = (flags.include?(:a) || flags.include?(:a)) ? '-1' : '-2' # rubocop:disable Lint/BinaryOperatorWithIdenticalOperands
+          b = (flags.include?(:b) || flags.include?(:b)) ? '-1' : '-2' # rubocop:disable Lint/BinaryOperatorWithIdenticalOperands
+          c = (flags.include?(:c) || flags.include?(:c)) ? '-1' : '-2' # rubocop:disable Lint/BinaryOperatorWithIdenticalOperands
+          d = (flags.include?(:d) || flags.include?(:d)) ? '-1' : '-2' # rubocop:disable Lint/BinaryOperatorWithIdenticalOperands
           self.wait_flag1 = a
           self.wait_flag2 = b
           self.wait_flag3 = c
@@ -356,9 +355,9 @@ module OrigenTesters
         # intended to be used in documentation
         def to_meta
           return @meta if @meta
+
           m = { 'Test' => name,
-                'Type' => type
-          }
+                'Type' => type }
           if type == :functional
             m['Pattern'] = pattern
           elsif type == :board_pmu || type == :pin_pmu