mirror of
https://github.com/ruby/ruby.git
synced 2022-11-09 12:17:21 -05:00
e276482efc
check permissions when necessary. RubyGems bug #465 * test/rubygems/test_gem_config_file.rb: Test for the above. * test/rubygems/test_gem_commands_push_command.rb: Remove duplicated test. git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@39491 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
443 lines
11 KiB
Ruby
443 lines
11 KiB
Ruby
require 'rubygems/test_case'
|
|
require 'rubygems/config_file'
|
|
|
|
class TestGemConfigFile < Gem::TestCase
|
|
|
|
def setup
|
|
super
|
|
|
|
@temp_conf = File.join @tempdir, '.gemrc'
|
|
|
|
@cfg_args = %W[--config-file #{@temp_conf}]
|
|
|
|
@orig_SYSTEM_WIDE_CONFIG_FILE = Gem::ConfigFile::SYSTEM_WIDE_CONFIG_FILE
|
|
Gem::ConfigFile.send :remove_const, :SYSTEM_WIDE_CONFIG_FILE
|
|
Gem::ConfigFile.send :const_set, :SYSTEM_WIDE_CONFIG_FILE,
|
|
File.join(@tempdir, 'system-gemrc')
|
|
Gem::ConfigFile::OPERATING_SYSTEM_DEFAULTS.clear
|
|
Gem::ConfigFile::PLATFORM_DEFAULTS.clear
|
|
|
|
@env_gemrc = ENV['GEMRC']
|
|
ENV['GEMRC'] = ''
|
|
|
|
util_config_file
|
|
end
|
|
|
|
def teardown
|
|
Gem::ConfigFile::OPERATING_SYSTEM_DEFAULTS.clear
|
|
Gem::ConfigFile::PLATFORM_DEFAULTS.clear
|
|
Gem::ConfigFile.send :remove_const, :SYSTEM_WIDE_CONFIG_FILE
|
|
Gem::ConfigFile.send :const_set, :SYSTEM_WIDE_CONFIG_FILE,
|
|
@orig_SYSTEM_WIDE_CONFIG_FILE
|
|
|
|
ENV['GEMRC'] = @env_gemrc
|
|
|
|
super
|
|
end
|
|
|
|
def test_initialize
|
|
assert_equal @temp_conf, @cfg.config_file_name
|
|
|
|
assert_equal false, @cfg.backtrace
|
|
assert_equal true, @cfg.update_sources
|
|
assert_equal Gem::ConfigFile::DEFAULT_BULK_THRESHOLD, @cfg.bulk_threshold
|
|
assert_equal true, @cfg.verbose
|
|
assert_equal [@gem_repo], Gem.sources
|
|
|
|
File.open @temp_conf, 'w' do |fp|
|
|
fp.puts ":backtrace: true"
|
|
fp.puts ":update_sources: false"
|
|
fp.puts ":bulk_threshold: 10"
|
|
fp.puts ":verbose: false"
|
|
fp.puts ":sources:"
|
|
fp.puts " - http://more-gems.example.com"
|
|
fp.puts "install: --wrappers"
|
|
fp.puts ":gempath:"
|
|
fp.puts "- /usr/ruby/1.8/lib/ruby/gems/1.8"
|
|
fp.puts "- /var/ruby/1.8/gem_home"
|
|
fp.puts ":ssl_verify_mode: 0"
|
|
fp.puts ":ssl_ca_cert: /etc/ssl/certs"
|
|
end
|
|
|
|
util_config_file
|
|
|
|
assert_equal true, @cfg.backtrace
|
|
assert_equal 10, @cfg.bulk_threshold
|
|
assert_equal false, @cfg.verbose
|
|
assert_equal false, @cfg.update_sources
|
|
assert_equal %w[http://more-gems.example.com], Gem.sources
|
|
assert_equal '--wrappers', @cfg[:install]
|
|
assert_equal(['/usr/ruby/1.8/lib/ruby/gems/1.8', '/var/ruby/1.8/gem_home'],
|
|
@cfg.path)
|
|
assert_equal 0, @cfg.ssl_verify_mode
|
|
assert_equal '/etc/ssl/certs', @cfg.ssl_ca_cert
|
|
end
|
|
|
|
def test_initialize_handle_arguments_config_file
|
|
util_config_file %W[--config-file #{@temp_conf}]
|
|
|
|
assert_equal @temp_conf, @cfg.config_file_name
|
|
end
|
|
|
|
def test_initialize_handle_arguments_config_file_with_other_params
|
|
util_config_file %W[--config-file #{@temp_conf} --backtrace]
|
|
|
|
assert_equal @temp_conf, @cfg.config_file_name
|
|
end
|
|
|
|
def test_initialize_handle_arguments_config_file_equals
|
|
util_config_file %W[--config-file=#{@temp_conf}]
|
|
|
|
assert_equal @temp_conf, @cfg.config_file_name
|
|
end
|
|
|
|
def test_initialize_operating_system_override
|
|
Gem::ConfigFile::OPERATING_SYSTEM_DEFAULTS[:bulk_threshold] = 1
|
|
Gem::ConfigFile::OPERATING_SYSTEM_DEFAULTS['install'] = '--no-env-shebang'
|
|
|
|
Gem::ConfigFile::PLATFORM_DEFAULTS[:bulk_threshold] = 2
|
|
|
|
util_config_file
|
|
|
|
assert_equal 2, @cfg.bulk_threshold
|
|
assert_equal '--no-env-shebang', @cfg[:install]
|
|
end
|
|
|
|
def test_initialize_platform_override
|
|
Gem::ConfigFile::PLATFORM_DEFAULTS[:bulk_threshold] = 2
|
|
Gem::ConfigFile::PLATFORM_DEFAULTS['install'] = '--no-env-shebang'
|
|
|
|
File.open Gem::ConfigFile::SYSTEM_WIDE_CONFIG_FILE, 'w' do |fp|
|
|
fp.puts ":bulk_threshold: 3"
|
|
end
|
|
|
|
util_config_file
|
|
|
|
assert_equal 3, @cfg.bulk_threshold
|
|
assert_equal '--no-env-shebang', @cfg[:install]
|
|
end
|
|
|
|
def test_initialize_system_wide_override
|
|
File.open Gem::ConfigFile::SYSTEM_WIDE_CONFIG_FILE, 'w' do |fp|
|
|
fp.puts ":backtrace: false"
|
|
fp.puts ":bulk_threshold: 2048"
|
|
end
|
|
|
|
File.open @temp_conf, 'w' do |fp|
|
|
fp.puts ":backtrace: true"
|
|
end
|
|
|
|
util_config_file
|
|
|
|
assert_equal 2048, @cfg.bulk_threshold
|
|
assert_equal true, @cfg.backtrace
|
|
end
|
|
|
|
def test_initialize_environment_variable_override
|
|
File.open Gem::ConfigFile::SYSTEM_WIDE_CONFIG_FILE, 'w' do |fp|
|
|
fp.puts ':backtrace: false'
|
|
fp.puts ':verbose: false'
|
|
fp.puts ':bulk_threshold: 2048'
|
|
end
|
|
|
|
conf1 = File.join @tempdir, 'gemrc1'
|
|
File.open conf1, 'w' do |fp|
|
|
fp.puts ':backtrace: true'
|
|
end
|
|
|
|
conf2 = File.join @tempdir, 'gemrc2'
|
|
File.open conf2, 'w' do |fp|
|
|
fp.puts ':verbose: true'
|
|
end
|
|
|
|
conf3 = File.join @tempdir, 'gemrc3'
|
|
File.open conf3, 'w' do |fp|
|
|
fp.puts ':verbose: :loud'
|
|
end
|
|
|
|
ENV['GEMRC'] = conf1 + ':' + conf2 + ';' + conf3
|
|
|
|
util_config_file
|
|
|
|
assert_equal true, @cfg.backtrace
|
|
assert_equal :loud, @cfg.verbose
|
|
assert_equal 2048, @cfg.bulk_threshold
|
|
end
|
|
|
|
def test_api_keys
|
|
assert_nil @cfg.instance_variable_get :@api_keys
|
|
|
|
temp_cred = File.join Gem.user_home, '.gem', 'credentials'
|
|
FileUtils.mkdir File.dirname(temp_cred)
|
|
File.open temp_cred, 'w', 0600 do |fp|
|
|
fp.puts ':rubygems_api_key: 701229f217cdf23b1344c7b4b54ca97'
|
|
end
|
|
|
|
util_config_file
|
|
|
|
assert_equal({:rubygems => '701229f217cdf23b1344c7b4b54ca97'},
|
|
@cfg.api_keys)
|
|
end
|
|
|
|
def test_check_credentials_permissions
|
|
skip 'chmod not supported' if win_platform?
|
|
|
|
@cfg.rubygems_api_key = 'x'
|
|
|
|
File.chmod 0644, @cfg.credentials_path
|
|
|
|
use_ui @ui do
|
|
assert_raises Gem::MockGemUi::TermError do
|
|
@cfg.load_api_keys
|
|
end
|
|
end
|
|
|
|
assert_empty @ui.output
|
|
|
|
expected = <<-EXPECTED
|
|
ERROR: Your gem push credentials file located at:
|
|
|
|
\t#{@cfg.credentials_path}
|
|
|
|
has file permissions of 0644 but 0600 is required.
|
|
|
|
You should reset your credentials at:
|
|
|
|
\thttps://rubygems.org/profile/edit
|
|
|
|
if you believe they were disclosed to a third party.
|
|
EXPECTED
|
|
|
|
assert_equal expected, @ui.error
|
|
end
|
|
|
|
def test_handle_arguments
|
|
args = %w[--backtrace --bunch --of --args here]
|
|
|
|
@cfg.handle_arguments args
|
|
|
|
assert_equal %w[--bunch --of --args here], @cfg.args
|
|
end
|
|
|
|
def test_handle_arguments_backtrace
|
|
assert_equal false, @cfg.backtrace
|
|
|
|
args = %w[--backtrace]
|
|
|
|
@cfg.handle_arguments args
|
|
|
|
assert_equal true, @cfg.backtrace
|
|
end
|
|
|
|
def test_handle_arguments_debug
|
|
old_dollar_DEBUG = $DEBUG
|
|
assert_equal false, $DEBUG
|
|
|
|
args = %w[--debug]
|
|
|
|
@cfg.handle_arguments args
|
|
|
|
assert_equal true, $DEBUG
|
|
ensure
|
|
$DEBUG = old_dollar_DEBUG
|
|
end
|
|
|
|
def test_handle_arguments_override
|
|
File.open @temp_conf, 'w' do |fp|
|
|
fp.puts ":backtrace: false"
|
|
end
|
|
|
|
util_config_file %W[--backtrace --config-file=#{@temp_conf}]
|
|
|
|
assert_equal true, @cfg.backtrace
|
|
end
|
|
|
|
def test_handle_arguments_traceback
|
|
assert_equal false, @cfg.backtrace
|
|
|
|
args = %w[--traceback]
|
|
|
|
@cfg.handle_arguments args
|
|
|
|
assert_equal true, @cfg.backtrace
|
|
end
|
|
|
|
def test_load_api_keys
|
|
temp_cred = File.join Gem.user_home, '.gem', 'credentials'
|
|
FileUtils.mkdir File.dirname(temp_cred)
|
|
File.open temp_cred, 'w', 0600 do |fp|
|
|
fp.puts ":rubygems_api_key: 701229f217cdf23b1344c7b4b54ca97"
|
|
fp.puts ":other: a5fdbb6ba150cbb83aad2bb2fede64c"
|
|
end
|
|
|
|
util_config_file
|
|
|
|
assert_equal({:rubygems => '701229f217cdf23b1344c7b4b54ca97',
|
|
:other => 'a5fdbb6ba150cbb83aad2bb2fede64c'}, @cfg.api_keys)
|
|
end
|
|
|
|
def test_load_api_keys_bad_permission
|
|
skip 'chmod not supported' if win_platform?
|
|
|
|
@cfg.rubygems_api_key = 'x'
|
|
|
|
File.chmod 0644, @cfg.credentials_path
|
|
|
|
assert_raises Gem::MockGemUi::TermError do
|
|
@cfg.load_api_keys
|
|
end
|
|
end
|
|
|
|
def test_really_verbose
|
|
assert_equal false, @cfg.really_verbose
|
|
|
|
@cfg.verbose = true
|
|
|
|
assert_equal false, @cfg.really_verbose
|
|
|
|
@cfg.verbose = 1
|
|
|
|
assert_equal true, @cfg.really_verbose
|
|
end
|
|
|
|
def test_rubygems_api_key_equals
|
|
@cfg.rubygems_api_key = 'x'
|
|
|
|
assert_equal 'x', @cfg.rubygems_api_key
|
|
|
|
expected = {
|
|
:rubygems_api_key => 'x',
|
|
}
|
|
|
|
assert_equal expected, YAML.load_file(@cfg.credentials_path)
|
|
|
|
unless win_platform? then
|
|
stat = File.stat @cfg.credentials_path
|
|
|
|
assert_equal 0600, stat.mode & 0600
|
|
end
|
|
end
|
|
|
|
def test_rubygems_api_key_equals_bad_permission
|
|
skip 'chmod not supported' if win_platform?
|
|
|
|
@cfg.rubygems_api_key = 'x'
|
|
|
|
File.chmod 0644, @cfg.credentials_path
|
|
|
|
assert_raises Gem::MockGemUi::TermError do
|
|
@cfg.rubygems_api_key = 'y'
|
|
end
|
|
|
|
expected = {
|
|
:rubygems_api_key => 'x',
|
|
}
|
|
|
|
assert_equal expected, YAML.load_file(@cfg.credentials_path)
|
|
|
|
stat = File.stat @cfg.credentials_path
|
|
|
|
assert_equal 0644, stat.mode & 0644
|
|
end
|
|
|
|
def test_write
|
|
@cfg.backtrace = true
|
|
@cfg.update_sources = false
|
|
@cfg.bulk_threshold = 10
|
|
@cfg.verbose = false
|
|
Gem.sources.replace %w[http://more-gems.example.com]
|
|
@cfg[:install] = '--wrappers'
|
|
|
|
@cfg.write
|
|
|
|
util_config_file
|
|
|
|
# These should not be written out to the config file.
|
|
assert_equal false, @cfg.backtrace, 'backtrace'
|
|
assert_equal Gem::ConfigFile::DEFAULT_BULK_THRESHOLD, @cfg.bulk_threshold,
|
|
'bulk_threshold'
|
|
assert_equal true, @cfg.update_sources, 'update_sources'
|
|
assert_equal true, @cfg.verbose, 'verbose'
|
|
|
|
assert_equal '--wrappers', @cfg[:install], 'install'
|
|
|
|
# this should be written out to the config file.
|
|
assert_equal %w[http://more-gems.example.com], Gem.sources
|
|
end
|
|
|
|
def test_write_from_hash
|
|
File.open @temp_conf, 'w' do |fp|
|
|
fp.puts ":backtrace: true"
|
|
fp.puts ":bulk_threshold: 10"
|
|
fp.puts ":update_sources: false"
|
|
fp.puts ":verbose: false"
|
|
fp.puts ":sources:"
|
|
fp.puts " - http://more-gems.example.com"
|
|
fp.puts "install: --wrappers"
|
|
end
|
|
|
|
util_config_file
|
|
|
|
@cfg.backtrace = :junk
|
|
@cfg.update_sources = :junk
|
|
@cfg.bulk_threshold = 20
|
|
@cfg.verbose = :junk
|
|
Gem.sources.replace %w[http://even-more-gems.example.com]
|
|
@cfg[:install] = '--wrappers --no-rdoc'
|
|
|
|
@cfg.write
|
|
|
|
util_config_file
|
|
|
|
# These should not be written out to the config file
|
|
assert_equal true, @cfg.backtrace, 'backtrace'
|
|
assert_equal 10, @cfg.bulk_threshold, 'bulk_threshold'
|
|
assert_equal false, @cfg.update_sources, 'update_sources'
|
|
assert_equal false, @cfg.verbose, 'verbose'
|
|
|
|
assert_equal '--wrappers --no-rdoc', @cfg[:install], 'install'
|
|
|
|
assert_equal %w[http://even-more-gems.example.com], Gem.sources
|
|
end
|
|
|
|
def test_ignore_invalid_config_file
|
|
File.open @temp_conf, 'w' do |fp|
|
|
fp.puts "some-non-yaml-hash-string"
|
|
end
|
|
|
|
# Avoid writing stuff to output when running tests
|
|
Gem::ConfigFile.class_eval { def warn(args); end }
|
|
|
|
# This should not raise exception
|
|
util_config_file
|
|
end
|
|
|
|
def test_load_ssl_verify_mode_from_config
|
|
File.open @temp_conf, 'w' do |fp|
|
|
fp.puts ":ssl_verify_mode: 1"
|
|
end
|
|
util_config_file
|
|
assert_equal(1, @cfg.ssl_verify_mode)
|
|
end
|
|
|
|
def test_load_ssl_ca_cert_from_config
|
|
File.open @temp_conf, 'w' do |fp|
|
|
fp.puts ":ssl_ca_cert: /home/me/certs"
|
|
end
|
|
util_config_file
|
|
assert_equal('/home/me/certs', @cfg.ssl_ca_cert)
|
|
end
|
|
|
|
def util_config_file(args = @cfg_args)
|
|
@cfg = Gem::ConfigFile.new args
|
|
end
|
|
|
|
def test_disable_default_gem_server
|
|
File.open @temp_conf, 'w' do |fp|
|
|
fp.puts ":disable_default_gem_server: true"
|
|
end
|
|
util_config_file
|
|
assert_equal(true, @cfg.disable_default_gem_server)
|
|
end
|
|
end
|
|
|