wavefront-client 3.5.3 → 3.5.4

data/lib/wavefront/client/version.rb

@@ -16,6 +16,6 @@ See the License for the specific language governing permissions and
 
 module Wavefront
   class Client
-    VERSION = "3.5.3"
+    VERSION = "3.5.4"
   end
 end

data/lib/wavefront/constants.rb

@@ -15,6 +15,7 @@ See the License for the specific language governing permissions and
 =end
 
 require 'pathname'
+require 'socket'
 
 module Wavefront
   module Constants
@@ -26,12 +27,31 @@ module Wavefront
     DEFAULT_OBSOLETE_METRICS = false
     FORMATS = [ :raw, :ruby, :graphite, :highcharts, :human ]
     ALERT_FORMATS = [:ruby, :json, :human]
+    SOURCE_FORMATS = [:ruby, :json, :human]
     DEFAULT_ALERT_FORMAT = :human
+    DEFAULT_SOURCE_FORMAT = :human
     GRANULARITIES = %w( s m h d )
     EVENT_STATE_DIR = Pathname.new('/var/tmp/wavefront/events')
     EVENT_LEVELS = %w(info smoke warn severe)
     DEFAULT_PROXY = 'wavefront'
     DEFAULT_PROXY_PORT = 2878
     DEFAULT_INFILE_FORMAT = 'tmv'
+
+    # The CLI will use these options if they are not supplied on the
+    # command line or in a config file
+    #
+    DEFAULT_OPTS = {
+      endpoint:     DEFAULT_HOST,          # API endpoint
+      proxy:        'wavefront',           # proxy endpoint
+      port:         DEFAULT_PROXY_PORT,    # proxy port
+      profile:      'default',             # stanza in config file
+      host:         Socket.gethostname,    # source host
+      prefixlength: DEFAULT_PREFIX_LENGTH, # no of prefix path elements
+      strict:       DEFAULT_STRICT,        # points outside query window
+      format:       DEFAULT_FORMAT,        # ts output format
+      alertformat:  DEFAULT_ALERT_FORMAT,  # alert command output format
+      infileformat: DEFAULT_INFILE_FORMAT, # batch writer file format
+      sourceformat: DEFAULT_SOURCE_FORMAT, # source more output format
+    }.freeze
   end
 end

data/lib/wavefront/events.rb

@@ -23,10 +23,17 @@ module Wavefront
     include Wavefront::Mixins
     DEFAULT_PATH = '/api/events/'
 
-    attr_reader :headers
+    attr_reader :headers, :noop, :verbose, :endpoint
 
-    def initialize(token)
+    def initialize(token, host = DEFAULT_HOST, debug = false, options = {})
+      #
+      # Following existing convention, 'host' is the Wavefront API endpoint.
+      #
       @headers = { :'X-AUTH-TOKEN' => token }
+      @endpoint = host
+      debug(debug)
+      @noop = options[:noop]
+      @verbose = options[:verbose]
     end
 
     def create(payload = {}, options = {})
@@ -45,6 +52,13 @@ module Wavefront
       uri = create_uri(path: [DEFAULT_PATH, payload[:startTime],
                        payload[:name]].join('/').squeeze('/'))
 
+      if (verbose || noop)
+        puts "DELETE #{uri.to_s}"
+        puts "HEADERS #{headers}"
+      end
+
+      return if noop
+
       RestClient.delete(uri.to_s, headers)
     end
 
@@ -52,7 +66,7 @@ module Wavefront
       #
       # Build the URI we use to send a 'create' request.
       #
