resolve pr comments

Author: TheZero0-ctrl

lib/generators/boring/devise/jwt/install/install_generator.rb

@@ -48,14 +48,12 @@ def verify_presence_of_devise_model
 
         def add_devise_jwt_gem
           say "Adding devise-jwt gem", :green
-          Bundler.with_unbundled_env do
-            run "bundle add devise-jwt"
-          end
+          gem "devise-jwt"
         end
 
         def add_devise_jwt_config_to_devise_initializer
           say "Adding devise-jwt configurations to a file `config/initializers/devise.rb`", :green
-          say "Value for jwt.secret will be used from `#{devise_jwt_secret_key}`. You can change this values if they don't match with your app.\n", :yellow
+
           jwt_config = <<~RUBY
             config.jwt do |jwt|
               jwt.secret = #{devise_jwt_secret_key}
@@ -72,10 +70,14 @@ def add_devise_jwt_config_to_devise_initializer
           inject_into_file "config/initializers/devise.rb",
                             optimize_indentation(jwt_config, 2),
                             before: /^end\s*\Z/m
+
+          say "❗️❗️\nValue for jwt.secret will be used from `#{devise_jwt_secret_key}`. You can change this values if they don't match with your app.\n",
+              :yellow
         end
 
         def configure_revocation_strategies
-          say "Configuring revocation strategies for #{options[:revocation_strategy]}", :green
+          say "Configuring #{options[:revocation_strategy]} revocation strategy",
+              :green
 
           case options[:revocation_strategy]
           when "JTIMatcher"
@@ -113,7 +115,7 @@ def configure_jti_matcher_strategy
                            verbose: false
 
           add_devise_jwt_module(
-            "self",
+            strategy: "self",
             include_content: "include Devise::JWT::RevocationStrategies::JTIMatcher"
           )
 
@@ -145,7 +147,7 @@ def configure_denylist_strategy
                            before: /^  end/,
                            verbose: false
 
-          add_devise_jwt_module("JwtDenylist")
+          add_devise_jwt_module(strategy: "JwtDenylist")
 
           jwt_denylist_content = <<~RUBY
             include Devise::JWT::RevocationStrategies::Denylist
@@ -183,10 +185,13 @@ def configure_allowlist_strategy
                            before: /^  end/,
                            verbose: false
 
-          add_devise_jwt_module("self", include_content: "include Devise::JWT::RevocationStrategies::Allowlist")
+          add_devise_jwt_module(
+            strategy: "self",
+            include_content: "include Devise::JWT::RevocationStrategies::Allowlist"
+          )
         end
 
-        def add_devise_jwt_module(strategy, include_content: nil)
+        def add_devise_jwt_module(strategy:, include_content: nil)
           model_name = options[:model_name].underscore
           model_content = File.read("app/models/#{model_name}.rb")
           devise_module_pattern = /devise\s*(?:(?:(?::\w+)|(?:\w+:\s*\w+))(?:(?:,\s*:\w+)|(?:,\s*\w+:\s*\w+))*)/ 

test/generators/devise/devise_jwt_install_generator_test.rb

@@ -33,7 +33,7 @@ def test_should_exit_if_devise_model_is_not_present
     end
   end
 
-  def test_should_install_devise_jwt_successfully
+  def test_should_configure_devise_jwt
     Dir.chdir(app_path) do
       setup_devise
       quietly { run_generator }