mblox 0.3.0 → 0.4.0

checksums.yaml

@@ -1,15 +1,15 @@
 ---
 !binary "U0hBMQ==":
   metadata.gz: !binary |-
-    NDhiNjUwZDE4N2YyOTU3MGI3NWQ2MjBmYmUxODYzOTAzZjdjOGNjMA==
+    NWJiZmZlZWIxMDViNmIxNGE3MzE2NWYzYzZlY2M1MWIyNDg0MzkxNQ==
   data.tar.gz: !binary |-
-    MDc4NzYxYWE4MjhkZGJiNDE5OTZiNGUyODBhODNhMWZkZDZlOGQ4NA==
+    ZDY0YmNkNzdjYmNhNDZkMDBhYjMxZTliNDdkYmNkZmVkMDhhMjk2MQ==
 SHA512:
   metadata.gz: !binary |-
-    OGNhNjBjYjI0NWUxMGJmZGU3ODQxZTIyMDNiNjIwNjk1NzNkNDdkNzQ3Zjc3
-    OTc1NWI5MDk3ZjU0OTAzZTE4YTE0YjY4YTAwZjEzOTM1YjJjZDllNWUyMDEz
-    NmUxMzQ2YjJmZWVhZjgyZTVlODA4YmNkMjZjY2U3ZjZiYmVjN2I=
+    MjE3NDJkODAwNThiOTZhYTQ0Y2EwZmQ5YzYxYzU2ZTgwZTQwNWQwODQwYzM0
+    ODcwZWU1NDIwODE3NmJiMDA0OWM2Y2VjZjg4OWI2NDUxMjNkYWY5ZDA5ZjE2
+    M2VhY2U4NzlmNGRjYjMyYzQ0ZGZjZGFmY2IyZThmZjFjODEyOGM=
   data.tar.gz: !binary |-
-    MTNiMThhMDMzZTRkZTJlZmIyNGQ0OWVjMzIxNTNkYzJiN2Y4YWY5NzdmYTdm
-    MmIxNjE5NDI3NDZkYWY3MTlmMmUzODAwYzgwYzg4NTAwNTY5ZWNiNTBmODU1
-    ZTJhYTRjNWRiOTkzZTk1ZWY0MTFiZTFmMWIwYjRhMGE0OGMzY2U=
+    MzllZTE0OGZiZTg0ZTg0NTEyYWM5NjllYjMzZDMxMzY4ZmI2YzcwYWU5YTBl
+    NTA0ZWIzYzEyYmFmYzhiZjFmOWY5NGFhODcyNTg1MGM1ODE3MTkxYTc4ZTNi
+    M2VjNzA5NDk1YWY3NzU3NzkyMWRlMDk5ZmM1ODkwZGE5OWFmNjY=

data/README.md

@@ -48,7 +48,8 @@ Once your application is configured, send messages:
 
     # The number to sending to must be a 10-digit number, including the area code.  Can be a String or Fixnum.
     phone_number = 2225555555 # or: phone_number = "2225555555"
-    Mblox::Sms.new(phone_number, "your message").send
+    responses = Mblox::Sms.new(phone_number, "your message").send # Returns an array of `Mblox::SmsResponse`'s
+    responses.first.should be_ok # each Mblox::SmsResponse responds to #ok? and #unroutable?
 
 ## Testing
 

data/lib/mblox/from_xml.rb

@@ -2,6 +2,8 @@ require 'nokogiri'
 
 module Mblox
   class MissingExpectedXmlContentError < StandardError; end
+  class ValidationError < StandardError; end
+
   class << self
     def from_xml(xml)
       Nokogiri::XML(xml) { |config| config.nonet }.tap do |_|

data/lib/mblox/sms.rb

@@ -13,7 +13,7 @@ module Mblox
     class InvalidSenderIdError < ::ArgumentError; end
     MAX_LENGTH = 160
     MAX_SECTION_LENGTH = MAX_LENGTH - "(MSG XXX/XXX): ".size
