queri 1.0.0 → 1.0.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: efd6822a01d26d086332c69907548f6d66a2a40b
-  data.tar.gz: e79b2a589fb598865acaa625a0c606dce897eea0
+  metadata.gz: c46d2d6c7692bf85dc17b981507c647140edd18c
+  data.tar.gz: 7b3068a94ad86a6b991d49c674a9116f22c6be2a
 SHA512:
-  metadata.gz: a9e70aee15638273b0934afe74418c607549e00084268a2992fc3eee81f32d13822d529d932b1634498d4bb1c467257361cf023b6ff94bd4b111b98d748a0446
-  data.tar.gz: 4e61d974581eee5bb987ab44a455da17d057234937853eaedacc88d83b2ec12924ee65cc7920e723e20d4f5e3a3e725ec2f8adb5a837367cb255102d3ec8bf86
+  metadata.gz: 9f2e65feb04706ea516319c9fa89a76ce639af989645a8e56b2f67eb2e2058c5ec404e0574f149dc6813a09ebf4a093d4d2cce2165e937492f127c3288e84a09
+  data.tar.gz: 409f9605ed2f3dbf5875f0aefe0bcc826ccb2ab8478f4a1ab6634cbd38313acabb8ac6b2f21593dfd0d712ae3e2e9f90ff9ac5a3df6f2b756967518a20946e41

data/lib/queri.rb

@@ -3,114 +3,67 @@ require "yaml"
 require "xmlrpc/client"
 require 'active_support/ordered_hash'
 
-require File.join( File.dirname(__FILE__), "queri", "realtime_arg_helpers" )
-Dir[ File.join( File.dirname(__FILE__), "**", "*.rb" ) ].each {|f| require f}
-
+lib = File.expand_path(File.dirname(__FILE__))
+$LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
+
+# Load helpers
+require "queri/helpers/time_helpers"
+require "queri/realtime_arg_helpers"
+require "queri/arg_helpers"
+require "queri/aggregate_report_helpers"
+require "queri/composite_agent_level_report_helpers"
+require "queri/composite_by_hour_report_helpers"
+require "queri/composite_report_helpers"
+
+# Load client/request libraries
+require "queri/xml_client"
+require "queri/request"
+
+# Load report methods
+require "queri/stats"
+require "queri/realtime"
+
+# Load Stats report classes
+require "queri/stats/agents_and_sessions"
+Dir[ File.join( File.dirname(__FILE__), "queri/stats/agents_and_sessions", "*.rb" ) ].each {|f| require f }
+require "queri/stats/answered_calls"
+Dir[ File.join( File.dirname(__FILE__), "queri/stats/answered_calls", "*.rb" ) ].each {|f| require f }
+require "queri/stats/call_distribution_by_day"
+Dir[ File.join( File.dirname(__FILE__), "queri/stats/call_distribution_by_day", "*.rb" ) ].each {|f| require f }
+require "queri/stats/call_distribution_by_day_of_week"
+Dir[ File.join( File.dirname(__FILE__), "queri/stats/call_distribution_by_day_of_week", "*.rb" ) ].each {|f| require f }
+require "queri/stats/call_distribution_by_hour"
+Dir[ File.join( File.dirname(__FILE__), "queri/stats/call_distribution_by_hour", "*.rb" ) ].each {|f| require f }
+require "queri/stats/call_outcomes"
+Dir[ File.join( File.dirname(__FILE__), "queri/stats/call_outcomes", "*.rb" ) ].each {|f| require f }
+require "queri/stats/details_of_agent_sessions_and_pauses"
+Dir[ File.join( File.dirname(__FILE__), "queri/stats/details_of_agent_sessions_and_pauses", "*.rb" ) ].each {|f| require f }
+require "queri/stats/details_of_answered_calls"
+Dir[ File.join( File.dirname(__FILE__), "queri/stats/details_of_answered_calls", "*.rb" ) ].each {|f| require f }
+require "queri/stats/details_of_unanswered_calls"
+Dir[ File.join( File.dirname(__FILE__), "queri/stats/details_of_unanswered_calls", "*.rb" ) ].each {|f| require f }
+require "queri/stats/distributions"
+Dir[ File.join( File.dirname(__FILE__), "queri/stats/distributions", "*.rb" ) ].each {|f| require f }
+require "queri/stats/unanswered_calls"
+Dir[ File.join( File.dirname(__FILE__), "queri/stats/unanswered_calls", "*.rb" ) ].each {|f| require f }
+
+# Load Realtime report classes
+require "queri/realtime/agents"
+require "queri/realtime/calls"
+require "queri/realtime/queues"
 
 module Queri
