Filter out all support files from evaled_gemfiles() without listing e…

…ach pattern separately
dependabot · Mar 2, 2024 · bc29131 · bc29131
1 parent 72a48f1
commit bc29131
Show file tree

Hide file tree

Showing 6 changed files with 7 additions and 10 deletions.
diff --git a/bundler/lib/dependabot/bundler/file_parser.rb b/bundler/lib/dependabot/bundler/file_parser.rb
@@ -243,11 +243,10 @@ def evaled_gemfiles
           .reject { |f| f.name.end_with?(".gemspec") }
           .reject { |f| f.name.end_with?(".specification") }
           .reject { |f| f.name.end_with?(".lock") }
-          .reject { |f| f.name.end_with?(".ruby-version") }
-          .reject { |f| f.name.end_with?(".tool-versions") }
           .reject { |f| f.name == "Gemfile" }
           .reject { |f| f.name == "gems.rb" }
           .reject { |f| f.name == "gems.locked" }
+          .reject(&:support_file?)
       end
 
       def lockfile

diff --git a/bundler/lib/dependabot/bundler/file_parser/file_preparer.rb b/bundler/lib/dependabot/bundler/file_parser/file_preparer.rb
@@ -50,11 +50,10 @@ def evaled_gemfiles
             .reject { |f| f.name.end_with?(".gemspec") }
             .reject { |f| f.name.end_with?(".specification") }
             .reject { |f| f.name.end_with?(".lock") }
-            .reject { |f| f.name.end_with?(".ruby-version") }
-            .reject { |f| f.name.end_with?(".tool-versions") }
             .reject { |f| f.name == "Gemfile" }
             .reject { |f| f.name == "gems.rb" }
             .reject { |f| f.name == "gems.locked" }
+            .reject(&:support_file?)
         end
 
         def specification_files

diff --git a/bundler/lib/dependabot/bundler/file_updater.rb b/bundler/lib/dependabot/bundler/file_updater.rb
@@ -127,11 +127,10 @@ def evaled_gemfiles
           .reject { |f| f.name.end_with?(".gemspec") }
           .reject { |f| f.name.end_with?(".specification") }
           .reject { |f| f.name.end_with?(".lock") }
-          .reject { |f| f.name.end_with?(".ruby-version") }
-          .reject { |f| f.name.end_with?(".tool-versions") }
           .reject { |f| f.name == "Gemfile" }
           .reject { |f| f.name == "gems.rb" }
           .reject { |f| f.name == "gems.locked" }
+          .reject(&:support_file?)
       end
 
       def updated_gemfile_content(file)

diff --git a/bundler/lib/dependabot/bundler/file_updater/lockfile_updater.rb b/bundler/lib/dependabot/bundler/file_updater/lockfile_updater.rb
@@ -278,8 +278,6 @@ def evaled_gemfiles
             .reject { |f| f.name.end_with?(".gemspec") }
             .reject { |f| f.name.end_with?(".specification") }
             .reject { |f| f.name.end_with?(".lock") }
-            .reject { |f| f.name.end_with?(".ruby-version") }
-            .reject { |f| f.name.end_with?(".tool-versions") }
             .reject { |f| f.name == "Gemfile" }
             .reject { |f| f.name == "gems.rb" }
             .reject { |f| f.name == "gems.locked" }

diff --git a/bundler/lib/dependabot/bundler/update_checker/file_preparer.rb b/bundler/lib/dependabot/bundler/update_checker/file_preparer.rb
@@ -128,11 +128,10 @@ def evaled_gemfiles
             .reject { |f| f.name.end_with?(".gemspec") }
             .reject { |f| f.name.end_with?(".specification") }
             .reject { |f| f.name.end_with?(".lock") }
-            .reject { |f| f.name.end_with?(".ruby-version") }
-            .reject { |f| f.name.end_with?(".tool-versions") }
             .reject { |f| f.name == "Gemfile" }
             .reject { |f| f.name == "gems.rb" }
             .reject { |f| f.name == "gems.locked" }
+            .reject(&:support_file?)
         end
 
         def lockfile

diff --git a/bundler/spec/spec_helper.rb b/bundler/spec/spec_helper.rb
@@ -27,6 +27,9 @@ def self.bundler_version
 
 def bundler_project_dependency_files(project, directory: "/")
   project_dependency_files(File.join("bundler#{PackageManagerHelper.bundler_version}", project), directory: directory)
+    .each do |dep|
+      dep.support_file = dep.name.end_with?(".ruby-version", ".tool-versions")
+    end
 end
 
 def bundler_project_dependency_file(project, filename:)