usps_flags 0.3.26 → 0.4.0

data/lib/usps_flags/helpers.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: false
+
 # Container class for helper methods.
 class USPSFlags::Helpers
   class << self
@@ -30,11 +32,12 @@ class USPSFlags::Helpers
     def resize_png(png_file, file: nil, outfile: nil, size:, size_key: nil)
       raise USPSFlags::Errors::PNGConversionError if outfile.nil? && file.nil?
       raise USPSFlags::Errors::PNGConversionError if outfile.nil? && size_key.nil?
+
       output_file_name = outfile || "#{USPSFlags.configuration.flags_dir}/PNG/#{file}.#{size_key}.png"
       MiniMagick::Tool::Convert.new do |convert|
-        convert << "-background" << "none"
-        convert << "-format" << "png"
-        convert << "-resize" << "#{size}"
+        convert << '-background' << 'none'
+        convert << '-format' << 'png'
+        convert << '-resize' << "#{size}"
         convert << png_file
         convert << output_file_name
       end
@@ -67,7 +70,7 @@ class USPSFlags::Helpers
     # This is used USPSFlags::Generate, and should never need to be called directly.
     # @private
     def png_sizes
-      {1500 => "H", 1000 => "K", 500 => "D", "thumb" => "T"}
+      { 1500 => 'H', 1000 => 'K', 500 => 'D', 'thumb' => 'T' }
     end
 
     # Gets size key for saving PNG images.
@@ -75,18 +78,18 @@ class USPSFlags::Helpers
     # This is used USPSFlags::Generate, and should never need to be called directly.
     # @private
     def size_and_key(size:, flag:)
-      return [size, size] unless size == "thumb"
+      return [size, size] unless size == 'thumb'
 
       size = case flag
-      when "ENSIGN"
-        200
-      when "US"
-        300
-      else
-        150
+             when 'ENSIGN'
+               200
+             when 'US'
+               300
+             else
+               150
       end
 
-      [size, "thumb"]
+      [size, 'thumb']
     end
 
     # Prints message(s) to the console and logs them.
@@ -114,11 +117,11 @@ class USPSFlags::Helpers
     # This should never need to be called directly.
     # @private
     def ensure_dir_for_file(path)
-      return false if path.nil? || path.empty? || !path.scan("/")
+      return false if path.nil? || path.empty? || !path.scan('/')
 
-      dirs = path.split("/")
+      dirs = path.split('/')
       dirs.pop
-      ::FileUtils.mkdir_p(dirs.join("/")).first
+      ::FileUtils.mkdir_p(dirs.join('/')).first
     end
 
     # Prints output to the console or saves to a file, then returns the generated data.
@@ -128,16 +131,17 @@ class USPSFlags::Helpers
     def output(svg, outfile: nil)
       if outfile.nil?
         puts svg, "\n"
-      elsif outfile != ""
+      elsif outfile != ''
         ensure_dir_for_file(outfile)
-        f = ::File.new(outfile, "w+")
+        f = ::File.new(outfile, 'w+')
         f.write(svg)
         f.close
       end
       svg
     end
 
-    private
+  private
+
     def load_valid_flags
       @squadron_past = %w[PLTC PC]
       @squadron_elected = %w[1LT LTC CDR]
@@ -148,7 +152,7 @@ class USPSFlags::Helpers
       @national_past = %w[PSTFC PRC PVC PCC]
       @national_elected = %w[STFC RC VC CC]
       @national_swallowtail = %w[NAIDE NFLT]
-      @special = %w[CRUISE OIC ENSIGN WHEEL]
+      @special = %w[CRUISE OIC ENSIGN] # WHEEL
       @us = %w[US]
 
       @past = @squadron_past + @district_past + @national_past
@@ -173,65 +177,50 @@ class USPSFlags::Helpers
         insignia: @officer - @past,
         swallowtail: @past + @squadron_swallowtail + @district_swallowtail + @national_swallowtail,
 
