mirror of
https://github.com/ruby/ruby.git
synced 2026-01-26 12:14:51 +00:00
[ruby/rubygems] bin/rubocop -a --only Style/RedundantParentheses
https://github.com/ruby/rubygems/commit/be3b09c786
This commit is contained in:
parent
2c546071df
commit
e68c0fc75b
@ -26,7 +26,7 @@ module Bundler
|
||||
end
|
||||
a, b, c, d, e = *words
|
||||
(16..79).each do |i|
|
||||
w[i] = SHA1_MASK & rotate((w[i - 3] ^ w[i - 8] ^ w[i - 14] ^ w[i - 16]), 1)
|
||||
w[i] = SHA1_MASK & rotate(w[i - 3] ^ w[i - 8] ^ w[i - 14] ^ w[i - 16], 1)
|
||||
end
|
||||
0.upto(79) do |i|
|
||||
case i
|
||||
|
||||
@ -121,7 +121,7 @@ module Bundler
|
||||
FileUtils.rm_rf(p)
|
||||
end
|
||||
git "clone", "--no-checkout", "--quiet", path.to_s, destination.to_s
|
||||
File.chmod(((File.stat(destination).mode | 0o777) & ~File.umask), destination)
|
||||
File.chmod((File.stat(destination).mode | 0o777) & ~File.umask, destination)
|
||||
rescue Errno::EEXIST => e
|
||||
file_path = e.message[%r{.*?((?:[a-zA-Z]:)?/.*)}, 1]
|
||||
raise GitError, "Bundler could not install a gem because it needs to " \
|
||||
|
||||
@ -158,7 +158,7 @@ class Gem::Commands::CertCommand < Gem::Command
|
||||
cert = Gem::Security.create_cert_email(
|
||||
email,
|
||||
key,
|
||||
(Gem::Security::ONE_DAY * expiration_length_days)
|
||||
Gem::Security::ONE_DAY * expiration_length_days
|
||||
)
|
||||
|
||||
Gem::Security.write cert, "gem-public_cert.pem"
|
||||
|
||||
@ -345,7 +345,7 @@ if you believe they were disclosed to a third party.
|
||||
require "fileutils"
|
||||
FileUtils.mkdir_p(dirname)
|
||||
|
||||
permissions = 0o600 & (~File.umask)
|
||||
permissions = 0o600 & ~File.umask
|
||||
File.open(credentials_path, "w", permissions) do |f|
|
||||
f.write self.class.dump_with_rubygems_yaml(config)
|
||||
end
|
||||
|
||||
@ -113,7 +113,7 @@ class Gem::Doctor
|
||||
next if installed_specs.include? basename
|
||||
next if /^rubygems-\d/.match?(basename)
|
||||
next if sub_directory == "specifications" && basename == "default"
|
||||
next if sub_directory == "plugins" && Gem.plugin_suffix_regexp =~ (basename)
|
||||
next if sub_directory == "plugins" && Gem.plugin_suffix_regexp =~ basename
|
||||
|
||||
type = File.directory?(child) ? "directory" : "file"
|
||||
|
||||
|
||||
@ -52,7 +52,7 @@ class Gem::Security::Signer
|
||||
re_signed_cert = Gem::Security.re_sign(
|
||||
expired_cert,
|
||||
private_key,
|
||||
(Gem::Security::ONE_DAY * Gem.configuration.cert_expiration_length_days)
|
||||
Gem::Security::ONE_DAY * Gem.configuration.cert_expiration_length_days
|
||||
)
|
||||
|
||||
Gem::Security.write(re_signed_cert, expired_cert_path)
|
||||
|
||||
@ -498,7 +498,7 @@ Removed '/CN=alternate/DC=example'
|
||||
|
||||
assert_equal "/CN=nobody/DC=example", cert.issuer.to_s
|
||||
|
||||
mask = 0o100600 & (~File.umask)
|
||||
mask = 0o100600 & ~File.umask
|
||||
|
||||
assert_equal mask, File.stat(path).mode unless Gem.win_platform?
|
||||
end
|
||||
@ -527,7 +527,7 @@ Removed '/CN=alternate/DC=example'
|
||||
|
||||
assert_equal "/CN=nobody/DC=example", cert.issuer.to_s
|
||||
|
||||
mask = 0o100600 & (~File.umask)
|
||||
mask = 0o100600 & ~File.umask
|
||||
|
||||
assert_equal mask, File.stat(path).mode unless Gem.win_platform?
|
||||
end
|
||||
@ -559,7 +559,7 @@ Removed '/CN=alternate/DC=example'
|
||||
|
||||
assert_equal "/CN=nobody/DC=example", cert.issuer.to_s
|
||||
|
||||
mask = 0o100600 & (~File.umask)
|
||||
mask = 0o100600 & ~File.umask
|
||||
|
||||
assert_equal mask, File.stat(path).mode unless Gem.win_platform?
|
||||
end
|
||||
@ -591,7 +591,7 @@ Removed '/CN=alternate/DC=example'
|
||||
|
||||
assert_equal "/CN=nobody/DC=example", cert.issuer.to_s
|
||||
|
||||
mask = 0o100600 & (~File.umask)
|
||||
mask = 0o100600 & ~File.umask
|
||||
|
||||
assert_equal mask, File.stat(path).mode unless Gem.win_platform?
|
||||
end
|
||||
|
||||
@ -506,7 +506,7 @@ class TestGemPackage < Gem::Package::TarTestCase
|
||||
extracted = File.join @destination, "lib/code.rb"
|
||||
assert_path_exist extracted
|
||||
|
||||
mask = 0o100666 & (~File.umask)
|
||||
mask = 0o100666 & ~File.umask
|
||||
|
||||
assert_equal mask.to_s(8), File.stat(extracted).mode.to_s(8) unless
|
||||
Gem.win_platform?
|
||||
|
||||
@ -39,7 +39,7 @@ unless Gem.java_platform? # jruby can't require the simple_gem file
|
||||
extracted = File.join @destination, "lib/foo.rb"
|
||||
assert_path_exist extracted
|
||||
|
||||
mask = 0o100644 & (~File.umask)
|
||||
mask = 0o100644 & ~File.umask
|
||||
|
||||
assert_equal mask, File.stat(extracted).mode unless Gem.win_platform?
|
||||
end
|
||||
|
||||
@ -263,19 +263,19 @@ class TestGemPlatform < Gem::TestCase
|
||||
x86_darwin8 = Gem::Platform.new "i686-darwin8.0"
|
||||
|
||||
util_set_arch "powerpc-darwin8"
|
||||
assert((ppc_darwin8 === Gem::Platform.local), "powerpc =~ universal")
|
||||
assert((uni_darwin8 === Gem::Platform.local), "powerpc =~ universal")
|
||||
refute((x86_darwin8 === Gem::Platform.local), "powerpc =~ universal")
|
||||
assert(ppc_darwin8 === Gem::Platform.local, "powerpc =~ universal")
|
||||
assert(uni_darwin8 === Gem::Platform.local, "powerpc =~ universal")
|
||||
refute(x86_darwin8 === Gem::Platform.local, "powerpc =~ universal")
|
||||
|
||||
util_set_arch "i686-darwin8"
|
||||
refute((ppc_darwin8 === Gem::Platform.local), "powerpc =~ universal")
|
||||
assert((uni_darwin8 === Gem::Platform.local), "x86 =~ universal")
|
||||
assert((x86_darwin8 === Gem::Platform.local), "powerpc =~ universal")
|
||||
refute(ppc_darwin8 === Gem::Platform.local, "powerpc =~ universal")
|
||||
assert(uni_darwin8 === Gem::Platform.local, "x86 =~ universal")
|
||||
assert(x86_darwin8 === Gem::Platform.local, "powerpc =~ universal")
|
||||
|
||||
util_set_arch "universal-darwin8"
|
||||
assert((ppc_darwin8 === Gem::Platform.local), "universal =~ ppc")
|
||||
assert((uni_darwin8 === Gem::Platform.local), "universal =~ universal")
|
||||
assert((x86_darwin8 === Gem::Platform.local), "universal =~ x86")
|
||||
assert(ppc_darwin8 === Gem::Platform.local, "universal =~ ppc")
|
||||
assert(uni_darwin8 === Gem::Platform.local, "universal =~ universal")
|
||||
assert(x86_darwin8 === Gem::Platform.local, "universal =~ x86")
|
||||
end
|
||||
|
||||
def test_nil_cpu_arch_is_treated_as_universal
|
||||
@ -283,18 +283,18 @@ class TestGemPlatform < Gem::TestCase
|
||||
with_uni_arch = Gem::Platform.new ["universal", "mingw32"]
|
||||
with_x86_arch = Gem::Platform.new ["x86", "mingw32"]
|
||||
|
||||
assert((with_nil_arch === with_uni_arch), "nil =~ universal")
|
||||
assert((with_uni_arch === with_nil_arch), "universal =~ nil")
|
||||
assert((with_nil_arch === with_x86_arch), "nil =~ x86")
|
||||
assert((with_x86_arch === with_nil_arch), "x86 =~ nil")
|
||||
assert(with_nil_arch === with_uni_arch, "nil =~ universal")
|
||||
assert(with_uni_arch === with_nil_arch, "universal =~ nil")
|
||||
assert(with_nil_arch === with_x86_arch, "nil =~ x86")
|
||||
assert(with_x86_arch === with_nil_arch, "x86 =~ nil")
|
||||
end
|
||||
|
||||
def test_nil_version_is_treated_as_any_version
|
||||
x86_darwin_8 = Gem::Platform.new "i686-darwin8.0"
|
||||
x86_darwin_nil = Gem::Platform.new "i686-darwin"
|
||||
|
||||
assert((x86_darwin_8 === x86_darwin_nil), "8.0 =~ nil")
|
||||
assert((x86_darwin_nil === x86_darwin_8), "nil =~ 8.0")
|
||||
assert(x86_darwin_8 === x86_darwin_nil, "8.0 =~ nil")
|
||||
assert(x86_darwin_nil === x86_darwin_8, "nil =~ 8.0")
|
||||
end
|
||||
|
||||
def test_nil_version_is_stricter_for_linux_os
|
||||
@ -388,24 +388,24 @@ class TestGemPlatform < Gem::TestCase
|
||||
arm64 = Gem::Platform.new "arm64-linux"
|
||||
|
||||
util_set_arch "armv5-linux"
|
||||
assert((arm === Gem::Platform.local), "arm === armv5")
|
||||
assert((armv5 === Gem::Platform.local), "armv5 === armv5")
|
||||
refute((armv7 === Gem::Platform.local), "armv7 === armv5")
|
||||
refute((arm64 === Gem::Platform.local), "arm64 === armv5")
|
||||
refute((Gem::Platform.local === arm), "armv5 === arm")
|
||||
assert(arm === Gem::Platform.local, "arm === armv5")
|
||||
assert(armv5 === Gem::Platform.local, "armv5 === armv5")
|
||||
refute(armv7 === Gem::Platform.local, "armv7 === armv5")
|
||||
refute(arm64 === Gem::Platform.local, "arm64 === armv5")
|
||||
refute(Gem::Platform.local === arm, "armv5 === arm")
|
||||
|
||||
util_set_arch "armv7-linux"
|
||||
assert((arm === Gem::Platform.local), "arm === armv7")
|
||||
refute((armv5 === Gem::Platform.local), "armv5 === armv7")
|
||||
assert((armv7 === Gem::Platform.local), "armv7 === armv7")
|
||||
refute((arm64 === Gem::Platform.local), "arm64 === armv7")
|
||||
refute((Gem::Platform.local === arm), "armv7 === arm")
|
||||
assert(arm === Gem::Platform.local, "arm === armv7")
|
||||
refute(armv5 === Gem::Platform.local, "armv5 === armv7")
|
||||
assert(armv7 === Gem::Platform.local, "armv7 === armv7")
|
||||
refute(arm64 === Gem::Platform.local, "arm64 === armv7")
|
||||
refute(Gem::Platform.local === arm, "armv7 === arm")
|
||||
|
||||
util_set_arch "arm64-linux"
|
||||
refute((arm === Gem::Platform.local), "arm === arm64")
|
||||
refute((armv5 === Gem::Platform.local), "armv5 === arm64")
|
||||
refute((armv7 === Gem::Platform.local), "armv7 === arm64")
|
||||
assert((arm64 === Gem::Platform.local), "arm64 === arm64")
|
||||
refute(arm === Gem::Platform.local, "arm === arm64")
|
||||
refute(armv5 === Gem::Platform.local, "armv5 === arm64")
|
||||
refute(armv7 === Gem::Platform.local, "armv7 === arm64")
|
||||
assert(arm64 === Gem::Platform.local, "arm64 === arm64")
|
||||
end
|
||||
|
||||
def test_equals3_universal_mingw
|
||||
@ -413,8 +413,8 @@ class TestGemPlatform < Gem::TestCase
|
||||
mingw_ucrt = Gem::Platform.new "x64-mingw-ucrt"
|
||||
|
||||
util_set_arch "x64-mingw-ucrt"
|
||||
assert((uni_mingw === Gem::Platform.local), "uni_mingw === mingw_ucrt")
|
||||
assert((mingw_ucrt === Gem::Platform.local), "mingw_ucrt === mingw_ucrt")
|
||||
assert(uni_mingw === Gem::Platform.local, "uni_mingw === mingw_ucrt")
|
||||
assert(mingw_ucrt === Gem::Platform.local, "mingw_ucrt === mingw_ucrt")
|
||||
end
|
||||
|
||||
def test_equals3_version
|
||||
@ -425,11 +425,11 @@ class TestGemPlatform < Gem::TestCase
|
||||
x86_darwin8 = Gem::Platform.new ["x86", "darwin", "8"]
|
||||
x86_darwin9 = Gem::Platform.new ["x86", "darwin", "9"]
|
||||
|
||||
assert((x86_darwin === Gem::Platform.local), "x86_darwin === x86_darwin8")
|
||||
assert((x86_darwin8 === Gem::Platform.local), "x86_darwin8 === x86_darwin8")
|
||||
assert(x86_darwin === Gem::Platform.local, "x86_darwin === x86_darwin8")
|
||||
assert(x86_darwin8 === Gem::Platform.local, "x86_darwin8 === x86_darwin8")
|
||||
|
||||
refute((x86_darwin7 === Gem::Platform.local), "x86_darwin7 === x86_darwin8")
|
||||
refute((x86_darwin9 === Gem::Platform.local), "x86_darwin9 === x86_darwin8")
|
||||
refute(x86_darwin7 === Gem::Platform.local, "x86_darwin7 === x86_darwin8")
|
||||
refute(x86_darwin9 === Gem::Platform.local, "x86_darwin9 === x86_darwin8")
|
||||
end
|
||||
|
||||
def test_equals_tilde
|
||||
|
||||
@ -56,7 +56,7 @@ class TestGemSecurityTrustDir < Gem::TestCase
|
||||
|
||||
assert_path_exist trusted
|
||||
|
||||
mask = 0o100600 & (~File.umask)
|
||||
mask = 0o100600 & ~File.umask
|
||||
|
||||
assert_equal mask, File.stat(trusted).mode unless Gem.win_platform?
|
||||
|
||||
@ -70,7 +70,7 @@ class TestGemSecurityTrustDir < Gem::TestCase
|
||||
|
||||
assert_path_exist @dest_dir
|
||||
|
||||
mask = 0o040700 & (~File.umask)
|
||||
mask = 0o040700 & ~File.umask
|
||||
mask |= 0o200000 if RUBY_PLATFORM.include?("aix")
|
||||
|
||||
assert_equal mask, File.stat(@dest_dir).mode unless Gem.win_platform?
|
||||
@ -91,7 +91,7 @@ class TestGemSecurityTrustDir < Gem::TestCase
|
||||
|
||||
@trust_dir.verify
|
||||
|
||||
mask = 0o40700 & (~File.umask)
|
||||
mask = 0o40700 & ~File.umask
|
||||
mask |= 0o200000 if RUBY_PLATFORM.include?("aix")
|
||||
|
||||
assert_equal mask, File.stat(@dest_dir).mode unless Gem.win_platform?
|
||||
|
||||
@ -2216,9 +2216,9 @@ dependencies: []
|
||||
s1 = util_spec "a", "1"
|
||||
s2 = util_spec "b", "1"
|
||||
|
||||
assert_equal(-1, (s1 <=> s2))
|
||||
assert_equal(0, (s1 <=> s1)) # rubocop:disable Lint/BinaryOperatorWithIdenticalOperands
|
||||
assert_equal(1, (s2 <=> s1))
|
||||
assert_equal(-1, s1 <=> s2)
|
||||
assert_equal(0, s1 <=> s1) # rubocop:disable Lint/BinaryOperatorWithIdenticalOperands
|
||||
assert_equal(1, s2 <=> s1)
|
||||
end
|
||||
|
||||
def test_spaceship_platform
|
||||
@ -2227,18 +2227,18 @@ dependencies: []
|
||||
s.platform = Gem::Platform.new "x86-my_platform1"
|
||||
end
|
||||
|
||||
assert_equal(-1, (s1 <=> s2))
|
||||
assert_equal(0, (s1 <=> s1)) # rubocop:disable Lint/BinaryOperatorWithIdenticalOperands
|
||||
assert_equal(1, (s2 <=> s1))
|
||||
assert_equal(-1, s1 <=> s2)
|
||||
assert_equal(0, s1 <=> s1) # rubocop:disable Lint/BinaryOperatorWithIdenticalOperands
|
||||
assert_equal(1, s2 <=> s1)
|
||||
end
|
||||
|
||||
def test_spaceship_version
|
||||
s1 = util_spec "a", "1"
|
||||
s2 = util_spec "a", "2"
|
||||
|
||||
assert_equal(-1, (s1 <=> s2))
|
||||
assert_equal(0, (s1 <=> s1)) # rubocop:disable Lint/BinaryOperatorWithIdenticalOperands
|
||||
assert_equal(1, (s2 <=> s1))
|
||||
assert_equal(-1, s1 <=> s2)
|
||||
assert_equal(0, s1 <=> s1) # rubocop:disable Lint/BinaryOperatorWithIdenticalOperands
|
||||
assert_equal(1, s2 <=> s1)
|
||||
end
|
||||
|
||||
def test_spec_file
|
||||
|
||||
Loading…
x
Reference in New Issue
Block a user