-      options[:host] ||= DEFAULT_HOST
+      options[:host] ||= endpoint
       options[:path] ||= DEFAULT_PATH
 
       URI::HTTPS.build(
@@ -84,7 +98,7 @@ module Wavefront
       #
       # Build the URI we use to send a 'close' request
       #
-      options[:host] ||= DEFAULT_HOST
+      options[:host] ||= endpoint
       options[:path] ||= DEFAULT_PATH
 
       URI::HTTPS.build(
@@ -94,6 +108,14 @@ module Wavefront
     end
 
     def make_call(uri, query)
+      if (verbose || noop)
+        puts "PUT #{uri.to_s}"
+        puts "QUERY #{query}"
+        puts "HEADERS #{headers}"
+      end
+
+      return if noop
+
       RestClient.post(uri.to_s, query, headers)
     end
 

data/lib/wavefront/metadata.rb

@@ -40,7 +40,7 @@ module Wavefront
     include Wavefront::Validators
     DEFAULT_PATH = '/api/manage/source/'.freeze
 
-    attr_reader :headers, :host, :verbose, :endpoint
+    attr_reader :headers, :host, :verbose, :endpoint, :noop
 
     def initialize(token, host=DEFAULT_HOST, debug=false, options = {})
       #
@@ -50,6 +50,7 @@ module Wavefront
       @base_uri = URI::HTTPS.build(:host => host, :path => DEFAULT_PATH)
       @endpoint = host
       @verbose = options[:verbose] || false
+      @noop = options[:noop] || false
       debug(debug)
     end
 
@@ -158,7 +159,8 @@ module Wavefront
         end
       end
 
-      JSON.parse(call_get(build_uri(nil, query: hash_to_qs(params))))
+      resp = call_get(build_uri(nil, query: hash_to_qs(params))) || '{}'
+      JSON.parse(resp)
     end
 
     def show_source(source)
@@ -169,7 +171,9 @@ module Wavefront
       # See the Wavefront API docs for the structure of the object.
       #
       fail Wavefront::Exception::InvalidSource unless valid_source?(source)
-      JSON.parse(call_get(build_uri(source)))
+      resp = call_get(build_uri(source)) || '{}'
+
+      JSON.parse(resp)
     end
 
     def set_description(source, desc)
@@ -206,18 +210,31 @@ module Wavefront
     end
 
     def call_get(uri)
-      puts "GET #{uri.to_s}" if verbose
+      if (verbose || noop)
+        puts 'GET ' + uri.to_s
+        puts 'HEADERS ' + headers.to_s
+      end
+      return if noop
       RestClient.get(uri.to_s, headers)
     end
 
     def call_delete(uri)
-      puts "DELETE #{uri.to_s}" if verbose
+      if (verbose || noop)
+        puts 'DELETE ' + uri.to_s
+        puts 'HEADERS ' + headers.to_s
+      end
+      return if noop
       RestClient.delete(uri.to_s, headers)
     end
 
     def call_post(uri, query = nil)
-      puts "POST #{uri.to_s} #{query}" if verbose
       h = headers
+      if (verbose || noop)
+        puts 'POST ' + uri.to_s
+        puts 'QUERY ' + query if query
+        puts 'HEADERS ' + h.to_s
+      end
+      return if noop
 
       RestClient.post(uri.to_s, query,
                       h.merge(:'Content-Type' => 'text/plain',

data/lib/wavefront/opt_handler.rb

@@ -0,0 +1,61 @@
+require 'inifile'
+require 'pathname'
+
+module Wavefront
+  #
+  # Options to commands can come from three sources, with the
+  # following order of precedence: program defaults, a configuration
+  # file, and command-line options. Docopt is not well suited to
+  # this, as it will "fill in" any missing options with defaults,
+  # producing a single hash which must be merged with values from
+  # the config file. Assuming we give the command-line higher
+  # precedence, a default value, not supplied by the user, will
+  # override a value in the config file. The other way round, and
+  # you can't override anything in the config file from the
+  # command-line. I think this behaviour is far from unique to
+  # Docopt.
+  #
+  # So, we have a hash of defaults, and we do the merging ourselves,
+  # in this class. We trick Docopt into not using the defaults by
+  # avoiding the magic string 'default: ' in our options stanzas.
+  #
+  class OptHandler
+    include Wavefront::Constants
+
+    attr_reader :opts, :cli_opts, :conf_file
+
+    def initialize(conf_file, cli_opts = {})
+      @conf_file = if cli_opts.key?(:config) && cli_opts[:config]
+                     Pathname.new(cli_opts[:config])
+                   else
+                     conf_file
+                   end
+
+      @cli_opts = cli_opts.reject { |_k, v| v.nil? }
+
+      @opts = DEFAULT_OPTS.merge(load_profile).merge(@cli_opts)
+    end
+
+    def load_profile
+      #
+      # Load in configuration options from the (optionally) given
+      # section of an ini-style configuration file. If the file's
+      # not there, we don't consider that an error. Returns a hash
+      # of options which matches what Docopt gives us.
+      #
+      unless conf_file.exist?
+        puts "config file '#{conf_file}' not found. Taking options " \
+             'from command-line.'
+        return {}
+      end
+
+      pf = cli_opts.fetch(:profile, 'default')
+
+      puts "reading '#{pf}' profile from '#{conf_file}'" if cli_opts[:debug]
+
+      IniFile.load(conf_file)[pf].each_with_object({}) do |(k, v), memo|
+        memo[k.to_sym] = v
+      end
+    end
+  end
+end

data/spec/cli_spec.rb

@@ -0,0 +1,584 @@
+#
+# cli_spec.rb
+# -----------
+#
+# This spec file adds acceptance testing for the CLI script `bin/wavefront`. It
+# runs all commands and subcommands with the `--noop` option, and ensures that
+# the expected API calls are made, or the correct error is thrown. It takes
+# ~15-25s to run, and is part of the CI pipeline.
+#
+# None of the tests in this file make real API calls, so no endpoint is
+# required.
+#
+# Tests are run as part of the usual `rake spec` job. Custom RSpec matchers and
+# support methods are in `spec_helper.rb`.
+#
+require 'pathname'
+require 'open3'
+require 'ostruct'
+require 'json'
+require 'socket'
+require 'date'
+require 'uri'
+require_relative './spec_helper'
+require 'wavefront/client/version'
+
+ROOT = Pathname.new(__FILE__).dirname.parent
+WF = ROOT + 'bin' + 'wavefront'
+LIB = ROOT + 'lib'
+CF = ROOT + 'spec' + 'wavefront' + 'resources' + 'conf.yaml'
+
+# Things from the sample config file, for shorthand
+#
+DEF_TOKEN = '12345678-abcd-1234-abcd-123456789012'
+OTHER_TOKEN = 'abcdefab-0123-abcd-0123-abcdefabcdef'
+
+# Some standard start and end times
+#
+TIME = {
+  start: {
+    eng: '12:00',
+    i:    DateTime.parse('12:00').to_time.to_i,
+    ms:   DateTime.parse('12:00').to_time.to_i * 1000
+  },
+  end: {
+    eng: '12:05',
+    i:    DateTime.parse('12:05').to_time.to_i,
+    ms:   DateTime.parse('12:05').to_time.to_i * 1000
+  }
+}
+
+describe 'usage' do
+  commands = %w(alerts event source ts write)
+
+  it 'prints usage and exits 1 with no args' do
+    o = wf
+    expect(o.stdout).to be_empty
+    expect(o.stderr_a.first).to eq('Usage:')
+    expect(o.status).to eq(1)
+  end
+
+  it 'prints detailed usage with --help' do
+    o = wf('--help')
+    expect(o.stdout).to be_empty
+    expect(o.stderr_a.first).to eq('Wavefront CLI')
+    expect(o.status).to eq(1)
+    commands.each { |cmd| expect(o.stderr).to match(/\n  #{cmd} /) }
+  end
+
+  commands.each do |cmd|
+    it "prints help for the #{cmd} command" do
+      o = wf("#{cmd} --help")
+      expect(o.status).to eq(1)
+      expect(o.stdout).to be_empty
+      expect(o.stderr_a.first).to eq('Usage:')
+      expect(o.stderr_a).to include('Global options:')
+    end
+  end
+
+  it 'displays the correct version number' do
+    include Wavefront::Client
+    o = wf('--version')
+    expect(o.status).to eq(1)
+    expect(o.stdout).to be_empty
+    expect(o.stderr).to eq(Wavefront::Client::VERSION)
+  end
+end
+
+describe 'alerts mode' do
+  it 'fails with no token if there is no token' do
+    o = wf('alerts -c/nf -E metrics.wavefront.com active')
+    expect(o.status).to eq(1)
+    expect(o.stderr).to eq('alerts query failed. Please supply an API token.')
+    expect(o.stdout).to eq(
+      "config file '/nf' not found. Taking options from command-line.")
+  end
+
+  it 'fails with a helpful message if an invalid state is given' do
+    o = wf('alerts -n -c/nf -t token -E metrics.wavefront.com badstate')
+    expect(o.status).to eq(1)
+    expect(o.stderr).to eq('alerts query failed. State must be one of: ' \
+                'active, affected_by_maintenance, all, invalid, snoozed.')
+    expect(o.stdout).to eq("config file '/nf' not found. Taking options " \
+                           'from command-line.')
+  end
+
+  it 'performs a verbose noop with a CLI endpoint' do
+    o = wf('alerts -n -c/nf -t token -E test.wavefront.com active')
+    expect(o.status).to eq(0)
+    expect(o.stderr).to be_empty
+    expect(o.stdout_a[-1]).to eq(
+      'GET https://test.wavefront.com/api/alert/active?t=token')
+  end
+
+  it 'performs a verbose noop with default config file options' do
+    o = wf("alerts -n -c #{CF} active")
+    expect(o.status).to eq(0)
+    expect(o.stderr).to be_empty
+    expect(o.stdout_a.last).to eq(
+      "GET https://default.wavefront.com/api/alert/active?t=#{DEF_TOKEN}")
+  end
+
+  it 'performs a verbose noop with config file and CLI options' do
+    o = wf("alerts -n -c #{CF} -E cli.wavefront.com active")
+    expect(o.status).to eq(0)
+    expect(o.stderr).to be_empty
+    expect(o.stdout_a.last).to eq(
+      "GET https://cli.wavefront.com/api/alert/active?t=#{DEF_TOKEN}")
+  end
+end
+
+describe 'source mode' do
+  cmds = %W(list show describe undescribe #{'tag add'} #{'tag delete'}
+            untag).each do |cmd|
+    it "#{cmd} fails with no token if there is no token" do
+      o = wf("source #{cmd} -c/nf arg1}")
+      expect(o.status).to eq(1)
+      expect(o.stderr).to eq('source query failed. Please supply an API token.')
+      expect(o.stdout).to eq(
+        "config file '/nf' not found. Taking options from command-line.")
+    end
+  end
+
+  describe 'list subcommand' do
+    it 'performs a verbose noop with default options' do
+      o = wf('source -c /nf -n -t token list ptn')
+      expect(o.stderr).to be_empty
+      expect(o.status).to eq(0)
+      expect(o.stdout_a[-1]).to eq('HEADERS {"X-AUTH-TOKEN"=>"token"}')
+      expect(o.stdout_a[-2]).to start_with(
+        'GET https://metrics.wavefront.com/api/manage/source/')
+      expect(o.stdout_a[-2]).to have_element([:desc, false])
+      expect(o.stdout_a[-2]).to have_element([:limit, 100])
+      expect(o.stdout_a[-2]).to have_element([:pattern, 'ptn'])
+      expect(o.stdout).to match(/Taking options from command-line/)
+    end
+
+    it 'performs a verbose noop with default config file options' do
+      o = wf("source -c #{CF} -n -s lasthost -l 55 list ptn")
+      expect(o.stderr).to be_empty
+      expect(o.status).to eq(0)
+      expect(o.stdout_a[-1]).to eq(
+        "HEADERS {\"X-AUTH-TOKEN\"=>\"#{DEF_TOKEN}\"}")
+      expect(o.stdout_a[-2]).to start_with(
+        'GET https://default.wavefront.com/api/manage/source/')
+      expect(o.stdout_a[-2]).to have_element([:desc, false])
+      expect(o.stdout_a[-2]).to have_element([:limit, 55])
+      expect(o.stdout_a[-2]).to have_element([:lastEntityId, 'lasthost'])
+      expect(o.stdout_a[-2]).to have_element([:pattern, 'ptn'])
+      expect(o.stdout).to_not match(/Taking options from command-line/)
+    end
+
+    it 'performs a verbose noop with config file and CLI options' do
+      o = wf("source -c #{CF} -P other -n -s lasthost -t token list ptn")
+      expect(o.stderr).to be_empty
+      expect(o.status).to eq(0)
+      expect(o.stdout_a[-1]).to eq(
+        'HEADERS {"X-AUTH-TOKEN"=>"token"}')
+      expect(o.stdout_a[-2]).to start_with(
+        'GET https://other.wavefront.com/api/manage/source/')
+      expect(o.stdout_a[-2]).to have_element([:desc, false])
+      expect(o.stdout_a[-2]).to have_element([:limit, 100])
+      expect(o.stdout_a[-2]).to have_element([:lastEntityId, 'lasthost'])
+      expect(o.stdout_a[-2]).to have_element([:pattern, 'ptn'])
+      expect(o.stdout).to_not match(/Taking options from command-line/)
+    end
+  end
+
+  describe 'show subcommand' do
+    it 'performs a verbose noop with default options' do
+      o = wf('source -c /nf -n -t token show testhost')
+      expect(o.stderr).to be_empty
+      expect(o.status).to eq(0)
+      expect(o.stdout_a[-1]).to eq('HEADERS {"X-AUTH-TOKEN"=>"token"}')
+      expect(o.stdout_a[-2]).to eq(
+        'GET https://metrics.wavefront.com/api/manage/source/testhost')
+      expect(o.stdout).to match(/Taking options from command-line/)
+    end
+
+    it 'performs a verbose noop with CLI options' do
+      o = wf('source -c /nf -n -t token -E cli.wavefront.com show testhost')
+      expect(o.stderr).to be_empty
+      expect(o.status).to eq(0)
+      expect(o.stdout_a[-1]).to eq('HEADERS {"X-AUTH-TOKEN"=>"token"}')
+      expect(o.stdout_a[-2]).to eq(
+        'GET https://cli.wavefront.com/api/manage/source/testhost')
+      expect(o.stdout).to match(/Taking options from command-line/)
+    end
+
+    it 'performs a verbose noop with CLI and config file options' do
+      o = wf("source -c #{CF} -n show testhost")
+      expect(o.stderr).to be_empty
+      expect(o.status).to eq(0)
+      expect(o.stdout_a[-1]).to eq(
+        "HEADERS {\"X-AUTH-TOKEN\"=>\"#{DEF_TOKEN}\"}")
+      expect(o.stdout_a[-2]).to eq(
+        'GET https://default.wavefront.com/api/manage/source/testhost')
+    end
+
+    it 'performs a verbose noop with CLI and non-default config file options' do
+      o = wf("source -c #{CF} -t token -P other -n show testhost")
+      expect(o.stderr).to be_empty
+      expect(o.status).to eq(0)
+      expect(o.stdout_a[-1]).to eq('HEADERS {"X-AUTH-TOKEN"=>"token"}')
+      expect(o.stdout_a[-2]).to eq(
+        'GET https://other.wavefront.com/api/manage/source/testhost')
+    end
+  end
+
+  describe 'describe subcommand' do
+    it 'performs a verbose noop with default options' do
+      o = wf('source -c /nf -n -t token describe "test desc"')
+      expect(o.stderr).to be_empty
+      expect(o.status).to eq(0)
+      expect(o.stdout_a[-1]).to eq('HEADERS {"X-AUTH-TOKEN"=>"token"}')
+      expect(o.stdout_a[-3]).to eq(
+        'POST https://metrics.wavefront.com/api/manage/source/'\
+        "#{Socket.gethostname}/description")
+      expect(o.stdout_a[-2]).to eq('QUERY test desc')
+      expect(o.stdout).to match(/Taking options from command-line/)
+    end
+
+    it 'performs a verbose noop with CLI and config options' do
+      o = wf("source -c #{CF} -H i-123456 -n -t token describe 'test desc'")
+      expect(o.stderr).to be_empty
+      expect(o.status).to eq(0)
+      expect(o.stdout_a[-1]).to eq('HEADERS {"X-AUTH-TOKEN"=>"token"}')
+      expect(o.stdout_a[-3]).to eq(
+        'POST https://default.wavefront.com/api/manage/source/'\
+        'i-123456/description')
+      expect(o.stdout_a[-2]).to eq('QUERY test desc')
+      expect(o.stdout).to_not match(/Taking options from command-line/)
+    end
+
+    it 'performs a verbose noop with CLI and non-default config options' do
+      o = wf("source -c #{CF} -P other -E cli.wavefront.com -H i-123456 " \
+             "-n describe 'test desc'")
+      expect(o.stderr).to be_empty
+      expect(o.status).to eq(0)
+      expect(o.stdout_a[-1]).to eq(
+        "HEADERS {\"X-AUTH-TOKEN\"=>\"#{OTHER_TOKEN}\"}")
+      expect(o.stdout_a[-3]).to eq(
+        'POST https://cli.wavefront.com/api/manage/source/'\
+        'i-123456/description')
+      expect(o.stdout_a[-2]).to eq('QUERY test desc')
+      expect(o.stdout).to_not match(/Taking options from command-line/)
+    end
+  end
+
+  describe 'undescribe subcommand' do
+    it 'performs a verbose noop with default options' do
+      o = wf('source -c /nf -n -t token undescribe thost')
+      expect(o.stderr).to be_empty
+      expect(o.status).to eq(0)
+      expect(o.stdout_a[-1]).to eq('HEADERS {"X-AUTH-TOKEN"=>"token"}')
+      expect(o.stdout_a[-3]).to eq(
+        'POST https://metrics.wavefront.com/api/manage/source/'\
+        'thost/description')
+      expect(o.stdout_a[-2]).to eq('QUERY ')
+      expect(o.stdout).to match(/Taking options from command-line/)
+    end
+
+    it 'performs a verbose noop with CLI and config options' do
+      o = wf("source -c #{CF} -n -t token undescribe thost")
+      expect(o.stderr).to be_empty
+      expect(o.status).to eq(0)
+      expect(o.stdout_a[-1]).to eq('HEADERS {"X-AUTH-TOKEN"=>"token"}')
+      expect(o.stdout_a[-3]).to eq(
+        'POST https://default.wavefront.com/api/manage/source/'\
+        'thost/description')
+      expect(o.stdout_a[-2]).to eq('QUERY ')
+      expect(o.stdout).to_not match(/Taking options from command-line/)
+    end
+
+    it 'performs a verbose noop with CLI and non-default config options' do
+      o = wf("source -n -c #{CF} -P other -E cli.wavefront.com undescribe thost")
+      expect(o.stderr).to be_empty
+      expect(o.status).to eq(0)
+      expect(o.stdout_a[-1]).to eq(
+        "HEADERS {\"X-AUTH-TOKEN\"=>\"#{OTHER_TOKEN}\"}")
+      expect(o.stdout_a[-3]).to eq(
+        'POST https://cli.wavefront.com/api/manage/source/thost/description')
+      expect(o.stdout_a[-2]).to eq('QUERY ')
+      expect(o.stdout).to_not match(/Taking options from command-line/)
+    end
+  end
+
+  describe 'tag subcommand' do
+    describe 'tag add' do
+      it 'performs a verbose noop with default options' do
+        o = wf('source -c /nf -n -t token tag add tag1 tag2')
+        expect(o.stderr).to be_empty
+        expect(o.status).to eq(0)
+        expect(o.stdout_a[-1]).to eq('HEADERS {"X-AUTH-TOKEN"=>"token"}')
+        expect(o.stdout_a[-2]).to eq(
+          'POST https://metrics.wavefront.com/api/manage/source/'\
+          "#{Socket.gethostname}/tags/tag2")
+        expect(o.stdout_a[-3]).to eq('HEADERS {"X-AUTH-TOKEN"=>"token"}')
+        expect(o.stdout_a[-4]).to eq(
+          'POST https://metrics.wavefront.com/api/manage/source/'\
+          "#{Socket.gethostname}/tags/tag1")
+        expect(o.stdout).to match(/Taking options from command-line/)
+      end
+
+      it 'performs a verbose noop with conf file and CLI options' do
+        o = wf("source -c #{CF} -P other -H i-123456 -n " \
+               '-E cli.wavefront.com tag add tag1')
+        expect(o.stderr).to be_empty
+        expect(o.status).to eq(0)
+        expect(o.stdout_a[-1]).to eq(
+          "HEADERS {\"X-AUTH-TOKEN\"=>\"#{OTHER_TOKEN}\"}")
+        expect(o.stdout_a[-2]).to eq(
+          'POST https://cli.wavefront.com/api/manage/source/'\
+          "i-123456/tags/tag1")
+      end
+    end
+
+    describe 'tag delete' do
+      it 'performs a verbose noop with default options' do
+        o = wf('source -c /nf -n -t token tag delete tag1 tag2')
+        expect(o.stderr).to be_empty
+        expect(o.status).to eq(0)
+        expect(o.stdout_a[-1]).to eq('HEADERS {"X-AUTH-TOKEN"=>"token"}')
+        expect(o.stdout_a[-2]).to eq(
+          'DELETE https://metrics.wavefront.com/api/manage/source/'\
+          "#{Socket.gethostname}/tags/tag2")
+        expect(o.stdout_a[-3]).to eq('HEADERS {"X-AUTH-TOKEN"=>"token"}')
+        expect(o.stdout_a[-4]).to eq(
+          'DELETE https://metrics.wavefront.com/api/manage/source/'\
+          "#{Socket.gethostname}/tags/tag1")
+        expect(o.stdout).to match(/Taking options from command-line/)
+      end
+
+      it 'performs a verbose noop with conf file and CLI options' do
+        o = wf("source -c #{CF} -P other -H i-123456 -n " \
+               '-E cli.wavefront.com tag delete tag1')
+        expect(o.stderr).to be_empty
+        expect(o.status).to eq(0)
+        expect(o.stdout_a[-1]).to eq(
+          "HEADERS {\"X-AUTH-TOKEN\"=>\"#{OTHER_TOKEN}\"}")
+        expect(o.stdout_a[-2]).to eq(
+          'DELETE https://cli.wavefront.com/api/manage/source/'\
+          "i-123456/tags/tag1")
+      end
+    end
+  end
+
+  describe 'untag subcommand' do
+    it 'performs a verbose noop with default options' do
+      o = wf('source -c /nf -n -t token untag thost')
+      expect(o.stderr).to be_empty
+      expect(o.status).to eq(0)
+      expect(o.stdout_a[-1]).to eq('HEADERS {"X-AUTH-TOKEN"=>"token"}')
+      expect(o.stdout_a[-2]).to eq(
+        'DELETE https://metrics.wavefront.com/api/manage/source/thost/tags')
+      expect(o.stdout).to match(/Taking options from command-line/)
+    end
+
+    it 'performs a verbose noop with CLI and config options' do
+      o = wf("source -c #{CF} -n -E cli.wavefront.com untag thost1 thost2")
+      expect(o.stderr).to be_empty
+      expect(o.status).to eq(0)
+      expect(o.stdout_a[-3]).to eq(
+        "HEADERS {\"X-AUTH-TOKEN\"=>\"#{DEF_TOKEN}\"}")
+      expect(o.stdout_a[-4]).to eq(
+        'DELETE https://cli.wavefront.com/api/manage/source/thost1/tags')
+      expect(o.stdout_a[-1]).to eq(
+        "HEADERS {\"X-AUTH-TOKEN\"=>\"#{DEF_TOKEN}\"}")
+      expect(o.stdout_a[-2]).to eq(
+        'DELETE https://cli.wavefront.com/api/manage/source/thost2/tags')
+      expect(o.stdout).to_not match(/Taking options from command-line/)
+    end
+  end
+end
+
+describe 'event mode' do
+  cmds = %w(create close delete).each do |cmd|
+    it "#{cmd} fails with no token if there is no token" do
+      o = wf("event #{cmd} -c/nf #{cmd == 'delete' ? 'arg1 arg2' : 'arg1'}")
+      expect(o.status).to eq(1)
+      expect(o.stderr).to eq('Please supply an API token.')
+      expect(o.stdout).to eq(
+        "config file '/nf' not found. Taking options from command-line.")
+    end
+  end
+
+  describe 'create subcommand' do
+    it 'performs a verbose noop with default options' do
+      o = wf('event create -c/nf -t token -n test_ev')
+      expect(o.status).to eq(0)
+      expect(o.stderr).to be_empty
+      expect(o.stdout_a[-3]).to eq(
+        'PUT https://metrics.wavefront.com/api/events/')
+      expect(o.stdout_a[-2]).to have_element([:h, Socket.gethostname])
+      expect(o.stdout_a[-1]).to eq('HEADERS {:"X-AUTH-TOKEN"=>"token"}')
+    end
+
+    it 'performs a verbose noop on a bounded event with CLI options' do
+      o = wf("event create -c/nf --start #{TIME[:start][:eng]} " \
+             "--end #{TIME[:end][:eng]} -t token -d 'some description' " \
+             '-l info -E test.wavefront.com -n test_ev')
+      expect(o.status).to eq(0)
+      expect(o.stderr).to be_empty
+      expect(o.stdout_a[-3]).to eq('PUT https://test.wavefront.com/api/events/')
+      expect(o.stdout_a[-2]).to have_element([:h, Socket.gethostname])
+      expect(o.stdout_a[-2]).to have_element([:s, TIME[:start][:ms]])
+      expect(o.stdout_a[-2]).to have_element([:e, TIME[:end][:ms]])
+      expect(o.stdout_a[-2]).to have_element([:l, 'info'])
+      expect(o.stdout_a[-2]).to have_element([:c, false])
+      expect(o.stdout_a[-2]).to have_element([:d, 'some description'])
+      expect(o.stdout_a[-1]).to eq('HEADERS {:"X-AUTH-TOKEN"=>"token"}')
+    end
+
+    it 'performs a verbose noop unbounded event with CLI and file options' do
+      o = wf("event create -c #{CF} -P other -s #{TIME[:start][:eng]} " \
+             "-t token -H i-123456,i-abcdef -d 'some description' -n test_ev")
+      expect(o.stderr).to be_empty
+      expect(o.status).to eq(0)
+      expect(o.stdout_a[-3]).to eq(
+        'PUT https://other.wavefront.com/api/events/')
+      expect(o.stdout_a[-2]).to have_element([:h, 'i-123456'])
+      expect(o.stdout_a[-2]).to have_element([:h, 'i-abcdef'])
+      expect(o.stdout_a[-2]).to have_element([:s, TIME[:start][:ms]])
+      expect(o.stdout_a[-2]).not_to match(/&e=/)
+      expect(o.stdout_a[-2]).to have_element([:c, false])
+      expect(o.stdout_a[-2]).to have_element([:d, 'some description'])
+    end
+
+    it 'performs a verbose noop on an instantaneous event with CLI and ' \
+       'file options' do
+      o = wf("event create -c #{CF} -i -H i-123456 -d 'some description' " \
+             '-l smoke -E test.wavefront.com -n test_ev')
+      expect(o.stderr).to be_empty
+      expect(o.status).to eq(0)
+      expect(o.stdout_a[-3]).to eq('PUT https://test.wavefront.com/api/events/')
+      expect(o.stdout_a[-2]).to have_element([:h, 'i-123456'])
+      expect(o.stdout_a[-2]).not_to match(/&s=/)
+      expect(o.stdout_a[-2]).not_to match(/&e=/)
+      expect(o.stdout_a[-2]).to have_element([:c, 'true'])
+      expect(o.stdout_a[-2]).to have_element([:l, 'smoke'])
+      expect(o.stdout_a[-2]).to have_element([:d, 'some description'])
+      expect(o.stdout_a[-1]).to eq(
+        "HEADERS {:\"X-AUTH-TOKEN\"=>\"#{DEF_TOKEN}\"}")
+    end
+  end
+
+  describe 'close subcommand' do
+    it 'refuses to close an unknown event' do
+      o = wf("event close -c #{CF} -t token -n unknown_event")
+      expect(o.stderr).to eq(
+        "event query failed. No event 'unknown_event' to close.")
+      expect(o.status).to eq(1)
+      expect(o.stdout).to be_empty
+    end
+
+    it 'performs a verbose noop with default options' do
+      o = wf("event close -c/nf -t token -n test_ev #{TIME[:start][:ms]}")
+      expect(o.stderr).to be_empty
+      expect(o.status).to eq(0)
+      expect(o.stdout_a[-3]).to eq(
+        'PUT https://metrics.wavefront.com/api/events/close')
+      expect(o.stdout_a[-2]).to have_element([:s, TIME[:start][:ms]])
+      expect(o.stdout_a[-2]).to have_element([:n, 'test_ev'])
+      expect(o.stdout_a[-1]).to eq('HEADERS {:"X-AUTH-TOKEN"=>"token"}')
+    end
+
+    it 'performs a verbose noop with CLI and file options' do
+      o = wf("event close -P other -c #{CF} -t token -n test_ev " \
+             "#{TIME[:start][:ms]}")
+      expect(o.stderr).to be_empty
+      expect(o.status).to eq(0)
+      expect(o.stdout_a[-3]).to eq(
+        'PUT https://other.wavefront.com/api/events/close')
+      expect(o.stdout_a[-2]).to have_element([:s, TIME[:start][:ms]])
+      expect(o.stdout_a[-2]).to have_element([:n, 'test_ev'])
+      expect(o.stdout_a[-1]).to eq('HEADERS {:"X-AUTH-TOKEN"=>"token"}')
+    end
+
+    it 'performs a verbose noop with CLI options' do
+      o = wf("event close -c/nf -E cli.wavefront.com -t token -n test_ev " +
+             TIME[:start][:ms].to_s)
+      expect(o.stderr).to be_empty
+      expect(o.status).to eq(0)
+      expect(o.stdout_a[-3]).to eq(
+        'PUT https://cli.wavefront.com/api/events/close')
+      expect(o.stdout_a[-2]).to have_element([:s, TIME[:start][:ms]])
+      expect(o.stdout_a[-2]).to have_element([:n, 'test_ev'])
+      expect(o.stdout_a[-1]).to eq('HEADERS {:"X-AUTH-TOKEN"=>"token"}')
+    end
+  end
+
+  describe 'delete subcommand' do
+    it 'performs a verbose noop with default options' do
+      o = wf("event delete -c/nf -t token -n #{TIME[:start][:ms]} test_ev")
+      expect(o.stderr).to be_empty
+      expect(o.status).to eq(0)
+      expect(o.stdout_a[-2]).to eq('DELETE https://metrics.wavefront.com/' \
+                                   "api/events/#{TIME[:start][:ms]}/test_ev")
+      expect(o.stdout_a[-1]).to eq('HEADERS {:"X-AUTH-TOKEN"=>"token"}')
+    end
+  end
+end
+
+describe 'ts mode' do
+  it 'fails with no granularity and default options' do
+    o = wf('ts -t token -c/nf "ts(dev.cli.test)"')
+    expect(o.status).to eq(1)
+    expect(o.stdout).to match(
+      /^config file.*not found. Taking options from command-line./)
+    expect(o.stderr).to start_with(
+      'ts query failed. You must specify a granularity')
+  end
+
+  it 'fails with no token if there is no token' do
+    o = wf('ts -c/nf -E metrics.wavefront.com "ts(dev.cli.test)"')
+    expect(o.status).to eq(1)
+    expect(o.stderr).to eq('ts query failed. Please supply an API token.')
+    expect(o.stdout).to eq(
+      "config file '/nf' not found. Taking options from command-line.")
+  end
+
+  it 'performs a verbose noop with default options' do
+    o = wf('ts -c/nf -t token -Sn "ts(dev.cli.test)"')
+    expect(o.status).to eq(0)
+    expect(o.stderr).to be_empty
+    expect(o.stdout_a[-2]).to eq('GET https://metrics.wavefront.com/chart/api')
+    q = raw(o.stdout_a.last)
+    expect(q['X-AUTH-TOKEN']).to eq('token')
+    expect(q[:params][:g]).to eq('s')
+    expect(q[:params][:q]).to eq('ts(dev.cli.test)')
+  end
+
+  it 'performs a verbose noop with a CLI endpoint' do
+    o = wf('ts -c/nf -t token -Sn -E test.wavefront.com "ts(dev.cli.test)"')
+    expect(o.status).to eq(0)
+    expect(o.stderr).to be_empty
+    expect(o.stdout_a[-2]).to eq('GET https://test.wavefront.com/chart/api')
+    q = raw(o.stdout_a.last)
+    expect(q['X-AUTH-TOKEN']).to eq('token')
+    expect(q[:params][:g]).to eq('s')
+    expect(q[:params][:q]).to eq('ts(dev.cli.test)')
+  end
+
+  it 'performs a verbose noop with default config file options' do
+    o = wf("ts -c #{CF} -Hn 'ts(dev.cli.test)'")
+    expect(o.status).to eq(0)
+    expect(o.stderr).to be_empty
+    expect(o.stdout_a[-2]).to eq('GET https://default.wavefront.com/chart/api')
+    q = raw(o.stdout_a.last)
+    expect(q['X-AUTH-TOKEN']).to eq(DEF_TOKEN)
+    expect(q[:params][:g]).to eq('h')
+    expect(q[:params][:q]).to eq('ts(dev.cli.test)')
+  end
+
+  it 'performs a verbose noop with config file and CLI options' do
+    o = wf("ts -c #{CF} -P other -mn -E cli.wavefront.com 'ts(dev.cli.test)'")
+    expect(o.status).to eq(0)
+    expect(o.stderr).to be_empty
+    expect(o.stdout_a[-2]).to eq('GET https://cli.wavefront.com/chart/api')
+    q = raw(o.stdout_a.last)
+    expect(q['X-AUTH-TOKEN']).to eq(OTHER_TOKEN)
+    expect(q[:params][:g]).to eq('m')
+    expect(q[:params][:q]).to eq('ts(dev.cli.test)')
+  end
+end