diff options
Diffstat (limited to 'lib/options.rb')
| -rw-r--r-- | lib/options.rb | 47 |
1 files changed, 29 insertions, 18 deletions
diff --git a/lib/options.rb b/lib/options.rb index 6ce2479..41a6a50 100644 --- a/lib/options.rb +++ b/lib/options.rb @@ -4,30 +4,41 @@ module RCM # Command line options, supports both Rake mode (args after --) # and standalone mode (direct args). Unknown options are ignored # so that test runners and other tools can pass their own flags. + # + # Defaults are set at module load time. Call Options.parse! once at + # the application entry point to overlay them with actual ARGV values. + # Tests that never call parse! safely get the default values. module Options @@options = { debug: false, dry: false, hosts: [] } - parser = OptionParser.new do |opts| - opts.banner = 'Usage: rake [task] -- [options] OR ruby config.rb [options]' - opts.on('-v', '--[no-]debug', 'debug output') { |v| @@options[:debug] = v } - opts.on('-d', '--dry', 'dry mode') { |v| @@options[:dry] = v } - opts.on('--hosts HOSTS', 'comma-separated list of target hostnames') do |v| - @@options[:hosts] = v.split(',').map(&:strip) + # Parse ARGV and update @@options. Resets to defaults before each + # parse so stale values cannot accumulate across repeated calls + # (e.g. between test cases). + def self.parse! + @@options = { debug: false, dry: false, hosts: [] } + + parser = OptionParser.new do |opts| + opts.banner = 'Usage: rake [task] -- [options] OR ruby config.rb [options]' + opts.on('-v', '--[no-]debug', 'debug output') { |v| @@options[:debug] = v } + opts.on('-d', '--dry', 'dry mode') { |v| @@options[:dry] = v } + opts.on('--hosts HOSTS', 'comma-separated list of target hostnames') do |v| + @@options[:hosts] = v.split(',').map(&:strip) + end end - end - # Rake passes args after '--'; standalone scripts pass args directly. - args = if ARGV.include?('--') - ARGV.slice_before('--').to_a.last.drop(1) - else - ARGV.dup - end + # Rake passes args after '--'; standalone scripts pass args directly. + args = if ARGV.include?('--') + ARGV.slice_before('--').to_a.last.drop(1) + else + ARGV.dup + end - # Ignore unknown options (e.g. from test runners or other tools) - begin - parser.parse!(args) - rescue OptionParser::InvalidOption - retry + # Ignore unknown options (e.g. flags from test runners or rake itself). + begin + parser.parse!(args) + rescue OptionParser::InvalidOption + retry + end end def option(key) |