Terraform requirements updater - update update_greatest_version to ha…

…ndle both less than and less-than/equal operators

- 'index_to_update' would sometimes pick the middle or first segement
  instead of the last segment leading to the wrong version
segment being incremented
- less-than/equals would always get incremented instead of taking the
  version as-is
- minor or patch version would sometimes get set to 0 once the
  'index_to_update' was set

terraform/lib/dependabot/terraform/requirements_updater.rb

@@ -11,6 +11,8 @@
 require "dependabot/terraform/version"
 require "dependabot/terraform/requirement"
 
+require 'logger'
+
 module Dependabot
   module Terraform
     # Takes an array of `requirements` hashes for a dependency at the old
@@ -190,17 +192,17 @@ def update_greatest_version(requirement, version_to_be_permitted)
         op, version = requirement.requirements.first
         version = version.release if version.prerelease?
 
-        index_to_update =
-          version.segments.map.with_index { |seg, i| seg.zero? ? 0 : i }.max
-
-        new_segments = version.segments.map.with_index do |_, index|
-          if index < index_to_update
+        # When 'less than'/'<', increment the last segment
+        if op == "<"
+          new_segments = version.segments.map.with_index do |_, index|
             version_to_be_permitted.segments[index]
-          elsif index == index_to_update
-            version_to_be_permitted.segments[index].to_i + 1
-          else
-            0
           end
+          new_segments[-1] += 1
+        # When 'less-than/equal'/'<=', use the new version as-is
+        elsif op == "<="
+          new_segments = version_to_be_permitted.segments
+        else
+          raise "Unexpected operation: #{op}"
         end
 
         requirement_class.new("#{op} #{new_segments.join('.')}")