-        bridge: @squadron_elected.last(2) + @squadron_past.last(2) + 
-          @district_elected.last(2) + @district_past.last(2) + 
+        bridge: @squadron_elected.last(2) + @squadron_past.last(2) +
+          @district_elected.last(2) + @district_past.last(2) +
           @national_elected.last(2) + @national_past.last(2),
 
         command: [@squadron_elected.last, @squadron_past.last,
-          @district_elected.last, @district_past.last,
-          @national_elected.last, @national_past.last]
+                  @district_elected.last, @district_past.last,
+                  @national_elected.last, @national_past.last]
       }[type]
     end
 
     def flag_style(rank)
-      if valid_flags(:past).include?(rank)
-        :past
-      elsif valid_flags(:swallowtail).include?(rank)
-        :swallowtail
-      else
-        :regular
-      end
+      return :past if valid_flags(:past).include?(rank)
+      return :swallowtail if valid_flags(:swallowtail).include?(rank)
+
+      :regular
     end
 
     def flag_color(rank)
-      if valid_flags(:command).include?(rank)
-        count = 3
-        :blue
-      elsif valid_flags(:bridge).include?(rank)
-        count = 2
-        :red
-      else
-        :white
-      end
+      return :blue if valid_flags(:command).include?(rank)
+      return :red if valid_flags(:bridge).include?(rank)
+
+      :white
     end
 
     def flag_level(rank)
-      if rank.match /N.*/
-        :n
-      elsif rank.match /D.*/
-        :d
-      elsif rank == "FLT"
-        :s
-      end
+      return :n if rank.match? /N.*/
+      return :d if rank.match? /D.*/
+      return :s if rank == 'FLT'
     end
 
     def flag_count(rank)
-      if valid_flags(:command).include?(rank)
-        3
-      elsif valid_flags(:bridge).include?(rank)
-        2
-      else
-        1
-      end
+      return 3 if valid_flags(:command).include?(rank)
+      return 2 if valid_flags(:bridge).include?(rank)
+
+      1
     end
 
     def flag_type(rank) # Complexity
-      specifics = {"PORTCAP" => :pc, "FLEETCAP" => :fc, "STFC" => :stf}
+      specifics = { 'PORTCAP' => :pc, 'FLEETCAP' => :fc, 'STFC' => :stf }
       if specifics.keys.include?(rank)
         specifics[rank]
-      elsif rank.match /.AIDE/
+      elsif rank.match? /.AIDE/
         :a
-      elsif rank.match /.?FLT/
+      elsif rank.match? /.?FLT/
         :f
       else
         get_line_flag_level(rank)
@@ -239,13 +228,9 @@ class USPSFlags::Helpers
     end
 
     def get_line_flag_level(rank)
-      if valid_flags(:squadron).include?(rank)
-        :s
-      elsif valid_flags(:district).include?(rank)
-        :d
-      elsif valid_flags(:national).include?(rank)
-        :n
-      end
+      return :s if valid_flags(:squadron).include?(rank)
+      return :d if valid_flags(:district).include?(rank)
+      return :n if valid_flags(:national).include?(rank)
     end
   end
 end

data/lib/usps_flags/helpers/builders.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: false
+
 # SVG building helpers.
 #
 # These methods should never need to be called directly, except when designing new flags.
@@ -10,38 +12,38 @@ class USPSFlags::Helpers::Builders
     # @private
     def grid(width: USPSFlags::Config::BASE_FLY, height: USPSFlags::Config::BASE_HOIST)
       <<~SVG
