Pin to bundler v2.1.0

- Resolve lint

bundler/helpers/v2/run.rb

@@ -11,7 +11,7 @@
 
 require "functions"
 
-MIN_BUNDLER_VERSION = "2.0.0"
+MIN_BUNDLER_VERSION = "2.1.0"
 
 def validate_bundler_version!
   return true if correct_bundler_version?

bundler/lib/dependabot/bundler/helpers.rb

@@ -13,15 +13,15 @@ module Helpers
       # it was created with an old version that didn't add this information
       FAILOVER = V1
 
-      BUNDLER_MAJOR_VERSION_REGEX = /BUNDLED WITH\s+(?<version>\d+)\./m
+      BUNDLER_MAJOR_VERSION_REGEX = /BUNDLED WITH\s+(?<version>\d+)\./m.freeze
 
       # NOTE: options is a manditory argument to ensure we pass it from all calling classes
       def self.bundler_version(lockfile, options:)
         # TODO: Remove once bundler 2 is fully supported
         return V1 unless options[:bundler_2_available]
         return DEFAULT unless lockfile
 
-        if matches = lockfile.content.match(BUNDLER_MAJOR_VERSION_REGEX)
+        if (matches = lockfile.content.match(BUNDLER_MAJOR_VERSION_REGEX))
           matches[:version].to_i >= 2 ? V2 : V1
         else
           FAILOVER
@@ -30,7 +30,8 @@ def self.bundler_version(lockfile, options:)
 
       def self.detected_bundler_version(lockfile)
         return "unknown" unless lockfile
-        if matches = lockfile.content.match(BUNDLER_MAJOR_VERSION_REGEX)
+
+        if (matches = lockfile.content.match(BUNDLER_MAJOR_VERSION_REGEX))
           matches[:version]
         else
           FAILOVER

bundler/spec/dependabot/bundler/helper_spec.rb

@@ -13,7 +13,7 @@
     LOCKFILE
   end
 
-  let(:lockfile_bundled_with_1) do
+  let(:lockfile_bundled_with_v1) do
     Dependabot::DependencyFile.new(name: "Gemfile.lock", content: <<~LOCKFILE)
       Mock Gemfile.lock Content Goes Here
 
@@ -22,7 +22,7 @@
     LOCKFILE
   end
 
-  let(:lockfile_bundled_with_2) do
+  let(:lockfile_bundled_with_v2) do
     Dependabot::DependencyFile.new(name: "Gemfile.lock", content: <<~LOCKFILE)
       Mock Gemfile.lock Content Goes Here
 
@@ -46,7 +46,7 @@ def described_method(lockfile)
     end
 
     context "when bundler 2 is not available" do
-      let(:options) { Hash.new }
+      let(:options) { {} }
 
       it "is 1 if there is no lockfile" do
         expect(described_method(no_lockfile)).to eql("1")
@@ -57,11 +57,11 @@ def described_method(lockfile)
       end
 
       it "is 1 if it was bundled with a v1.x version" do
-        expect(described_method(lockfile_bundled_with_1)).to eql("1")
+        expect(described_method(lockfile_bundled_with_v1)).to eql("1")
       end
 
       it "is 1 if it was bundled with a v2.x version" do
-        expect(described_method(lockfile_bundled_with_2)).to eql("1")
+        expect(described_method(lockfile_bundled_with_v2)).to eql("1")
       end
 
       it "is 1 if it was bundled with a future version" do
@@ -83,11 +83,11 @@ def described_method(lockfile)
       end
 
       it "is 1 if it was bundled with a v1.x version" do
-        expect(described_method(lockfile_bundled_with_1)).to eql("1")
+        expect(described_method(lockfile_bundled_with_v1)).to eql("1")
       end
 
       it "is 2 if it was bundled with a v2.x version" do
-        expect(described_method(lockfile_bundled_with_2)).to eql("2")
+        expect(described_method(lockfile_bundled_with_v2)).to eql("2")
       end
 
       it "is 2 if it was bundled with a future version" do
@@ -110,11 +110,11 @@ def described_method(lockfile)
     end
 
     it "is 1 if it was bundled with a v1.x version" do
-      expect(described_method(lockfile_bundled_with_1)).to eql("1")
+      expect(described_method(lockfile_bundled_with_v1)).to eql("1")
     end
 
     it "is 2 if it was bundled with a v2.x version" do
-      expect(described_method(lockfile_bundled_with_2)).to eql("2")
+      expect(described_method(lockfile_bundled_with_v2)).to eql("2")
     end
 
     it "is 1 if it was bundled with a future version" do