Merge pull request #2395 from cub8/output-currently-compiling-gems

Print currently compiling gems. Add verbose attribute to gem command.

Author: KaanOzkan

lib/tapioca/cli.rb

@@ -301,6 +301,7 @@ def gem(*gems)
         rbi_formatter: rbi_formatter(options),
         halt_upon_load_error: options[:halt_upon_load_error],
         lsp_addon: options[:lsp_addon],
+        verbose: options[:verbose],
       }
 
       command = if verify

lib/tapioca/commands/abstract_gem.rb

@@ -8,7 +8,7 @@ class AbstractGem < Command
       include SorbetHelper
       include RBIFilesHelper
 
-      #: (gem_names: Array[String], exclude: Array[String], include_dependencies: bool, prerequire: String?, postrequire: String, typed_overrides: Hash[String, String], outpath: Pathname, file_header: bool, include_doc: bool, include_loc: bool, include_exported_rbis: bool, ?number_of_workers: Integer?, ?auto_strictness: bool, ?dsl_dir: String, ?rbi_formatter: RBIFormatter, ?halt_upon_load_error: bool, ?lsp_addon: bool?) -> void
+      #: (gem_names: Array[String], exclude: Array[String], include_dependencies: bool, prerequire: String?, postrequire: String, typed_overrides: Hash[String, String], outpath: Pathname, file_header: bool, include_doc: bool, include_loc: bool, include_exported_rbis: bool, ?number_of_workers: Integer?, ?auto_strictness: bool, ?dsl_dir: String, ?rbi_formatter: RBIFormatter, ?halt_upon_load_error: bool, ?lsp_addon: bool?, ?verbose: bool?) -> void
       def initialize(
         gem_names:,
         exclude:,
@@ -26,7 +26,8 @@ def initialize(
         dsl_dir: DEFAULT_DSL_DIR,
         rbi_formatter: DEFAULT_RBI_FORMATTER,
         halt_upon_load_error: true,
-        lsp_addon: false
+        lsp_addon: false,
+        verbose: false
       )
         @gem_names = gem_names
         @exclude = exclude
@@ -41,6 +42,7 @@ def initialize(
         @dsl_dir = dsl_dir
         @rbi_formatter = rbi_formatter
         @lsp_addon = lsp_addon
+        @verbose = verbose
 
         super()
 
@@ -58,6 +60,7 @@ def initialize(
       #: (Gemfile::GemSpec gem) -> void
       def compile_gem_rbi(gem)
         gem_name = set_color(gem.name, :yellow, :bold)
+        say("Currently compiling #{gem_name}", :cyan) if @verbose
 
         rbi = RBI::File.new(strictness: @typed_overrides[gem.name] || "true")
 

spec/tapioca/cli/gem_spec.rb

@@ -381,6 +381,44 @@ module Reopened; end
           assert_success_status(result)
         end
 
+        it "must output currently compiling status with verbose flag" do
+          foo = mock_gem("foo", "0.0.1") do
+            write!("lib/foo.rb", FOO_RB)
+          end
+
+          @project.require_mock_gem(foo)
+          @project.bundle_install!
+
+          result = @project.tapioca("gem foo --verbose")
+
+          assert_stdout_includes(result, "Currently compiling foo")
+          assert_stdout_includes(result, "Compiled foo")
+          assert_stdout_includes(result, "create  sorbet/rbi/gems/foo@0.0.1.rbi")
+
+          assert_project_file_equal("sorbet/rbi/gems/foo@0.0.1.rbi", FOO_RBI)
+          assert_empty_stderr(result)
+          assert_success_status(result)
+        end
+
+        it "must output currently compiling with -V flag" do
+          foo = mock_gem("foo", "0.0.1") do
+            write!("lib/foo.rb", FOO_RB)
+          end
+
+          @project.require_mock_gem(foo)
+          @project.bundle_install!
+
+          result = @project.tapioca("gem foo -V")
+
+          assert_stdout_includes(result, "Currently compiling foo")
+          assert_stdout_includes(result, "Compiled foo")
+          assert_stdout_includes(result, "create  sorbet/rbi/gems/foo@0.0.1.rbi")
+
+          assert_project_file_equal("sorbet/rbi/gems/foo@0.0.1.rbi", FOO_RBI)
+          assert_empty_stderr(result)
+          assert_success_status(result)
+        end
+
         it "must generate a gem RBI without the ones exported from the gem when called with `--no-exported-gem-rbis`" do
           foo = mock_gem("foo", "0.0.1") do
             write!("lib/foo.rb", FOO_RB)