-    LEGAL_CHARACTERS = "~\`!\"#\$\%&'\(\)*+,-.\/:;<=>?@_£¤¥§¿i¡ÄÅÆÇÉÑÖØÜßáäåæèéìñòöøóùüú\n\r\tí "
+    LEGAL_CHARACTERS = "~\`!\"#\$\%&'\(\)*+,-.\/:;<=>?@_£¤¥§¿i¡ÄÅÆÇÉÑÖØÜßâáäåæèéìñòöøóùüú\n\r\tí "
     ILLEGAL_CHARACTERS = /([^a-zA-Z0-9#{LEGAL_CHARACTERS}\\])/
 
     attr_reader :phone, :message
@@ -53,7 +53,7 @@ module Mblox
         request.body = request_body
         response = self.class.http.start{ |http| http.request(request) }.body
         Mblox.log "Mblox responds with:\n#{response}"
-        SmsResponse.new(response)
+        SmsResponse.from_xml(response)
       end
 
       def build(message)

data/lib/mblox/sms_receipt.rb

@@ -1,50 +1,78 @@
+require 'active_support/core_ext/hash/keys.rb'
 require 'mblox/from_xml'
 
 module Mblox
   class SmsReceipt
-    attr_reader :batch_id, :subscriber_number, :timestamp, :msg_reference, :status, :reason, :operator
-    def initialize(xml)
-      data = Mblox.from_xml(xml).xpath '//NotificationService'
-      raise MissingExpectedXmlContentError, "Xml should have contained a 'NotificationService' node, but was #{xml}" if data.blank?
-
-      data = data.xpath '//NotificationList'
-      raise MissingExpectedXmlContentError, "Xml should have contained a 'NotificationService' -> 'NotificationList' node, but was #{xml}" if data.blank?
-
-      data = data.xpath '//Notification'
-      raise MissingExpectedXmlContentError, "Xml should have contained a 'NotificationService' -> 'NotificationList' -> 'Notification' node, but was #{xml}" if data.blank?
-      @batch_id = data.attribute('BatchID').value.to_i
-
-      data = data.xpath '//Subscriber'
-      raise MissingExpectedXmlContentError, "Xml should have contained a 'NotificationService' -> 'NotificationList' -> 'Notification' -> 'Subscriber' node, but was #{xml}" if data.blank?
-
-      @subscriber_number = value_at(:SubscriberNumber, data)
-      @subscriber_number = @subscriber_number[1..-1] if '1' == @subscriber_number[0]
-
-      timestamp = value_at(:TimeStamp, data)
-      unless timestamp.blank?
-        @timestamp = begin
-          Time.strptime("#{timestamp}+0000", '%Y%m%d%H%M%z')
-        rescue ArgumentError
-          nil
-        end
-      end
+    ATTRIBUTES = [:batch_id, :subscriber_number, :msg_reference, :status, :timestamp, :reason, :operator]
+    attr_reader *ATTRIBUTES
 
-      @timestamp = @timestamp.to_datetime if @timestamp
-      @msg_reference = value_at(:MsgReference, data)
-      @status = value_at(:Status, data)
-      reason = value_at(:Reason, data)
-      @reason = reason.blank? ? nil : reason.to_i
-      data = data.xpath('//Tags').xpath('//Tag')
-      return if data.empty?
-      data.each do |d|
-        @operator = d.child.content.to_i if "Operator" == data.attribute('Name').content
-        return if @operator
+    def initialize args
+      args = args.symbolize_keys
+      ATTRIBUTES.each do |attr|
+        __send__("#{attr}=", args[attr])
+        args.delete(attr)
       end
+      raise ::ArgumentError, "Unrecognized attributes: #{args.inspect}" unless args.empty?
+      missing_fields = ATTRIBUTES.reject { |attr| [:timestamp, :reason, :operator].include?(attr) || __send__(attr) }
+      if 1 == missing_fields.count
+        raise ValidationError, "#{missing_fields.first} cannot be blank"
+      elsif missing_fields.count > 1
+        raise ValidationError, "The following fields cannot be blank: #{missing_fields.join(', ')}"
+      end
+      raise ValidationError, "batch_id must be a Fixnum" unless batch_id.is_a?(Fixnum)
+      raise ValidationError, "reason must be a Fixnum" unless reason.nil? || reason.is_a?(Fixnum)
+      raise ValidationError, "timestamp must be a DateTime" unless timestamp.nil? || timestamp.is_a?(DateTime)
     end
     private
-      def value_at(path, data)
-        data = data.xpath("//#{path}")
-        (data.empty? || data.children.empty?) ? nil : data.first.child.content
+      attr_writer *ATTRIBUTES
+
+    class << self
+      def from_xml(xml)
+        args = {}
+        data = Mblox.from_xml(xml).xpath '//NotificationService'
+        raise MissingExpectedXmlContentError, "Xml should have contained a 'NotificationService' node, but was #{xml}" if data.blank?
+
+        data = data.xpath '//NotificationList'
+        raise MissingExpectedXmlContentError, "Xml should have contained a 'NotificationService' -> 'NotificationList' node, but was #{xml}" if data.blank?
+
+        data = data.xpath '//Notification'
+        raise MissingExpectedXmlContentError, "Xml should have contained a 'NotificationService' -> 'NotificationList' -> 'Notification' node, but was #{xml}" if data.blank?
+        args[:batch_id]= data.attribute('BatchID').value.to_i
+
+        data = data.xpath '//Subscriber'
+        raise MissingExpectedXmlContentError, "Xml should have contained a 'NotificationService' -> 'NotificationList' -> 'Notification' -> 'Subscriber' node, but was #{xml}" if data.blank?
+
+        args[:subscriber_number] = value_at(:SubscriberNumber, data)
+        args[:subscriber_number] = args[:subscriber_number][1..-1] if '1' == args[:subscriber_number][0]
+
+        timestamp = value_at(:TimeStamp, data)
+        unless timestamp.blank?
+          args[:timestamp] = begin
+            Time.strptime("#{timestamp}+0000", '%Y%m%d%H%M%z')
+          rescue ArgumentError
+            nil
+          end
+        end
+
+        args[:timestamp] = args[:timestamp].to_datetime if args[:timestamp]
+        args[:msg_reference] = value_at(:MsgReference, data)
+        args[:status] = value_at(:Status, data)
+        reason = value_at(:Reason, data)
+        args[:reason] = reason.blank? ? nil : reason.to_i
+        data = data.xpath('//Tags').xpath('//Tag')
+        unless data.empty?
+          data.each do |d|
+            args[:operator] = d.child.content.to_i if "Operator" == data.attribute('Name').content
+            break if args[:operator]
+          end
+        end
+        new(args)
       end
+      private
+        def value_at(path, data)
+          data = data.xpath("//#{path}")
+          (data.empty? || data.children.empty?) ? nil : data.first.child.content
+        end
+    end
   end
 end

data/lib/mblox/sms_response.rb

@@ -36,28 +36,28 @@ module Mblox
       UNROUTABLE = new(10, UNROUTABLE_TEXT)
     end
 
-    attr_reader :request, :result, :subscriber_result
-    def initialize(xml)
-      data = Mblox.from_xml(xml).xpath '//NotificationRequestResult'
-
-      raise MissingExpectedXmlContentError, "Xml should have contained a 'NotificationRequestResult' node, but was #{xml}" if data.blank?
-      header = data.xpath '//NotificationResultHeader'
-      raise MissingExpectedXmlContentError, "Xml should have contained a 'NotificationRequestResult' -> 'NotificationResultHeader' node, but was #{xml}" if header.blank?
-      @request = Result.from_xml(header, :RequestResult)
-      @request = nil unless @request.valid?
-
-      result_list = data.xpath '//NotificationResultList'
-      raise MissingExpectedXmlContentError, "Xml should have contained a 'NotificationRequestResult' -> 'NotificationResultList' node, but was #{xml}" if result_list.blank?
-      result_list = result_list.xpath '//NotificationResult'
-      raise MissingExpectedXmlContentError, "Xml should have contained a 'NotificationRequestResult' -> 'NotificationResultList' -> 'NotificationResult' node, but was #{xml}" if result_list.blank?
-      @result = Result.from_xml(result_list, :NotificationResult)
-      @result = nil unless @result.valid?
-
-      if @result.ok?
-        result_list = result_list.xpath '//SubscriberResult'
-        raise MissingExpectedXmlContentError, "Xml should have contained a 'NotificationRequestResult' -> 'NotificationResultList' -> 'NotificationResult' -> 'SubscriberResult' node, but was #{xml}" if result_list.blank?
-        @subscriber_result = Result.from_xml(result_list, :SubscriberResult)
-        @subscriber_result = nil unless @subscriber_result.valid?
+
+    ATTRIBUTES = [:request, :result, :subscriber_result]
+    attr_reader *ATTRIBUTES
+
+    def initialize args
+      args = args.symbolize_keys
+      ATTRIBUTES.each do |attr|
+        __send__("#{attr}=", args[attr])
+        args.delete(attr)
+      end
+      raise ::ArgumentError, "Unrecognized attributes: #{args.inspect}" unless args.empty?
+      missing_fields = ATTRIBUTES.reject { |attr| __send__(attr) }
+      if 1 == missing_fields.count
+        raise ValidationError, "#{missing_fields.first} cannot be blank"
+      elsif missing_fields.count > 1
+        raise ValidationError, "The following fields cannot be blank: #{missing_fields.join(', ')}"
+      end
+      wrong_type_fields = ATTRIBUTES.reject { |attr| __send__(attr).is_a?(self.class::Result) }
+      if 1 == wrong_type_fields.count
+        raise ValidationError, "#{wrong_type_fields.first} must be of type Mblox::SmsResponse::Result"
+      elsif wrong_type_fields.count > 1
+        raise ValidationError, "The following fields must be of type Mblox::SmsResponse::Result: #{wrong_type_fields.join(', ')}"
       end
     end
 
@@ -68,5 +68,36 @@ module Mblox
     def unroutable?
       @request.ok? && @result.ok? && Result::UNROUTABLE == @subscriber_result
     end
+
+    private
+      attr_writer *ATTRIBUTES
+
+    class << self
+      def from_xml(xml)
+        args = {}
+        data = Mblox.from_xml(xml).xpath '//NotificationRequestResult'
+
+        raise MissingExpectedXmlContentError, "Xml should have contained a 'NotificationRequestResult' node, but was #{xml}" if data.blank?
+        header = data.xpath '//NotificationResultHeader'
+        raise MissingExpectedXmlContentError, "Xml should have contained a 'NotificationRequestResult' -> 'NotificationResultHeader' node, but was #{xml}" if header.blank?
+        args[:request] = Result.from_xml(header, :RequestResult)
+        args[:request] = nil unless args[:request].valid?
+
+        result_list = data.xpath '//NotificationResultList'
+        raise MissingExpectedXmlContentError, "Xml should have contained a 'NotificationRequestResult' -> 'NotificationResultList' node, but was #{xml}" if result_list.blank?
+        result_list = result_list.xpath '//NotificationResult'
+        raise MissingExpectedXmlContentError, "Xml should have contained a 'NotificationRequestResult' -> 'NotificationResultList' -> 'NotificationResult' node, but was #{xml}" if result_list.blank?
+        args[:result] = Result.from_xml(result_list, :NotificationResult)
+        args[:result] = nil unless args[:result].valid?
+
+        if args[:result].ok?
+          result_list = result_list.xpath '//SubscriberResult'
+          raise MissingExpectedXmlContentError, "Xml should have contained a 'NotificationRequestResult' -> 'NotificationResultList' -> 'NotificationResult' -> 'SubscriberResult' node, but was #{xml}" if result_list.blank?
+          args[:subscriber_result] = Result.from_xml(result_list, :SubscriberResult)
+          args[:subscriber_result] = nil unless args[:subscriber_result].valid?
+        end
+        new(args)
+      end
+    end
   end
 end

data/lib/mblox/version.rb

@@ -1,3 +1,3 @@
 module Mblox
-  VERSION = "0.3.0"
+  VERSION = "0.4.0"
 end

data/spec/sms_receipt_spec.rb

@@ -21,6 +21,7 @@ describe Mblox::SmsReceipt do
     3
   end
 
+  describe "from_xml" do
   def valid
     xml = Builder::XmlMarkup.new
     xml.instruct!(:xml, :version => 1.0, :encoding => "ISO-8859-1", :standalone => :yes)
@@ -92,6 +93,8 @@ describe Mblox::SmsReceipt do
           n.Subscriber do |s|
             s.SubscriberNumber("2#{subscriber_number}")
             s.TimeStamp('Abcdefg')
+            s.MsgReference(msg_reference)
+            s.Status(status)
             s.Reason
           end
         end
@@ -101,7 +104,7 @@ describe Mblox::SmsReceipt do
   end
 
   it "should access attributes for valid data" do
-    target = described_class.new(valid)
+    target = described_class.from_xml(valid)
     target.batch_id.should == batch_id
     target.subscriber_number.should == subscriber_number
     target.timestamp.should == DateTime.new(2013,10,7,17,36)
@@ -112,43 +115,80 @@ describe Mblox::SmsReceipt do
   end
 
   it "should raise error when missing root node" do
-    expect{described_class.new('Abcdefg')}.to raise_error(Mblox::MissingExpectedXmlContentError, "'Abcdefg' is not parseable as XML")
+    expect{described_class.from_xml('Abcdefg')}.to raise_error(Mblox::MissingExpectedXmlContentError, "'Abcdefg' is not parseable as XML")
   end
 
   it "should raise error when missing notification service node" do
-    expect{described_class.new(missing_notification_service)}.to raise_error(Mblox::MissingExpectedXmlContentError, "Xml should have contained a 'NotificationService' node, but was #{missing_notification_service}")
+    expect{described_class.from_xml(missing_notification_service)}.to raise_error(Mblox::MissingExpectedXmlContentError, "Xml should have contained a 'NotificationService' node, but was #{missing_notification_service}")
   end
 
   it "should raise error when missing notification list node" do
-    expect{described_class.new(missing_notification_list)}.to raise_error(Mblox::MissingExpectedXmlContentError, "Xml should have contained a 'NotificationService' -> 'NotificationList' node, but was #{missing_notification_list}")
+    expect{described_class.from_xml(missing_notification_list)}.to raise_error(Mblox::MissingExpectedXmlContentError, "Xml should have contained a 'NotificationService' -> 'NotificationList' node, but was #{missing_notification_list}")
   end
 
   it "should raise error when missing notification node" do
-    expect{described_class.new(missing_notification)}.to raise_error(Mblox::MissingExpectedXmlContentError, "Xml should have contained a 'NotificationService' -> 'NotificationList' -> 'Notification' node, but was #{missing_notification}")
+    expect{described_class.from_xml(missing_notification)}.to raise_error(Mblox::MissingExpectedXmlContentError, "Xml should have contained a 'NotificationService' -> 'NotificationList' -> 'Notification' node, but was #{missing_notification}")
   end
 
   it "should raise error when missing subscriber node" do
-    expect{described_class.new(missing_subscriber)}.to raise_error(Mblox::MissingExpectedXmlContentError, "Xml should have contained a 'NotificationService' -> 'NotificationList' -> 'Notification' -> 'Subscriber' node, but was #{missing_subscriber}")
+    expect{described_class.from_xml(missing_subscriber)}.to raise_error(Mblox::MissingExpectedXmlContentError, "Xml should have contained a 'NotificationService' -> 'NotificationList' -> 'Notification' -> 'Subscriber' node, but was #{missing_subscriber}")
   end
 
   describe "subscriber_number" do
     it "should not drop leading character unless it is '1'" do
-      target = described_class.new(unexpected_nested_values)
+      target = described_class.from_xml(unexpected_nested_values)
       target.subscriber_number.should == "2#{subscriber_number}"
     end
   end
 
   describe "reason" do
     it "should leave reason blank if it is nil" do
-      target = described_class.new(unexpected_nested_values)
+      target = described_class.from_xml(unexpected_nested_values)
       target.reason.should be_nil
     end
   end
 
   describe "timestamp" do
     it "should fail gracefully if it can't be converted into a DateTime" do
-      target = described_class.new(unexpected_nested_values)
+      target = described_class.from_xml(unexpected_nested_values)
       target.timestamp.should be_nil
     end
   end
+  end
+
+  describe "initialize" do
+    let(:args) { { :batch_id => batch_id, :subscriber_number => subscriber_number, :msg_reference => msg_reference, :status => status, :timestamp => DateTime.new(2013,10,7,17,36), :reason => reason, :operator => 10487 } }
+
+    [:batch_id, :subscriber_number, :msg_reference, :status].each do |attr|
+      it "should raise an error if #{attr} is missing" do
+        expect{described_class.new(args.merge(attr => nil))}.to raise_error(Mblox::ValidationError, "#{attr} cannot be blank")
+      end
+    end
+
+    [:timestamp, :reason, :operator].each do |attr|
+      it "should not raise an error if #{attr} is missing" do
+        expect{described_class.new(args.merge(attr => nil))}.to_not raise_error
+      end
+    end
+
+    it "should raise an error if batch_id, subscriber_number, msg_reference and status are missing" do
+      expect{described_class.new(args.merge(:batch_id => nil, :subscriber_number => nil, :msg_reference => nil, :status => nil))}.to raise_error(Mblox::ValidationError, "The following fields cannot be blank: batch_id, subscriber_number, msg_reference, status")
+    end
+
+    it "should raise an error if batch_id is not a Fixnum" do
+      expect{described_class.new(args.merge(:batch_id => 'ABC'))}.to raise_error(Mblox::ValidationError, "batch_id must be a Fixnum")
+    end
+
+    it "should raise an error if reason is not a Fixnum" do
+      expect{described_class.new(args.merge(:reason => 'ABC'))}.to raise_error(Mblox::ValidationError, "reason must be a Fixnum")
+    end
+
+    it "should raise an error if timestamp is not a DateTime" do
+      expect{described_class.new(args.merge(:timestamp => Time.now))}.to raise_error(Mblox::ValidationError, "timestamp must be a DateTime")
+    end
+
+    it "should raise an error if an unrecognized attribute is present" do
+      expect{described_class.new(args.merge(:extra_attribute => 'ABC'))}.to raise_error(::ArgumentError, 'Unrecognized attributes: {:extra_attribute=>"ABC"}')
+    end
+  end
 end

data/spec/sms_response_spec.rb

@@ -0,0 +1,26 @@
+require "spec_helper"
+
+describe Mblox::SmsResponse do
+  let(:args) { { :request => Mblox::SmsResponse::Result.new(9, "SomeRequest"), :result => Mblox::SmsResponse::Result.new(10, "SomeResult") , :subscriber_result => Mblox::SmsResponse::Result.new(11, "SomeSubscriberResult")  } }
+
+  [:request, :result, :subscriber_result].each do |attr|
+    describe attr do
+      it "cannot be blank" do
+        expect{described_class.new(args.merge(:"#{attr}" => nil))}.to raise_error(Mblox::ValidationError, "#{attr} cannot be blank")
+      end
+      it "must be a Result" do
+        expect{described_class.new(args.merge(:"#{attr}" => 123))}.to raise_error(Mblox::ValidationError, "#{attr} must be of type Mblox::SmsResponse::Result")
+      end
+    end
+  end
+
+  it "should raise an error if request, result and subscriber_result are missing" do
+    expect{described_class.new({})}.to raise_error(Mblox::ValidationError, "The following fields cannot be blank: request, result, subscriber_result")
+  end
+  it "should raise an error if request, result and subscriber_result are the wrong types" do
+    expect{described_class.new(:request => 'A', :result => Time.now, :subscriber_result => 32)}.to raise_error(Mblox::ValidationError, "The following fields must be of type Mblox::SmsResponse::Result: request, result, subscriber_result")
+  end
+  it "should raise an error if an unrecognized attribute is present" do
+    expect{described_class.new(args.merge(:extra_attribute => 'ABC'))}.to raise_error(::ArgumentError, 'Unrecognized attributes: {:extra_attribute=>"ABC"}')
+  end
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: mblox
 version: !ruby/object:Gem::Version
-  version: 0.3.0
+  version: 0.4.0
 platform: ruby
 authors:
 - Isaac Betesh
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-11-20 00:00:00.000000000 Z
+date: 2014-01-22 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -147,6 +147,7 @@ files:
 - spec/configuration_spec.rb
 - spec/sms_receipt_spec.rb
 - spec/sms_response_result_spec.rb
+- spec/sms_response_spec.rb
 - spec/sms_spec.rb
 - spec/spec_helper.rb
 homepage: ''
@@ -188,16 +189,18 @@ summary: ! '# Mblox  This gem is for subscribers to Mblox to send SMS messages.
   is :raise_error # Other options are :truncate and :split config.on_message_too_long
   = :truncate end  Once your application is configured, send messages:  # The number
   to sending to must be a 10-digit number, including the area code.  Can be a String
-  or Fixnum. phone_number = 2225555555 # or: phone_number = "2225555555" Mblox::Sms.new(phone_number,
-  "your message").send  ## Testing  Copy config.yml.example to config.yml and set
-  all the values in that file.  Run:  rspec  You should recieve 6 SMS messages to
-  your phone within several seconds.  ## Contributing  1. Fork it 2. Create your feature
-  branch (`git checkout -b my-new-feature`) 3. Commit your changes (`git commit -am
-  ''Add some feature''`) 4. Push to the branch (`git push origin my-new-feature`)
-  5. Create new Pull Request'
+  or Fixnum. phone_number = 2225555555 # or: phone_number = "2225555555" responses
+  = Mblox::Sms.new(phone_number, "your message").send # Returns an array of `Mblox::SmsResponse`''s
+  responses.first.should be_ok # each Mblox::SmsResponse responds to #ok? and #unroutable?  ##
+  Testing  Copy config.yml.example to config.yml and set all the values in that file.  Run:  rspec  You
+  should recieve 6 SMS messages to your phone within several seconds.  ## Contributing  1.
+  Fork it 2. Create your feature branch (`git checkout -b my-new-feature`) 3. Commit
+  your changes (`git commit -am ''Add some feature''`) 4. Push to the branch (`git
+  push origin my-new-feature`) 5. Create new Pull Request'
 test_files:
 - spec/configuration_spec.rb
 - spec/sms_receipt_spec.rb
 - spec/sms_response_result_spec.rb
+- spec/sms_response_spec.rb
 - spec/sms_spec.rb
 - spec/spec_helper.rb