-  LOGFILE = ""
-  PERIOD = ""
-  AGENT_FILTER = ""
-
-  @config = {
-    :host => "127.0.0.1",
-    :path => "/path/to/queuemetrics.do",
-    :port => "8080",
-    :username => "username",
-    :password => "some_secret"
-  }
-
-  @valid_config_keys = @config.keys
-
-  @@server = nil
-
-  private_constant :LOGFILE, :PERIOD, :AGENT_FILTER
-
-  def self.config
-    @config
-  end
-
-  def self.configure opts={}
-    raise ArgumentError, "expected hash, got something else" if opts.class != Hash
-    unless opts.empty?
-      if (@valid_config_keys - opts.keys).any?
-        puts "WARNING:  erroneous keys given to ::config. Acceptible keys include #{@valid_config_keys}"
-      end
-      if (opts.keys & @valid_config_keys).empty?
-        raise ArgumentError, "erroneous keys given to ::config. Acceptible keys include #{@valid_config_keys}"
-      else
-        opts.each {|k,v| @config[k.to_sym] = v if @valid_config_keys.include?(k.to_sym)}
-        @@server = XMLRPC::Client.new(host = @config[:host], path = @config[:path], port = @config[:port])
-      end
+  class << self
+    def config
+      Queri::XmlClient.config
     end
-  end
 
-  def self.configure_with path_to_yaml_file
-    raise ArgumentError, "expected path to yaml file as string, got something else" if path_to_yaml_file.class != String
-    begin
-      config = YAML.load(IO.read(path_to_yaml_file))
-    rescue Errno::ENOENT
-      raise LoadError, "the specified configuration file could not be found"
-    rescue Psych::SyntaxError
-      raise LoadError, "the specified configuration file contains invalid syntax"
+    def configure opts={}
+      Queri::XmlClient.configure opts
     end
-    configure(config)
-    @@server = XMLRPC::Client.new(host = @config[:host], path = @config[:path], port = @config[:port])
-  end
 
-  def self.send_request *args
-    raise LoadError, "client configs not set. Assign by passing a hash to Queri.configure or a yaml file path to Queri.configure_with" if @@server.nil?
-    request = Request.new(*args)
-    response = @@server.call(*request.parameters)
-    return response[request.report.class.xml_code]
-  end
-
-  class Request
-    attr_reader :report
-
-    def initialize *args
-      raise ArgumentError, "expected arguments: queues(Array), report(Stats or Realtime obj), period_start(Time or Nil), period_end(Time or Nil)" unless valid_args?(args)
-      @queues, @report, @period_start, @period_end = validate_args(args)
-    end
-
-    def parameters
-      if @period_start.nil? && @period_end.nil?
-        [@report.class.query_method, @queues.join("|"), Queri.config[:username], Queri.config[:password], LOGFILE, AGENT_FILTER, [@report.class.xml_code]]
-      else
-        [@report.class.query_method, @queues.join("|"), Queri.config[:username], Queri.config[:password], LOGFILE, PERIOD, @period_start, @period_end, AGENT_FILTER, [@report.class.xml_code]]
-      end
+    def configure_with path_to_yaml_file
+      Queri::XmlClient.configure_with path_to_yaml_file
     end
-
-    private
-
-      def valid_args? args
-        valid = true
-        valid = false if args[0].class != Array
-        valid = false if args[1].class == NilClass
-        valid = false if (args[2].class != NilClass && args[2].class != Time)
-        valid = false if (args[3].class != NilClass && args[3].class != Time)
-        return valid
-      end
-
-      def validate_args args
-        raise ArgumentError, "queues array cannot be empty" if args[0].empty?
-        if args[1].class.name.include?("Queri::Stats::")
-          if (args[2].nil? || args[3].nil?)
-            raise ArgumentError, "Queri::Stats reports require non-nil period start and end times"
-          end
-        elsif args[1].class.name.include?("Queri::Realtime::")
-          unless (args[2].nil? && args[3].nil?)
-            raise ArgumentError, "Queri::Realtime reports require nil period start and end times"
-          end
-        end
-        args[2] = format_time(args[2])
-        args[3] = format_time(args[3])
-        return args
-      end
-
-      def format_time time
-        return time unless time.respond_to?(:strftime)
-        time.strftime("%Y-%m-%d.%H:%M:%S")
-      end
   end
 end

