diff options
| author | Markus Roberts <Markus@reality.com> | 2010-07-09 18:07:15 -0700 |
|---|---|---|
| committer | Markus Roberts <Markus@reality.com> | 2010-07-09 18:07:15 -0700 |
| commit | 543225970225de5697734bfaf0a6eee996802c04 (patch) | |
| tree | ecc6f639c43cf1812e64f9c6ce7eacc0922b57ff /test/network/xmlrpc | |
| parent | 8f15707251cdb58d53e82c4bbd332a38c2d31b4c (diff) | |
| download | puppet-543225970225de5697734bfaf0a6eee996802c04.tar.gz puppet-543225970225de5697734bfaf0a6eee996802c04.tar.xz puppet-543225970225de5697734bfaf0a6eee996802c04.zip | |
Code smell: Avoid needless decorations
* Replaced 704 occurances of (.*)\b([a-z_]+)\(\) with \1\2
3 Examples:
The code:
ctx = OpenSSL::SSL::SSLContext.new()
becomes:
ctx = OpenSSL::SSL::SSLContext.new
The code:
skip()
becomes:
skip
The code:
path = tempfile()
becomes:
path = tempfile
* Replaced 31 occurances of ^( *)end *#.* with \1end
3 Examples:
The code:
becomes:
The code:
end # Dir.foreach
becomes:
end
The code:
end # def
becomes:
end
Diffstat (limited to 'test/network/xmlrpc')
| -rwxr-xr-x | test/network/xmlrpc/processor.rb | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/test/network/xmlrpc/processor.rb b/test/network/xmlrpc/processor.rb index 69f4c2fdc..3bf7b7fa3 100755 --- a/test/network/xmlrpc/processor.rb +++ b/test/network/xmlrpc/processor.rb @@ -35,7 +35,7 @@ class TestXMLRPCProcessor < Test::Unit::TestCase @processor.send(:setup_processor) assert(! @processor.handler_loaded?(:ca), "already have ca handler loaded") assert_nothing_raised do - @processor.add_handler(ca.interface, ca.new()) + @processor.add_handler(ca.interface, ca.new) end assert(@processor.handler_loaded?(:puppetca), "ca handler not loaded by symbol") @@ -46,7 +46,7 @@ class TestXMLRPCProcessor < Test::Unit::TestCase ca = Puppet::Network::Handler.ca @processor.send(:setup_processor) assert_nothing_raised do - @processor.add_handler(ca.interface, ca.new()) + @processor.add_handler(ca.interface, ca.new) end fakeparser = Class.new do |
