Merge pull request rails#53083 from jeromedalbert/fix-bcrypt-auth-generator

Fix bcrypt gem not added by auth generator

Author: byroot

railties/lib/rails/generators/rails/authentication/authentication_generator.rb

@@ -41,7 +41,7 @@ def enable_bcrypt
           uncomment_lines "Gemfile", /gem "bcrypt"/
           Bundler.with_original_env { execute_command :bundle, "install --quiet" }
         else
-          Bundler.with_original_env { execute_command :bundle, "add bcrypt --quiet" }
+          Bundler.with_original_env { execute_command :bundle, "add bcrypt", capture: true }
         end
       end
 

railties/test/generators/authentication_generator_test.rb

@@ -12,7 +12,7 @@ def setup
     Rails.application.config.root = Pathname(destination_root)
 
     self.class.tests Rails::Generators::AppGenerator
-    run_generator([destination_root])
+    run_generator([destination_root, "--no-skip-bundle"])
 
     self.class.tests Rails::Generators::AuthenticationGenerator
   end
@@ -55,6 +55,16 @@ def test_authentication_generator
     end
   end
 
+  def test_authentication_generator_without_bcrypt_in_gemfile
+    File.write("Gemfile", File.read("Gemfile").sub(/# gem "bcrypt".*\n/, ""))
+
+    run_generator
+
+    assert_file "Gemfile" do |content|
+      assert_match(/\ngem "bcrypt"/, content)
+    end
+  end
+
   def test_authentication_generator_with_api_flag
     run_generator(["--api"])