data/lib/queri/aggregate_report_helpers.rb

@@ -2,7 +2,7 @@ module AggregateReportHelpers
   private
 
     def parse_response
-      r = Queri.send_request(@queues, self, @period_start, @period_end)
+      r = Queri::XmlClient.send_request(@queues, self, @period_start, @period_end)
       pairs = self.class.key_translations.each_pair
       r.shift
       new_metrics = r.map do |metric|

data/lib/queri/composite_agent_level_report_helpers.rb

@@ -2,7 +2,7 @@ module CompositeAgentLevelReportHelpers
   private
 
     def parse_response
-      r = Queri.send_request(@queues, self, @period_start, @period_end)
+      r = Queri::XmlClient.send_request(@queues, self, @period_start, @period_end)
       new_keys = self.class.keys
       r.shift
       new_metrics = Hash.new {|h,k| h[k] = []}

data/lib/queri/composite_by_hour_report_helpers.rb

@@ -2,7 +2,7 @@ module CompositeByHourReportHelpers
   private
 
     def parse_response
-      r = Queri.send_request(@queues, self, @period_start, @period_end)
+      r = Queri::XmlClient.send_request(@queues, self, @period_start, @period_end)
       k_t = self.class.key_translations
       column_headers = r.shift
       new_headers = []

data/lib/queri/composite_report_helpers.rb

@@ -2,7 +2,7 @@ module CompositeReportHelpers
   private
 
     def parse_response
-      r = Queri.send_request(@queues, self, @period_start, @period_end)
+      r = Queri::XmlClient.send_request(@queues, self, @period_start, @period_end)
       new_keys = self.class.keys
       r.shift
       r.map do |agent|

data/lib/queri/helpers/time_helpers.rb

@@ -0,0 +1,10 @@
+module Queri
+  module Helpers
+    module TimeHelpers
+      def self.time_string_to_seconds time_string
+        base_60_digits = time_string[ (time_string.index(/\d/))..-1 ].split(?:).reverse.map(&:to_i)
+        base_60_digits.map.with_index {|d,i| d * (60 ** i)}.reduce(:+)
+      end
+    end
+  end
+end

data/lib/queri/request.rb

@@ -0,0 +1,52 @@
+module Queri
+  module XmlClient
+    class Request
+      attr_reader :report
+
+      def initialize args
+        raise ArgumentError, "expected arguments: queues(Array), report(Stats or Realtime obj), period_start(Time or Nil), period_end(Time or Nil)" unless valid_args?(args)
+        @queues, @report, @period_start, @period_end = validate_args(args)
+      end
+
+      def parameters
+        if @period_start.nil? && @period_end.nil?
+          [@report.class.query_method, @queues.join("|"), Queri::XmlClient.config[:username], Queri::XmlClient.config[:password], LOGFILE, AGENT_FILTER, [@report.class.xml_code]]
+        else
+          [@report.class.query_method, @queues.join("|"), Queri::XmlClient.config[:username], Queri::XmlClient.config[:password], LOGFILE, PERIOD, @period_start, @period_end, AGENT_FILTER, [@report.class.xml_code]]
+        end
+      end
+
+      private
+
+        def valid_args? args
+          valid = true
+          valid = false if args[0].class != Array
+          valid = false if args[1].class == NilClass
+          valid = false if (args[2].class != NilClass && args[2].class != Time)
+          valid = false if (args[3].class != NilClass && args[3].class != Time)
+          return valid
+        end
+
+        def validate_args args
+          raise ArgumentError, "queues array cannot be empty" if args[0].empty?
+          if args[1].class.name.include?("Queri::Stats::")
+            if (args[2].nil? || args[3].nil?)
+              raise ArgumentError, "Queri::Stats reports require non-nil period start and end times"
+            end
+          elsif args[1].class.name.include?("Queri::Realtime::")
+            unless (args[2].nil? && args[3].nil?)
+              raise ArgumentError, "Queri::Realtime reports require nil period start and end times"
+            end
+          end
+          args[2] = format_time(args[2])
+          args[3] = format_time(args[3])
+          return args
+        end
+
+        def format_time time
+          return time unless time.respond_to?(:strftime)
+          time.strftime("%Y-%m-%d.%H:%M:%S")
+        end
+    end
+  end
+end

