From bf2c03e375e91e8418967a593362313487f2f0d0 Mon Sep 17 00:00:00 2001
From: Vikram Yadav <vyadav@pivotal.io>
Date: Tue, 3 Dec 2019 12:49:04 -0500
Subject: [PATCH] [Fixed] Bundler ignored groups failure

---
 lib/license_finder/package_managers/bundler.rb           | 2 +-
 spec/lib/license_finder/package_managers/bundler_spec.rb | 8 ++++----
 2 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/lib/license_finder/package_managers/bundler.rb b/lib/license_finder/package_managers/bundler.rb
index 94bd86ca3..08364d06d 100644
--- a/lib/license_finder/package_managers/bundler.rb
+++ b/lib/license_finder/package_managers/bundler.rb
@@ -24,7 +24,7 @@ def self.package_management_command
     end
 
     def prepare_command
-      ignored_groups_argument = !ignored_groups.empty? ? "--without #{ignored_groups.join(' ')}" : ''
+      ignored_groups_argument = !ignored_groups.empty? ? "--without #{ignored_groups.to_a.join(' ')}" : ''
 
       "bundle install #{ignored_groups_argument}".strip
     end
diff --git a/spec/lib/license_finder/package_managers/bundler_spec.rb b/spec/lib/license_finder/package_managers/bundler_spec.rb
index 1b88a28ca..6d6c00f8d 100644
--- a/spec/lib/license_finder/package_managers/bundler_spec.rb
+++ b/spec/lib/license_finder/package_managers/bundler_spec.rb
@@ -28,14 +28,14 @@ def build_gemspec(name, version, dependency = nil)
 
     describe '.prepare_command' do
       context 'with ignored groups' do
-        subject { Bundler.new(ignored_groups: %w[dev test], project_path: Pathname.new('.'), definition: definition) }
+        subject { Bundler.new(ignored_groups: Set.new(%w[dev test]), project_path: Pathname.new('.'), definition: definition) }
         it 'returns the correct prepare method' do
           expect(subject.prepare_command).to eq('bundle install --without dev test')
         end
       end
 
       context 'without ignored groups' do
-        subject { Bundler.new(ignored_groups: %w[], project_path: Pathname.new('.'), definition: definition) }
+        subject { Bundler.new(ignored_groups: Set.new, project_path: Pathname.new('.'), definition: definition) }
         it 'returns the correct prepare method' do
           expect(subject.prepare_command).to eq('bundle install')
         end
@@ -44,7 +44,7 @@ def build_gemspec(name, version, dependency = nil)
 
     describe '.current_packages' do
       subject do
-        Bundler.new(ignored_groups: %w[dev test], project_path: Pathname.new('.'), definition: definition).current_packages
+        Bundler.new(ignored_groups: Set.new(%w[dev test]), project_path: Pathname.new('.'), definition: definition).current_packages
       end
 
       it 'should have 2 dependencies' do
@@ -71,7 +71,7 @@ def build_gemspec(name, version, dependency = nil)
       let(:custom_gemfile) { fixture_path('custom_gemfile') }
 
       subject do
-        Bundler.new(project_path: custom_gemfile, ignored_groups: %w[dev test])
+        Bundler.new(project_path: custom_gemfile, ignored_groups: Set.new(%w[dev test]))
       end
 
       it 'defaults to Gemfile/Gemfile.lock' do