-        <circle cx="0" cy="0" r="#{width/60}" fill="#000000" fill-opacity="0.4" />
-        <circle cx="#{width}" cy="0" r="#{width/60}" fill="#000000" fill-opacity="0.4" />
-        <circle cx="#{width}" cy="#{height}" r="#{width/60}" fill="#000000" fill-opacity="0.4" />
-        <circle cx="0" cy="#{height}" r="#{width/60}" fill="#000000" fill-opacity="0.4" />
+        <circle cx="0" cy="0" r="#{width / 60}" fill="#000000" fill-opacity="0.4" />
+        <circle cx="#{width}" cy="0" r="#{width / 60}" fill="#000000" fill-opacity="0.4" />
+        <circle cx="#{width}" cy="#{height}" r="#{width / 60}" fill="#000000" fill-opacity="0.4" />
+        <circle cx="0" cy="#{height}" r="#{width / 60}" fill="#000000" fill-opacity="0.4" />
 
-        <circle cx="#{width*1/4}" cy="#{height/2}" r="#{width/60}" fill="#999999" fill-opacity="0.4" />
-        <circle cx="#{width*3/4}" cy="#{height/2}" r="#{width/60}" fill="#999999" fill-opacity="0.4" />
+        <circle cx="#{width * 1 / 4}" cy="#{height / 2}" r="#{width / 60}" fill="#999999" fill-opacity="0.4" />
+        <circle cx="#{width * 3 / 4}" cy="#{height / 2}" r="#{width / 60}" fill="#999999" fill-opacity="0.4" />
 
-        <circle cx="#{width/2}" cy="#{height*1/4}" r="#{width/60}"  fill="#999999" fill-opacity="0.4" />
-        <circle cx="#{width/2}" cy="#{height/2}" r="#{width/60}"  fill="#000000" fill-opacity="0.4" />
-        <circle cx="#{width/2}" cy="#{height/2}" r="#{width/75}"  fill="#CCCCCC" fill-opacity="0.4" />
-        <circle cx="#{width/2}" cy="#{height/2}" r="#{width/100}" fill="#000000" fill-opacity="0.4" />
-        <circle cx="#{width/2}" cy="#{height*3/4}" r="#{width/60}"  fill="#999999" fill-opacity="0.4" />
+        <circle cx="#{width / 2}" cy="#{height * 1 / 4}" r="#{width / 60}"  fill="#999999" fill-opacity="0.4" />
+        <circle cx="#{width / 2}" cy="#{height / 2}" r="#{width / 60}"  fill="#000000" fill-opacity="0.4" />
+        <circle cx="#{width / 2}" cy="#{height / 2}" r="#{width / 75}"  fill="#CCCCCC" fill-opacity="0.4" />
+        <circle cx="#{width / 2}" cy="#{height / 2}" r="#{width / 100}" fill="#000000" fill-opacity="0.4" />
+        <circle cx="#{width / 2}" cy="#{height * 3 / 4}" r="#{width / 60}"  fill="#999999" fill-opacity="0.4" />
 
-        <line x1="0" y1="0" x2="#{width}" y2="0" stroke="#333333" stroke-width="#{width/600}" stroke-opacity="0.5" />
-        <line x1="0" y1="#{height*1/4}" x2="#{width}" y2="#{height*1/4}" stroke="#333333" stroke-width="#{width/600}" stroke-opacity="0.5" />
-        <line x1="0" y1="#{height/2}" x2="#{width}" y2="#{height/2}" stroke="#333333" stroke-width="#{width/600}" stroke-opacity="0.5" />
-        <line x1="0" y1="#{height*3/4}" x2="#{width}" y2="#{height*3/4}" stroke="#333333" stroke-width="#{width/600}" stroke-opacity="0.5" />
-        <line x1="0" y1="#{height}" x2="#{width}" y2="#{height}" stroke="#333333" stroke-width="#{width/600}" stroke-opacity="0.5" />
+        <line x1="0" y1="0" x2="#{width}" y2="0" stroke="#333333" stroke-width="#{width / 600}" stroke-opacity="0.5" />
+        <line x1="0" y1="#{height * 1 / 4}" x2="#{width}" y2="#{height * 1 / 4}" stroke="#333333" stroke-width="#{width / 600}" stroke-opacity="0.5" />
+        <line x1="0" y1="#{height / 2}" x2="#{width}" y2="#{height / 2}" stroke="#333333" stroke-width="#{width / 600}" stroke-opacity="0.5" />
+        <line x1="0" y1="#{height * 3 / 4}" x2="#{width}" y2="#{height * 3 / 4}" stroke="#333333" stroke-width="#{width / 600}" stroke-opacity="0.5" />
+        <line x1="0" y1="#{height}" x2="#{width}" y2="#{height}" stroke="#333333" stroke-width="#{width / 600}" stroke-opacity="0.5" />
 