data/lib/queri/version.rb

@@ -1,3 +1,3 @@
 module Queri
-  VERSION = "1.0.0"
+  VERSION = "1.0.1"
 end

data/lib/queri/xml_client.rb

@@ -0,0 +1,60 @@
+module Queri
+  module XmlClient
+    LOGFILE = ""
+    PERIOD = ""
+    AGENT_FILTER = ""
+
+    @config = {
+      :host => "127.0.0.1",
+      :path => "/path/to/queuemetrics.do",
+      :port => "8080",
+      :username => "username",
+      :password => "some_secret"
+    }
+
+    @valid_config_keys = @config.keys
+
+    @@server = nil
+
+    private_constant :LOGFILE, :PERIOD, :AGENT_FILTER
+
+    def self.config
+      @config
+    end
+
+    def self.configure opts={}
+      raise ArgumentError, "expected hash, got something else" if opts.class != Hash
+      unless opts.empty?
+        if (@valid_config_keys - opts.keys).any?
+          puts "WARNING:  erroneous keys given to ::config. Acceptible keys include #{@valid_config_keys}"
+        end
+        if (opts.keys & @valid_config_keys).empty?
+          raise ArgumentError, "erroneous keys given to ::config. Acceptible keys include #{@valid_config_keys}"
+        else
+          opts.each {|k,v| @config[k.to_sym] = v if @valid_config_keys.include?(k.to_sym)}
+          @@server = XMLRPC::Client.new(host = @config[:host], path = @config[:path], port = @config[:port])
+        end
+      end
+    end
+
+    def self.configure_with path_to_yaml_file
+      raise ArgumentError, "expected path to yaml file as string, got something else" if path_to_yaml_file.class != String
+      begin
+        config = YAML.load(IO.read(path_to_yaml_file))
+      rescue Errno::ENOENT
+        raise LoadError, "the specified configuration file could not be found"
+      rescue Psych::SyntaxError
+        raise LoadError, "the specified configuration file contains invalid syntax"
+      end
+      configure(config)
+      @@server = XMLRPC::Client.new(host = @config[:host], path = @config[:path], port = @config[:port])
+    end
+
+    def self.send_request *args
+      raise LoadError, "client configs not set. Assign by passing a hash to Queri.configure or a yaml file path to Queri.configure_with" if @@server.nil?
+      request = Request.new(args)
+      response = @@server.call(*request.parameters)
+      return response[request.report.class.xml_code]
+    end
+  end
+end

data/spec/helpers/time_helpers_spec.rb

@@ -0,0 +1,114 @@
+require 'spec_helper'
+
+module Queri
+  module Helpers
+    describe TimeHelpers do
+      describe "::time_string_to_seconds" do
+        let(:time) { "01:00  " }
+
+        subject { TimeHelpers.time_string_to_seconds(time) }
+
+        it "should respond" do
+          expect{ subject }.to_not raise_error
+        end
+
+        it "should return a Fixnum" do
+          subject.should be_a Fixnum
+        end
+
+        it "should properly parse a base case time" do
+          TimeHelpers.time_string_to_seconds("00:00  ").should eq 0
+        end
+
+        context "given leading non-numeric characters" do
+          let(:time) { "a time01:00" }
+
+          it "should accurately parse the string" do
+            subject.should eq 60
+          end
+        end
+
+        context "given seconds in the time string" do
+          let(:seconds_count) { 23 }
+          let(:time) { "00:#{seconds_count}  " }
+
+          it "should return that value" do
+            subject.should eq seconds_count
+          end
+
+          context "greater than the number of seconds in a minute" do
+            let(:seconds_count) { 61 }
+            let(:time) { "00:#{seconds_count}  " }
+
+            it "should return that value" do
+              subject.should eq seconds_count
+            end
+          end
+        end
+
+        context "given minutes in the time string" do
+          let(:minutes_count) { 5 }
+          let(:time) { "#{minutes_count}:00  " }
+
+          it "should return that value cast to seconds" do
+            subject.should eq minutes_count * 60
+          end
+
+          context "and seconds" do
+            let(:seconds_count) { 21 }
+            let(:time) { "#{minutes_count}:#{seconds_count}  " }
+
+            it "should convert that value appropriately" do
+              subject.should eq (minutes_count * 60) + seconds_count
+            end
+          end
+
+          context "greater than the number of minutes in an hour" do
+            let(:minutes_count) { 61 }
+            let(:time) { "#{minutes_count}:00  " }
+
+            it "should return that value" do
+              subject.should eq minutes_count * 60
+            end
+          end
+        end
+
+        context "given hours in the time string" do
+          let(:hours_count) { 2 }
+          let(:time) { "#{hours_count}:00:00  " }
+
+          it "should return that value cast to seconds" do
+            subject.should eq hours_count * 3600
+          end
+
+          context "and minutes" do
+            let(:minutes_count) { 35 }
+            let(:time) { "#{hours_count}:#{minutes_count}:00  " }
+
+            it "should convert that value appropriately" do
+              subject.should eq (hours_count * 3600) + (minutes_count * 60)
+            end
+          end
+
+          context "and seconds" do
+            let(:seconds_count) { 17 }
+            let(:time) { "#{hours_count}:00:#{seconds_count}  " }
+
+            it "should convert that value appropriately" do
+              subject.should eq (hours_count * 3600) + seconds_count
+            end
+          end
+
+          context "greater than the number of hours in a day" do
+            let(:hours_count) { 25 }
+            let(:time) { "#{hours_count}:00:00  " }
+
+            it "should return that value" do
+              subject.should eq hours_count * 3600
+            end
+          end
+        end
+      end
+    end
+  end
+end

