1
0
Fork 0
mirror of https://github.com/ruby/ruby.git synced 2022-11-09 12:17:21 -05:00

[rubygems/rubygems] Revert "Cleaup unnecessary code"

Unclear why, but https://github.com/rubygems/rubygems/commit/2e05dadbc5de created some
warnings in ruby-core CI, so let's revert it.

https://github.com/rubygems/rubygems/commit/729ce3a6e1
This commit is contained in:
David Rodríguez 2022-10-03 11:21:42 +02:00 committed by git
parent b91f685a26
commit b7a61cb485
2 changed files with 20 additions and 0 deletions

View file

@ -1,6 +1,15 @@
# frozen_string_literal: true
require "rubygems/command_manager"
##
# This is an example of exactly what NOT to do.
#
# DO NOT include code like this in your rubygems_plugin.rb
module Gem::Commands
remove_const(:InterruptCommand) if defined?(InterruptCommand)
end
class Gem::Commands::InterruptCommand < Gem::Command
def initialize
super("interrupt", "Raises an Interrupt Exception", {})

View file

@ -20,6 +20,8 @@ class TestGem < Gem::TestCase
common_installer_setup
@additional = %w[a b].map {|d| File.join @tempdir, d }
util_remove_interrupt_command
end
def test_self_finish_resolve
@ -1548,10 +1550,14 @@ class TestGem < Gem::TestCase
with_plugin("load") { Gem.load_env_plugins }
assert_equal :loaded, TEST_PLUGIN_LOAD rescue nil
util_remove_interrupt_command
# Should attempt to cause a StandardError
with_plugin("standarderror") { Gem.load_env_plugins }
assert_equal :loaded, TEST_PLUGIN_STANDARDERROR rescue nil
util_remove_interrupt_command
# Should attempt to cause an Exception
with_plugin("exception") { Gem.load_env_plugins }
assert_equal :loaded, TEST_PLUGIN_EXCEPTION rescue nil
@ -2101,6 +2107,11 @@ You may need to `bundle install` to install missing gems
spec
end
def util_remove_interrupt_command
Gem::Commands.send :remove_const, :InterruptCommand if
Gem::Commands.const_defined? :InterruptCommand
end
def util_cache_dir
File.join Gem.dir, "cache"
end