-        <line y1="0" x1="0" y2="#{height}" x2="0" stroke="#333333" stroke-width="#{width/600}" stroke-opacity="0.5" />
-        <line y1="0" x1="#{width*1/6}" y2="#{height}" x2="#{width*1/6}" stroke="#333333" stroke-width="#{width/600}" stroke-opacity="0.5" />
-        <line y1="0" x1="#{width*2/6}" y2="#{height}" x2="#{width*2/6}" stroke="#333333" stroke-width="#{width/600}" stroke-opacity="0.5" />
-        <line y1="0" x1="#{width*3/6}" y2="#{height}" x2="#{width*3/6}" stroke="#333333" stroke-width="#{width/600}" stroke-opacity="0.5" />
-        <line y1="0" x1="#{width*4/6}" y2="#{height}" x2="#{width*4/6}" stroke="#333333" stroke-width="#{width/600}" stroke-opacity="0.5" />
-        <line y1="0" x1="#{width*5/6}" y2="#{height}" x2="#{width*5/6}" stroke="#333333" stroke-width="#{width/600}" stroke-opacity="0.5" />
-        <line y1="0" x1="#{width}" y2="#{height}" x2="#{width}" stroke="#333333" stroke-width="#{width/600}" stroke-opacity="0.5" />
+        <line y1="0" x1="0" y2="#{height}" x2="0" stroke="#333333" stroke-width="#{width / 600}" stroke-opacity="0.5" />
+        <line y1="0" x1="#{width * 1 / 6}" y2="#{height}" x2="#{width * 1 / 6}" stroke="#333333" stroke-width="#{width / 600}" stroke-opacity="0.5" />
+        <line y1="0" x1="#{width * 2 / 6}" y2="#{height}" x2="#{width * 2 / 6}" stroke="#333333" stroke-width="#{width / 600}" stroke-opacity="0.5" />
+        <line y1="0" x1="#{width * 3 / 6}" y2="#{height}" x2="#{width * 3 / 6}" stroke="#333333" stroke-width="#{width / 600}" stroke-opacity="0.5" />
+        <line y1="0" x1="#{width * 4 / 6}" y2="#{height}" x2="#{width * 4 / 6}" stroke="#333333" stroke-width="#{width / 600}" stroke-opacity="0.5" />
+        <line y1="0" x1="#{width * 5 / 6}" y2="#{height}" x2="#{width * 5 / 6}" stroke="#333333" stroke-width="#{width / 600}" stroke-opacity="0.5" />
+        <line y1="0" x1="#{width}" y2="#{height}" x2="#{width}" stroke="#333333" stroke-width="#{width / 600}" stroke-opacity="0.5" />
 