data/spec/queri_spec.rb

@@ -1,32 +1,6 @@
 require 'spec_helper'
 
 describe Queri do
-  let(:queues) { ["10000"] }
-  let(:report) { double(Queri::Realtime::Queues) }
-
-  describe "constants" do
-    describe "for log file" do
-      let(:const_symbol) { :LOGFILE }
-      let(:constant) { Queri::LOGFILE }
-
-      it_behaves_like "a private constant"
-    end
-
-    describe "for period" do
-      let(:const_symbol) { :PERIOD }
-      let(:constant) { Queri::PERIOD }
-
-      it_behaves_like "a private constant"
-    end
-
-    describe "for agent filter" do
-      let(:const_symbol) { :AGENT_FILTER }
-      let(:constant) { Queri::AGENT_FILTER }
-
-      it_behaves_like "a private constant"
-    end
-  end
-
   describe "::config" do
     subject { Queri.config }
 
@@ -75,7 +49,7 @@ describe Queri do
       let(:configs) { Hash[:password, "new_secret", :foo, "bar"] }
 
       it "should warn the user against bad data" do
-        Queri.should_receive(:puts).with("WARNING:  erroneous keys given to ::config. Acceptible keys include #{Queri.instance_variable_get(:@valid_config_keys)}")
+        Queri::XmlClient.should_receive(:puts).with("WARNING:  erroneous keys given to ::config. Acceptible keys include #{Queri::XmlClient.instance_variable_get(:@valid_config_keys)}")
         subject
       end
 
@@ -178,91 +152,4 @@ describe Queri do
       end
     end
   end
