Merge pull request #562 from Shopify/uk-revert-bundler-change

lib/tapioca/gemfile.rb

@@ -114,13 +114,14 @@ def ignore?(gemfile_dir)
 
       sig { returns(T::Array[Pathname]) }
       def files
-        spec = @spec
-        if default_gem? && spec.is_a?(::Gem::Specification)
-          spec.files.map do |file|
+        if default_gem?
+          # `Bundler::RemoteSpecification` delegates missing methods to
+          # `Gem::Specification`, so `files` actually always exists on spec.
+          T.unsafe(@spec).files.map do |file|
             ruby_lib_dir.join(file)
           end
         else
-          spec.full_require_paths.flat_map do |path|
+          @spec.full_require_paths.flat_map do |path|
             Pathname.glob((Pathname.new(path) / "**/*.rb").to_s)
           end
         end

spec/support/repo/Gemfile

@@ -21,6 +21,7 @@ gem("sidekiq", "6.2.1")
 gem("activesupport", "6.1.3.2")
 # Needed to test Git gems
 gem("ast", git: "https://github.com/whitequark/ast", ref: "e07a4f66e05ac7972643a8841e336d327ea78ae1")
+gem("did_you_mean")
 
 # Needed to test missing gems
 platform :truffleruby do

spec/tapioca/cli/gem_spec.rb

@@ -106,6 +106,18 @@ def fizz; end
           refute_path_exists("#{outdir}/[email protected]")
         end
 
+        it "must generate RBI for a default gem" do
+          output = tapioca("gem did_you_mean")
+
+          assert_includes(output, <<~OUTPUT)
+            Processing 'did_you_mean' gem:
+              Compiling did_you_mean, this may take a few seconds...   Done
+          OUTPUT
+
+          did_you_mean_rbi_file = T.must(Dir.glob("#{outdir}/did_you_mean@*.rbi").first)
+          assert_includes(File.read(did_you_mean_rbi_file), "module DidYouMean")
+        end
+
         it "must generate gem RBI in correct output directory" do
           output = tapioca("gem foo", use_default_outdir: true)