-        <line x1="#{width/2}" y1="#{height*1/4}" x2="#{width*3/4}" y2="#{height/2}" stroke="#999999" stroke-width="#{width/600}" stroke-opacity="0.5" />
-        <line x1="#{width*3/4}" y1="#{height/2}" x2="#{width/2}" y2="#{height*3/4}" stroke="#999999" stroke-width="#{width/600}" stroke-opacity="0.5" />
-        <line x1="#{width/2}" y1="#{height*3/4}" x2="#{width*1/4}" y2="#{height/2}" stroke="#999999" stroke-width="#{width/600}" stroke-opacity="0.5" />
-        <line x1="#{width*1/4}" y1="#{height/2}" x2="#{width/2}" y2="#{height*1/4}" stroke="#999999" stroke-width="#{width/600}" stroke-opacity="0.5" />
+        <line x1="#{width / 2}" y1="#{height * 1 / 4}" x2="#{width * 3 / 4}" y2="#{height / 2}" stroke="#999999" stroke-width="#{width / 600}" stroke-opacity="0.5" />
+        <line x1="#{width * 3 / 4}" y1="#{height / 2}" x2="#{width / 2}" y2="#{height * 3 / 4}" stroke="#999999" stroke-width="#{width / 600}" stroke-opacity="0.5" />
+        <line x1="#{width / 2}" y1="#{height * 3 / 4}" x2="#{width * 1 / 4}" y2="#{height / 2}" stroke="#999999" stroke-width="#{width / 600}" stroke-opacity="0.5" />
+        <line x1="#{width * 1 / 4}" y1="#{height / 2}" x2="#{width / 2}" y2="#{height * 1 / 4}" stroke="#999999" stroke-width="#{width / 600}" stroke-opacity="0.5" />
 
       SVG
     end
@@ -52,19 +54,19 @@ class USPSFlags::Helpers::Builders
     # @private
     def locator
       <<~SVG
-        <circle cx="0" cy="0" r="#{USPSFlags::Config::BASE_FLY*2}" fill="#000000" fill-opacity="0.4" />
+        <circle cx="0" cy="0" r="#{USPSFlags::Config::BASE_FLY * 2}" fill="#000000" fill-opacity="0.4" />
         <circle cx="0" cy="0" r="#{USPSFlags::Config::BASE_FLY}" fill="#333333" fill-opacity="0.4" />
-        <circle cx="0" cy="0" r="#{USPSFlags::Config::BASE_FLY/2}" fill="#666666" fill-opacity="0.4" />
-        <circle cx="0" cy="0" r="#{USPSFlags::Config::BASE_FLY/4}" fill="#999999" fill-opacity="0.4" />
-        <circle cx="0" cy="0" r="#{USPSFlags::Config::BASE_FLY/8}" fill="#CCCCCC" fill-opacity="0.4" />
-        <circle cx="0" cy="0" r="#{USPSFlags::Config::BASE_FLY/16}" fill="#FFFFFF" fill-opacity="0.4" />
+        <circle cx="0" cy="0" r="#{USPSFlags::Config::BASE_FLY / 2}" fill="#666666" fill-opacity="0.4" />
+        <circle cx="0" cy="0" r="#{USPSFlags::Config::BASE_FLY / 4}" fill="#999999" fill-opacity="0.4" />
+        <circle cx="0" cy="0" r="#{USPSFlags::Config::BASE_FLY / 8}" fill="#CCCCCC" fill-opacity="0.4" />
+        <circle cx="0" cy="0" r="#{USPSFlags::Config::BASE_FLY / 16}" fill="#FFFFFF" fill-opacity="0.4" />
 
