mirror of
https://github.com/ruby/ruby.git
synced 2022-11-09 12:17:21 -05:00
b957c3dbcb
This changes "test/rubygems/test_case.rb" to "test/rubygems/helper.rb",
and "test/rubygems/test_utilities.rb" to "test/rubygems/utilities.rb".
The two files are a helper for tests, not test files. However, a file
starting with "test_" prefix is handled as a test file directly loaded
by test-unit because Rakefile specifies:
```
t.test_files = FileList['test/**/test_*.rb']
```
Directly loading test/rubygems/test_utilities.rb caused "uninitialized
constant Gem::TestCase". This issue was fixed by
59c6820971, but the fix caused a
"circular require" warning because test_utilities.rb and test_case.rb
are now requiring each other.
Anyway, adding "test_" prefix to a test helper file is confusing, so
this changeset reverts the fix and solve the issue by renaming them.
https://github.com/rubygems/rubygems/commit/6460e018df
61 lines
1.7 KiB
Ruby
61 lines
1.7 KiB
Ruby
# frozen_string_literal: true
|
|
require_relative 'helper'
|
|
require 'rubygems/commands/server_command'
|
|
|
|
class TestGemCommandsServerCommand < Gem::TestCase
|
|
def setup
|
|
super
|
|
|
|
@cmd = Gem::Commands::ServerCommand.new
|
|
end
|
|
|
|
def test_handle_options
|
|
@cmd.send :handle_options, %w[-p 8808 --no-daemon]
|
|
|
|
assert_equal false, @cmd.options[:daemon]
|
|
assert_equal [], @cmd.options[:gemdir]
|
|
assert_equal 8808, @cmd.options[:port]
|
|
|
|
@cmd.send :handle_options, %w[-p 9999 -d /nonexistent --daemon]
|
|
|
|
assert_equal true, @cmd.options[:daemon]
|
|
assert_equal [File.expand_path('/nonexistent')], @cmd.options[:gemdir]
|
|
assert_equal 9999, @cmd.options[:port]
|
|
end
|
|
|
|
def test_handle_options_gemdir
|
|
@cmd.send :handle_options, %w[--dir a --dir b]
|
|
|
|
assert_equal [File.expand_path('a'), File.expand_path('b')],
|
|
@cmd.options[:gemdir]
|
|
end
|
|
|
|
def test_handle_options_port
|
|
@cmd.send :handle_options, %w[-p 0]
|
|
assert_equal 0, @cmd.options[:port]
|
|
|
|
@cmd.send :handle_options, %w[-p 65535]
|
|
assert_equal 65535, @cmd.options[:port]
|
|
|
|
begin
|
|
@cmd.send :handle_options, %w[-p discard]
|
|
assert_equal 9, @cmd.options[:port]
|
|
rescue OptionParser::InvalidArgument
|
|
# for container environment on GitHub Actions
|
|
end
|
|
|
|
e = assert_raise OptionParser::InvalidArgument do
|
|
@cmd.send :handle_options, %w[-p nonexistent]
|
|
end
|
|
|
|
assert_equal 'invalid argument: -p nonexistent: no such named service',
|
|
e.message
|
|
|
|
e = assert_raise OptionParser::InvalidArgument do
|
|
@cmd.send :handle_options, %w[-p 65536]
|
|
end
|
|
|
|
assert_equal 'invalid argument: -p 65536: not a port number',
|
|
e.message
|
|
end
|
|
end
|