-
-  describe "::send_request" do
-    before do
-      @client = double(XMLRPC::Client)
-      @client.stub(:call).and_return( {"xml_code" => [:keys, :for, :report]} )
-    end
-
-    subject { Queri.send_request(queues, report) }
-
-    context "without configurations" do
-      before do
-        @saved_configs = Queri.config.dup
-        Queri.class_variable_set(:@@server, nil)
-      end
-
-      after do
-        Queri.configure(@saved_configs)
-      end
-
-      it "should raise LoadError" do
-        expect{ subject }.to raise_error(LoadError)
-      end
-    end
-
-    context "with configurations" do
-
-      before do
-        Queri.configure_with(File.join(File.dirname(__FILE__), 'config.yml'))
-        report.stub(:class).and_return( Queri::Realtime::Queues )
-      end
-
-      it "should respond" do
-        expect{ subject }.to_not raise_error
-      end
-
-      it "should call the XMLRPC client" do
-        xml_client = Queri.class_variable_get(:@@server)
-        Queri.class_variable_set(:@@server, @client)
-        subject
-        @client.should have_received(:call)
-        Queri.class_variable_set(:@@server, xml_client)
-      end
-
-      context "given no arguments" do
-        subject { Queri.send_request }
-
-        it "should raise ArgumentError" do
-          expect{ subject }.to raise_error(ArgumentError)
-        end
-      end
-
-      context "given an empty queues array" do
-        subject { Queri.send_request([], report) }
-
-        it "should raise ArgumentError" do
-          expect{ subject }.to raise_error(ArgumentError)
-        end
-      end
-
-      context "given a Stats report object" do
-        context "and period_start or period_end are nil" do
-          let(:report) { double(Queri::Stats::AnsweredCalls::AllCalls) }
-          let(:period_start) { Time.now - 10 }
-          let(:period_end) { nil }
-
-          subject { Queri.send_request(queues, report, period_start, period_end) }
-
-          it "should raise ArgumentError" do
-            expect{ subject }.to raise_error(ArgumentError)
-          end
-        end
-      end
-
-      context "given a Realtime report object" do
-        context "and period_start or period_end are not nil" do
-          let(:period_start) { Time.now - 10 }
-          let(:period_end) { nil }
-
-          subject { Queri.send_request(queues, report, period_start, period_end) }
-
-          it "should raise ArgumentError" do
-            expect{ subject }.to raise_error(ArgumentError)
-          end
-        end
-      end
-    end
-  end
 end

data/spec/stats/answered_calls/transfers_spec.rb

@@ -6,7 +6,7 @@ describe Queri::Stats::AnsweredCalls::Transfers do
 
   before do
     Queri.configure_with(File.join(File.dirname(__FILE__), '..', '..', 'config.yml'))