-        <line x1="-#{USPSFlags::Config::BASE_FLY}" y1="-#{USPSFlags::Config::BASE_HOIST}" x2="#{USPSFlags::Config::BASE_FLY}" y2="#{USPSFlags::Config::BASE_HOIST}"  stroke="#FFFFFF" stroke-width="#{USPSFlags::Config::BASE_FLY/600}" />
-        <line x1="-#{USPSFlags::Config::BASE_FLY}" y1="#{USPSFlags::Config::BASE_HOIST}" x2="#{USPSFlags::Config::BASE_FLY}" y2="-#{USPSFlags::Config::BASE_HOIST}" stroke="#FFFFFF" stroke-width="#{USPSFlags::Config::BASE_FLY/600}" />
-        <line x1="0" y1="#{USPSFlags::Config::BASE_HOIST}" x2="0" y2="-#{USPSFlags::Config::BASE_HOIST}" stroke="#FFFFFF" stroke-width="#{USPSFlags::Config::BASE_FLY/600}" />
-        <line x1="-#{USPSFlags::Config::BASE_FLY}" y1="0" x2="#{USPSFlags::Config::BASE_FLY}" y2="0" stroke="#FFFFFF" stroke-width="#{USPSFlags::Config::BASE_FLY/600}" />
+        <line x1="-#{USPSFlags::Config::BASE_FLY}" y1="-#{USPSFlags::Config::BASE_HOIST}" x2="#{USPSFlags::Config::BASE_FLY}" y2="#{USPSFlags::Config::BASE_HOIST}"  stroke="#FFFFFF" stroke-width="#{USPSFlags::Config::BASE_FLY / 600}" />
+        <line x1="-#{USPSFlags::Config::BASE_FLY}" y1="#{USPSFlags::Config::BASE_HOIST}" x2="#{USPSFlags::Config::BASE_FLY}" y2="-#{USPSFlags::Config::BASE_HOIST}" stroke="#FFFFFF" stroke-width="#{USPSFlags::Config::BASE_FLY / 600}" />
+        <line x1="0" y1="#{USPSFlags::Config::BASE_HOIST}" x2="0" y2="-#{USPSFlags::Config::BASE_HOIST}" stroke="#FFFFFF" stroke-width="#{USPSFlags::Config::BASE_FLY / 600}" />
+        <line x1="-#{USPSFlags::Config::BASE_FLY}" y1="0" x2="#{USPSFlags::Config::BASE_FLY}" y2="0" stroke="#FFFFFF" stroke-width="#{USPSFlags::Config::BASE_FLY / 600}" />
 
-        <rect x="0" y="0" width="#{USPSFlags::Config::BASE_FLY/30}" height="#{USPSFlags::Config::BASE_FLY/30}" fill="#333333" fill-opacity="0.6" />
+        <rect x="0" y="0" width="#{USPSFlags::Config::BASE_FLY / 30}" height="#{USPSFlags::Config::BASE_FLY / 30}" fill="#333333" fill-opacity="0.6" />
       SVG
     end
   end

data/lib/usps_flags/helpers/spec_arrows.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: false
+
 # Trident specification sheet arrows.
 #
 # These methods should never need to be called directly.
@@ -8,10 +10,10 @@ class USPSFlags::Helpers::SpecArrows
     #
     # This is used USPSFlags::Core.trident_spec, and should never need to be called directly.
     # @private
-    def vertical(x, top, bottom, pointer_top = nil, pointer_bottom = nil, fly: nil, unit: nil, label_offset: (USPSFlags::Config::BASE_FLY/120), label_offset_y: 0, label_align: "left")
+    def vertical(x, top, bottom, pointer_top = nil, pointer_bottom = nil, fly: nil, unit: nil, label_offset: (USPSFlags::Config::BASE_FLY / 120), label_offset_y: 0, label_align: 'left')
       load_common_config(fly)
       label, label_fraction = get_labels(bottom, top)
-      svg = ""
+      svg = ''
 
       svg << arrow_pointer(x, pointer_top, top, top) unless pointer_top.nil?
       svg << arrow_pointer(x, pointer_bottom, bottom, bottom) unless pointer_bottom.nil?
