writing groups instead of group in the Gemfile

rubygems · Apr 5, 2017 · 1754e50 · 1754e50
1 parent 1f466c0
commit 1754e50
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 2 deletions.
diff --git a/lib/bundler/injector.rb b/lib/bundler/injector.rb
@@ -51,7 +51,14 @@ def new_gem_lines
       @new_deps.map do |d|
         name = "'#{d.name}'"
         requirement = ", '#{d.requirement}'"
-        group = ", :group => #{d.groups.inspect}" if d.groups != Array(:default)
+        if d.groups != Array(:default)
+          group =
+            if d.groups.size == 1
+              ", :group => #{d.groups.inspect}"
+            else
+              ", :groups => #{d.groups.inspect}"
+            end
+        end
         source = ", :source => '#{d.source}'" unless d.source.nil?
         %(gem #{name}#{requirement}#{group}#{source})
       end.join("\n")

diff --git a/spec/commands/inject_spec.rb b/spec/commands/inject_spec.rb
@@ -72,7 +72,7 @@
     it "add gem with multiple groups in gemfile" do
       bundle "inject 'rack-obama' '>0' --group=development,test"
       gemfile = bundled_app("Gemfile").read
-      str = "gem 'rack-obama', '> 0', :group => [:development, :test]"
+      str = "gem 'rack-obama', '> 0', :groups => [:development, :test]"
       expect(gemfile).to include str
     end
   end