-    Queri.stub(:send_request).and_return(
+    Queri::XmlClient.stub(:send_request).and_return(
       [["Transfer to", "N. Calls", " ", "..."],
         ["211", "121", "10.0%", ""],
         ["311", "122", "10.0%", ""]]

data/spec/stats/unanswered_calls/unanswered_outbound_calls_by_agent_spec.rb

@@ -1,6 +1,6 @@
 require 'spec_helper'
 
-describe Queri::Stats::UnansweredCalls::UnansweredOutboundCallsByAgent do
+describe Queri::Stats::UnansweredCalls::UnansweredOutboundCallsByAgent, :focus do
   let(:report_class) { Queri::Stats::UnansweredCalls::UnansweredOutboundCallsByAgent }
   let(:xml_code) { report_class.xml_code }
 

data/spec/support/shared/shared_examples_for_private_constant.rb

@@ -1,6 +1,6 @@
 shared_examples "a private constant" do
   it "should be defined" do
-    Queri.const_defined?(const_symbol).should be_true
+    module_name.const_defined?(const_symbol).should be_true
   end
 
   it "should not be publically accessible" do

data/spec/xml_client_spec.rb

@@ -0,0 +1,117 @@
+require 'spec_helper'
+
+describe Queri::XmlClient do
+  let(:module_name) { Queri::XmlClient }
+  let(:queues) { ["10000"] }
+  let(:report) { double(Queri::Realtime::Queues) }
+
+  describe "constants" do
+    describe "for log file" do
+      let(:const_symbol) { :LOGFILE }
+      let(:constant) { Queri::XmlClient::LOGFILE }
+
+      it_behaves_like "a private constant"
+    end
+
+    describe "for period" do
+      let(:const_symbol) { :PERIOD }
+      let(:constant) { Queri::XmlClient::PERIOD }
+
+      it_behaves_like "a private constant"
+    end
+
+    describe "for agent filter" do
+      let(:const_symbol) { :AGENT_FILTER }
+      let(:constant) { Queri::XmlClient::AGENT_FILTER }
+
+      it_behaves_like "a private constant"
+    end
+  end
+
+  describe "::send_request" do
+    before do
+      @client = double(XMLRPC::Client)
+      @client.stub(:call).and_return( {"xml_code" => [:keys, :for, :report]} )
+    end
+
+    subject { Queri::XmlClient.send_request(queues, report) }
+
+    context "without configurations" do
+      before do
+        @saved_configs = Queri::XmlClient.config.dup
+        Queri::XmlClient.class_variable_set(:@@server, nil)
+      end
+
+      after do
+        Queri::XmlClient.configure(@saved_configs)
+      end
+
+      it "should raise LoadError" do
+        expect{ subject }.to raise_error(LoadError)
+      end
+    end
+
+    context "with configurations" do
+
+      before do
+        Queri::XmlClient.configure_with(File.join(File.dirname(__FILE__), 'config.yml'))
+        report.stub(:class).and_return( Queri::Realtime::Queues )
+      end
+
+      it "should respond" do
+        expect{ subject }.to_not raise_error
+      end
+
+      it "should call the XMLRPC client" do
+        xml_client = Queri::XmlClient.class_variable_get(:@@server)
+        Queri::XmlClient.class_variable_set(:@@server, @client)
+        subject
+        @client.should have_received(:call)
+        Queri::XmlClient.class_variable_set(:@@server, xml_client)
+      end
+
+      context "given no arguments" do
+        subject { Queri::XmlClient.send_request }
+
+        it "should raise ArgumentError" do
+          expect{ subject }.to raise_error(ArgumentError)
+        end
+      end
+
+      context "given an empty queues array" do
+        subject { Queri::XmlClient.send_request([], report) }
+
+        it "should raise ArgumentError" do
+          expect{ subject }.to raise_error(ArgumentError)
+        end
+      end
+
+      context "given a Stats report object" do
+        context "and period_start or period_end are nil" do
+          let(:report) { double(Queri::Stats::AnsweredCalls::AllCalls) }
+          let(:period_start) { Time.now - 10 }
+          let(:period_end) { nil }
+
+          subject { Queri::XmlClient.send_request(queues, report, period_start, period_end) }
+
+          it "should raise ArgumentError" do
+            expect{ subject }.to raise_error(ArgumentError)
+          end
+        end
+      end
+
+      context "given a Realtime report object" do
+        context "and period_start or period_end are not nil" do
+          let(:period_start) { Time.now - 10 }
+          let(:period_end) { nil }
+
+          subject { Queri::XmlClient.send_request(queues, report, period_start, period_end) }
+
+          it "should raise ArgumentError" do
+            expect{ subject }.to raise_error(ArgumentError)
+          end
+        end
+      end
+    end
+  end
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: queri
 version: !ruby/object:Gem::Version
-  version: 1.0.0
+  version: 1.0.1
 platform: ruby
 authors:
 - Brad Rice
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-02-06 00:00:00.000000000 Z
+date: 2014-03-01 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -85,11 +85,13 @@ files:
 - lib/queri/composite_agent_level_report_helpers.rb
 - lib/queri/composite_by_hour_report_helpers.rb
 - lib/queri/composite_report_helpers.rb
+- lib/queri/helpers/time_helpers.rb
 - lib/queri/realtime.rb
 - lib/queri/realtime/agents.rb
 - lib/queri/realtime/calls.rb
 - lib/queri/realtime/queues.rb
 - lib/queri/realtime_arg_helpers.rb
+- lib/queri/request.rb
 - lib/queri/stats.rb
 - lib/queri/stats/agents_and_sessions.rb
 - lib/queri/stats/agents_and_sessions/agent_availability.rb
@@ -182,7 +184,9 @@ files:
 - lib/queri/stats/unanswered_calls/unanswered_calls_distribution_by_length.rb
 - lib/queri/stats/unanswered_calls/unanswered_outbound_calls_by_agent.rb
 - lib/queri/version.rb
+- lib/queri/xml_client.rb
 - queri.gemspec
+- spec/helpers/time_helpers_spec.rb
 - spec/queri_spec.rb
 - spec/realtime/agents_spec.rb
 - spec/realtime/calls_spec.rb
@@ -292,6 +296,7 @@ files:
 - spec/support/shared/shared_examples_for_report_method.rb
 - spec/support/shared/shared_examples_for_stats_report.rb
 - spec/support/time_helper.rb
+- spec/xml_client_spec.rb
 homepage: ''
 licenses:
 - MIT
@@ -317,6 +322,7 @@ signing_key:
 specification_version: 4
 summary: Send requests to QM's XMLRPC client and receive human-readable responses
 test_files:
+- spec/helpers/time_helpers_spec.rb
 - spec/queri_spec.rb
 - spec/realtime/agents_spec.rb
 - spec/realtime/calls_spec.rb
@@ -426,3 +432,4 @@ test_files:
 - spec/support/shared/shared_examples_for_report_method.rb
 - spec/support/shared/shared_examples_for_stats_report.rb
 - spec/support/time_helper.rb
+- spec/xml_client_spec.rb