@@ -20,7 +22,7 @@ class USPSFlags::Helpers::SpecArrows
         <path d="M#{x} #{top} l #{@arrow_size} #{@arrow_size} M#{x} #{top} l -#{@arrow_size} #{@arrow_size} M#{x} #{top} l 0 #{bottom - top} l #{@arrow_size} -#{@arrow_size} M#{x} #{bottom} l -#{@arrow_size} -#{@arrow_size}" stroke="#{@color}" stroke-width="#{@stroke_width}" fill="none" />
         <g>
           <style><![CDATA[tspan{font-size: #{USPSFlags::Config::FRACTION_SCALE}%;}]]></style>
-          <text x="#{x + label_offset}" y="#{(top+bottom)/2+(USPSFlags::Config::BASE_HOIST/150)+label_offset_y}" font-family="sans-serif" font-size="#{@font_size}px" fill="#041E42" text-anchor="#{label_align}">#{label} <tspan>#{label_fraction}</tspan> #{unit}</text>
+          <text x="#{x + label_offset}" y="#{(top + bottom) / 2 + (USPSFlags::Config::BASE_HOIST / 150) + label_offset_y}" font-family="sans-serif" font-size="#{@font_size}px" fill="#041E42" text-anchor="#{label_align}">#{label} <tspan>#{label_fraction}</tspan> #{unit}</text>
         </g>
       SVG
 
@@ -31,10 +33,10 @@ class USPSFlags::Helpers::SpecArrows
     #
     # This is used USPSFlags::Core.trident_spec, and should never need to be called directly.
     # @private
-    def horizontal(y, left, right, pointer_left = nil, pointer_right = nil, fly: nil, unit: nil, label_offset: (USPSFlags::Config::BASE_FLY/45), label_offset_x: 0, label_align: "middle")
+    def horizontal(y, left, right, pointer_left = nil, pointer_right = nil, fly: nil, unit: nil, label_offset: (USPSFlags::Config::BASE_FLY / 45), label_offset_x: 0, label_align: 'middle')
       load_common_config(fly)
       label, label_fraction = get_labels(right, left)
-      svg = ""
+      svg = ''
 
       svg << arrow_pointer(left, left, pointer_left, y) unless pointer_left.nil?
       svg << arrow_pointer(right, right, y, pointer_right) unless pointer_right.nil?
@@ -43,20 +45,21 @@ class USPSFlags::Helpers::SpecArrows
         <path d="M#{left} #{y} l #{@arrow_size} #{@arrow_size} M#{left} #{y} l #{@arrow_size} -#{@arrow_size} M#{left} #{y} l #{right - left} 0 l -#{@arrow_size} -#{@arrow_size} M#{right} #{y} l -#{@arrow_size} #{@arrow_size}" stroke="#{@color}" stroke-width="#{@stroke_width}" fill="none" />
         <g>
           <style><![CDATA[tspan{font-size: #{USPSFlags::Config::FRACTION_SCALE}%;}]]></style>
-          <text x="#{(left+right)/2+label_offset_x}" y="#{y + label_offset}" font-family="sans-serif" font-size="#{@font_size}px" fill="#041E42" text-anchor="#{label_align}">#{label} <tspan>#{label_fraction}</tspan> #{unit}</text>
+          <text x="#{(left + right) / 2 + label_offset_x}" y="#{y + label_offset}" font-family="sans-serif" font-size="#{@font_size}px" fill="#041E42" text-anchor="#{label_align}">#{label} <tspan>#{label_fraction}</tspan> #{unit}</text>
         </g>
       SVG
 
       svg
     end
 
-    private
+  private
+
     def load_common_config(fly)
-      @color = "#CCCCCC"
-      @stroke_width = (USPSFlags::Config::BASE_FLY/600)
-      @stroke_dash = "10, 10"
-      @font_size = (USPSFlags::Config::BASE_FLY/60)
-      @arrow_size = (USPSFlags::Config::BASE_FLY/120)
+      @color = '#CCCCCC'
+      @stroke_width = (USPSFlags::Config::BASE_FLY / 600)
+      @stroke_dash = '10, 10'
+      @font_size = (USPSFlags::Config::BASE_FLY / 60)
+      @arrow_size = (USPSFlags::Config::BASE_FLY / 120)
       @fly = fly || USPSFlags::Config::BASE_FLY
     end
 
@@ -70,7 +73,7 @@ class USPSFlags::Helpers::SpecArrows
       label = (a - b) * Rational(@fly, USPSFlags::Config::BASE_FLY)
       if label == label.to_i
         label = label.to_i
-        label_fraction = ""
+        label_fraction = ''
       else
         label, label_fraction = label.to_simplified_a
       end

data/spec/rational_spec.rb

@@ -1,17 +1,19 @@
+# frozen_string_literal: false
+
 require 'spec_helper'
 
 describe Rational do
-  describe "monkey patches" do
-    it "should render a simplified fraction as an Array" do
-      expect(Rational(4,3).to_simplified_a).to eql([1, Rational(1,3)])
+  describe 'monkey patches' do
+    it 'should render a simplified fraction as an Array' do
+      expect(Rational(4, 3).to_simplified_a).to eql([1, Rational(1, 3)])
     end
 
-    it "should render a proper fraction as String" do
-      expect(Rational(2,3).to_simplified_a).to eql("2/3")
+    it 'should render a proper fraction as String' do
+      expect(Rational(2, 3).to_simplified_a).to eql('2/3')
     end
 
-    it "should render a simplified fraction as a String" do
-      expect(Rational(4,3).to_simplified_s).to eql("1 1/3")
+    it 'should render a simplified fraction as a String' do
+      expect(Rational(4, 3).to_simplified_s).to eql('1 1/3')
     end
   end
 end

data/spec/spec_helper.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: false
+
 require 'bundler/setup'
 Bundler.setup
 require 'simplecov'
@@ -10,8 +12,8 @@ RSpec::Expectations.configuration.on_potential_false_positives = :nothing
 
 RSpec.configure do |config|
   config.before(:suite) do
-    $tmp_flags_dir = "tmp/flags"
-    $tmp_alt_flags_dir = "tmp/alt_flags"
+    $tmp_flags_dir = 'tmp/flags'
+    $tmp_alt_flags_dir = 'tmp/alt_flags'
 
     USPSFlags.configure do |c|
       c.flags_dir = $tmp_flags_dir
@@ -20,6 +22,6 @@ RSpec.configure do |config|
   end
 
   config.after(:suite) do
-    ::FileUtils.rm_rf("tmp") if ::Dir.exist?("tmp")
+    ::FileUtils.rm_rf('tmp') if ::Dir.exist?('tmp')
   end
 end

data/spec/usps_flags/config_spec.rb

@@ -1,29 +1,31 @@
+# frozen_string_literal: false
+
 require 'spec_helper'
 
 describe USPSFlags::Config do
-  describe "class variable accessors" do
-    it "should return the current flags directory" do
+  describe 'class variable accessors' do
+    it 'should return the current flags directory' do
       expect(USPSFlags.configuration.flags_dir).to eql($tmp_flags_dir)
     end
 
-    it "should return the current flags directory" do
-      default_log_path = $tmp_flags_dir + "/log"
+    it 'should return the current flags directory' do
+      default_log_path = $tmp_flags_dir + '/log'
       expect(USPSFlags.configuration.log_path).to eql(default_log_path)
     end
 
-    it "should return a Boolean from clear" do
+    it 'should return a Boolean from clear' do
       expect([true, false]).to include(USPSFlags.configuration.clear)
     end
   end
 
-  describe "trident" do
-    it "should return a Hash from trident" do
+  describe 'trident' do
+    it 'should return a Hash from trident' do
       expect(USPSFlags.configuration.trident).to be_a(Hash)
     end
   end
 
-  describe "configuration constructor" do
-    it "should return a properly constructed configuration" do
+  describe 'configuration constructor' do
+    it 'should return a properly constructed configuration' do
       USPSFlags.configure do |config|
         config.flags_dir = $tmp_flags_dir
       end
@@ -33,11 +35,11 @@ describe USPSFlags::Config do
     end
   end
 
-  describe "Rails configuration" do
+  describe 'Rails configuration' do
     before(:each) do
       class Rails
         def self.root
-          "tmp/rails_app"
+          'tmp/rails_app'
         end
       end
 
@@ -46,8 +48,8 @@ describe USPSFlags::Config do
       end
     end
 
-    it "should use the default Rails log directory" do
-      expect(@config.log_path).to eql("tmp/rails_app/log")
+    it 'should use the default Rails log directory' do
+      expect(@config.log_path).to eql('tmp/rails_app/log')
     end
   end
 end