mote_sms 1.2.0 → 1.3.0.rc1

data/lib/mote_sms/transports/mobile_technics_transport.rb

@@ -3,6 +3,8 @@ require 'net/http'
 require 'phony'
 require 'logger'
 
+require 'mote_sms/transports/http_client'
+
 module MoteSMS
 
   # MoteSMS::MobileTechnicsTransport provides the implementation to
@@ -16,40 +18,14 @@ module MoteSMS
   #    # => ['000-791234', '001-7987324']
   #
   class MobileTechnicsTransport
-
     # Maximum recipients allowed by API
     MAX_RECIPIENT = 100
 
-    # Path to certificates
-    CERTS_PATH = File.expand_path File.join(File.dirname(__FILE__), '..', 'ssl_certs')
-
     # Custom exception subclass.
     ServiceError = Class.new(::Exception)
 
     # Readable attributes
-    attr_reader :endpoint, :username, :password, :options
-
-    # Internal: The default certificate store, adds all *CA.pem files
-    # from mote_sms/ssl_certs directory.
-    #
-    # Returns a OpenSSL::X509::Store
-    def self.default_cert_store
-      @cert_store ||= OpenSSL::X509::Store.new.tap do |store|
-        Dir["#{CERTS_PATH}/*CA.pem"].each { |c| store.add_file c }
-      end
-    end
-
-    # Internal: Load a X509::Certificate based on the hostname, used to handle
-    # server certificate fingerprinting.
-    #
-    # host - The String with the hostname
-    #
-    # Returns OpenSSL::X509::Certificate or nil if no certificate for this host
-    #  is found
-    def self.fingerprint_cert(host)
-      cert = "#{CERTS_PATH}/#{host}.pem"
-      OpenSSL::X509::Certificate.new(File.read(cert)) if File.exists?(cert)
-    end
+    attr_reader :endpoint, :username, :password, :options, :http_client
 
     # Public: Global default parameters for sending messages, Procs/lambdas
     # are evaluated on #deliver. Ensure to use only symbols as keys. Contains
@@ -61,13 +37,8 @@ module MoteSMS
     #
     # Returns Hash with options.
     def self.defaults
-      @@options ||= {
-        allow_adaption: true,
-        ssl: ->(http) {
-          http.verify_mode = OpenSSL::SSL::VERIFY_PEER
-          http.verify_depth = 9
-          http.cert_store = self.default_cert_store
-        }
+      @options ||= {
+        allow_adaption: true
       }
     end
 
@@ -76,7 +47,7 @@ module MoteSMS
     #
     # Returns Logger instance.
     def self.logger
-      @@logger ||= ::Logger.new($stdout)
+      @logger ||= ::Logger.new($stdout)
     end
 
     # Public: Change the logger used to log all HTTP requests to
@@ -86,7 +57,7 @@ module MoteSMS
     #
     # Returns nothing.
     def self.logger=(logger)
-      @@logger = logger
+      @logger = logger
     end
 
     # Public: Create a new instance using specified endpoint, username
@@ -99,11 +70,17 @@ module MoteSMS
     #           :ssl - SSL client options
     #
     # Returns a new instance.
-    def initialize(endpoint, username, password, options = nil)
+    def initialize(endpoint, username, password, options = {})
       @endpoint = URI.parse(endpoint)
       @username = username
       @password = password
-      @options = options || {}
+
+      @options = self.class.defaults.merge(options)
+
+      @http_client = Transports::HttpClient.new(endpoint,
+        proxy_address: @options[:proxy_address],
+        proxy_port: @options[:proxy_port],
+        ssl: @options[:ssl])
     end
 
     # Public: Delivers message using mobile technics HTTP/S API.
@@ -112,84 +89,26 @@ module MoteSMS
     # options - The Hash with service specific options.
     #
     # Returns Array with sender ids.
-    def deliver(message, options = {})
+    def deliver(message, deliver_options = {})
       raise ArgumentError, "too many recipients, max. is #{MAX_RECIPIENT} (current: #{message.to.length})" if message.to.length > MAX_RECIPIENT
 
-      # Prepare request
-      options = prepare_options options
-      http = http_client options
-      request = http_request post_params(message, options)
+      request = Net::HTTP::Post.new(endpoint.request_uri).tap do |req|
+        req.body = URI.encode_www_form post_params(message, options.merge(deliver_options))
+        req.content_type = 'application/x-www-form-urlencoded; charset=utf-8'
+      end
 
-      # Log as `curl` request
       self.class.logger.debug "curl -X#{request.method} '#{endpoint}' -d '#{request.body}'"
 
-      # Perform request
-      resp = http.request(request)
+      resp = http_client.request(request)
 
-      # Handle errors
       raise ServiceError, "endpoint did respond with #{resp.code}" unless resp.code.to_i == 200
       raise ServiceError, "unable to deliver message to all recipients (CAUSE: #{resp.body.strip})" unless resp.body.split("\n").all? { |l| l =~ /Result_code: 00/ }
 
-      # extract Nth-SmsIds
       resp['X-Nth-SmsId'].split(',')
     end
 
     private
 
-    # Internal: Prepare request including body, headers etc.
-    #
-    # uri - The URI from the endpoint.
-    # params - The Array with the attributes.
-    #
-    # Returns Net::HTTP::Post instance.
-    def http_request(params)
-      Net::HTTP::Post.new(endpoint.request_uri).tap do |request|
-        request.body = URI.encode_www_form params
-        request.content_type = 'application/x-www-form-urlencoded; charset=utf-8'
-      end
-    end
-
-    # Internal: Build new Net::HTTP instance, enables SSL if requested.
-    #
-    # options - The Hash with all options
-    #
-    # Returns Net::HTTP client instance.
-    def http_client(options)
-      Net::HTTP.new(endpoint.host, endpoint.port).tap do |http|
-        if endpoint.instance_of?(URI::HTTPS)
-          cert = self.class.fingerprint_cert(endpoint.host)
-          http.use_ssl = true
-          http.verify_callback = ->(ok, store) { verify_fingerprint(cert.serial, ok, store) } if cert
-          options[:ssl].call(http) if options[:ssl].respond_to?(:call)
-        end
-      end
-    end
-
-    # Public: Verify SSL server certifcate when a certificate is available in
-    # mote_sms/ssl_certs/{host}.pem. Implemented to return false if first
-    # certificate in chain does not match the expected serial.
-    #
-    # serial - The expected server certificates serial
-    # ok - The Boolean forwarded by verify_callback
-    # store - The OpenSSL::X509::Store instance with the chain
-    #
-    # Returns Boolean
-    def verify_fingerprint(serial, ok, store)
-      return false unless store.chain.first.serial == serial
-      ok
-    end
-
-    # Internal: Merge defaults from class and instance with options
-    # supplied to #deliver. Removes `:http` options, because those
-    # are only for the HTTP client to set ssl verify mode et all.
-    #
-    # options - The Hash to merge with #defaults and #options.
-    #
-    # Returns Hash.
-    def prepare_options(options)
-      options = self.class.defaults.merge(self.options).merge(options)
-    end
-
     # Internal: Prepare parameters for sending POST to endpoint, merges defaults,
     # local and per call options, adds message related informations etc etc.
     #
@@ -198,7 +117,7 @@ module MoteSMS
     #
     # Returns Array with params.
     def post_params(message, options)
-      params = options.reject { |key, v| key == :ssl }
+      params = options.reject { |key, v| [:proxy_address, :proxy_port, :ssl].include?(key) }
       params.merge! username: self.username,
                     password: self.password,
                     origin: message.from ? message.from.to_number : params[:origin],

data/lib/mote_sms/transports/swisscom_transport.rb

@@ -0,0 +1,106 @@
+require 'phony'
+require 'logger'
+
+require 'mote_sms/transports/http_client'
+
+module MoteSMS
+
+  # MoteSMS::MobileTechnicsTransport provides the implementation to
+  # send messages using nth.ch bulk SMS HTTP/S API. Each customer has
+  # custom endpoint (with port) and username/password.
+  #
+  # Examples:
+  #
+  #    transport = MoteSMS::SwisscomTransport.new 'https://api.swisscom.com/', 'ApIkEy'
+  #    transport.deliver message
+  #    # => ['000-791234', '001-7987324']
+  #
+  class SwisscomTransport
+    # Maximum recipients allowed by API
+    MAX_RECIPIENT = 1
+
+    # Custom exception subclass.
+    ServiceError = Class.new(::Exception)
+
+    # Readable attributes
+
+    # Public: Logger used to log HTTP requests to mobile
+    # technics API endpoint.
+    #
+    # Returns Logger instance.
+    def self.logger
+      @@logger ||= ::Logger.new($stdout)
+    end
+
+    # Public: Change the logger used to log all HTTP requests to
+    # the endpoint.
+    #
+    # logger - The Logger instance, should at least respond to #debug, #error.
+    #
+    # Returns nothing.
+    def self.logger=(logger)
+      @@logger = logger
+    end
+
+    attr_reader :endpoint, :api_key, :from_number, :options, :http_client
+
+    # Public: Create a new instance using specified endpoint, api_key
+    # and password.
+    #
+    # endpoint - The swisscom base url of the API
+    # api_key - The String with the API key.
+    # from_number - The phone number to send from (mandatory @ swisscom)
+    # options - The Hash with additional URL params passed to mobile techics endpoint
+    #
+    # Returns a new instance.
+    def initialize(endpoint, api_key, from_number = nil, options = {})
+      @endpoint = URI.parse(endpoint)
+      @api_key = api_key
+      @from_number = from_number
+      @options = options
+
+      @http_client = Transports::HttpClient.new(endpoint,
+        proxy_address: options[:proxy_address],
+        proxy_port: options[:proxy_port],
+        ssl: options[:ssl])
+    end
+
+    # Public: Delivers message using mobile technics HTTP/S API.
+    #
+    # message - The MoteSMS::Message instance to send.
+    # options - The Hash with service specific options.
+    #
+    # Returns Array with sender ids.
+    def deliver(message, options = {})
+      raise ArgumentError, "too many recipients, max. is #{MAX_RECIPIENT} (current: #{message.to.length})" if message.to.length > MAX_RECIPIENT
+
+      # Prepare request
+      request = Net::HTTP::Post.new("/messaging/v1/sms").tap do |request|
+        request.body = post_params(message)
+        request.content_type = 'application/json; charset=utf-8'
+        request['Accept'] = 'application/json; charset=utf-8'
+        request['client_id'] = api_key
+      end
+
+      # Log as `curl` request
+      self.class.logger.debug "curl -X#{request.method} '#{endpoint}' -d '#{request.body}'"
+
+      # Perform request
+      resp = http.request(request)
+
+      # Handle errors
+      raise ServiceError, "endpoint did respond with #{resp.code} and #{resp.body}" unless resp.code.to_i == 201
+      self.class.logger.debug resp.body
+    end
+
+    private
+
+    def post_params(message)
+      { to: prepare_numbers(message.to), text: message.body }
+    end
+
+    def prepare_numbers(number_list)
+      number_list.normalized_numbers.map { |n| Phony.formatted(n, format: :international_absolute, spaces: '') }.first
+    end
+  end
+end

data/lib/mote_sms/transports.rb

@@ -2,7 +2,9 @@ module MoteSMS
 
   # All transports live within mote_sms/transports, though should be
   # available in ruby as `MoteSMS::<Some>Transport`.
+  autoload :SslTransport, 'mote_sms/transports/concerns/ssl_transport'
   autoload :TestTransport, 'mote_sms/transports/test_transport'
   autoload :MobileTechnicsTransport, 'mote_sms/transports/mobile_technics_transport'
   autoload :ActionMailerTransport, 'mote_sms/transports/action_mailer_transport'
+  autoload :SwisscomTransport, 'mote_sms/transports/swisscom_transport'
 end

data/lib/mote_sms/version.rb

@@ -1,3 +1,3 @@
 module MoteSMS
-  VERSION = "1.2.0"
+  VERSION = "1.3.0.rc1"
 end

data/lib/mote_sms.rb

@@ -1,14 +1,33 @@
 require 'mote_sms/transports'
 
 module MoteSMS
-  autoload :Number,     'mote_sms/number'
-  autoload :NumberList, 'mote_sms/number_list'
-  autoload :Message,    'mote_sms/message'
+  autoload :Number,      'mote_sms/number'
+  autoload :NumberList,  'mote_sms/number_list'
+  autoload :Message,     'mote_sms/message'
+  autoload :DeliveryJob, 'mote_sms/delivery_job'
 
   autoload :VERSION,    'mote_sms/version'
 
   # No default transport.
   @@transport = nil
+  @@delayed_delivery_queue = :default
+
+  # Public: Get globally defined queue name for ActiveJob, if any.
+  # Defaults to `nil`.
+  #
+  # Returns global ActiveJob queue name.
+  def self.delayed_delivery_queue
+    @@delayed_delivery_queue
+  end
+
+  # Public: Set global queue name for ActiveJob
+  #
+  # queue - A string or symbol that represents the queue to use for ActiveJob
+  #
+  # Returns nothing.
+  def self.delayed_delivery_queue=(queue)
+    @@delayed_delivery_queue = queue
+  end
 
   # Public: Get globally defined transport method, if any.
   # Defaults to `nil`.
@@ -40,6 +59,6 @@ module MoteSMS
   # Returns result of #deliver.
   def self.deliver(&block)
     raise ArgumentError, 'Block missing' unless block_given?
-    Message.new(&block).deliver
+    Message.new(&block).deliver_now
   end
 end

data/mote_sms.gemspec

@@ -3,26 +3,29 @@ require File.expand_path('../lib/mote_sms/version', __FILE__)
 
 Gem::Specification.new do |gem|
   gem.name          = 'mote_sms'
-  gem.authors       = ['Lukas Westermann']
-  gem.email         = ['lukas.westermann@at-point.ch']
-  gem.summary       = %q{Deliver SMS using MobileTechnics HTTP API.}
-  gem.description   = %q{Unofficial ruby adapter for MobileTechnics HTTP Bulk SMS API.
+  gem.authors       = ['Lukas Westermann', 'Loris Gavillet']
+  gem.email         = ['lukas.westermann@at-point.ch', 'loris@at-point.ch']
+  gem.summary       = %q{Deliver SMS using Swisscom / MobileTechnics REST API.}
+  gem.description   = %q{Unofficial ruby adapter for Swisscom and MobileTechnics Bulk SMS APIs.
                          Tries to mimick mail API, so users can switch e.g. ActionMailer
                          with this SMS provider.}
-  gem.homepage      = 'https://at-point.ch/opensource'
+  gem.homepage      = 'https://github.com/at-point/mote_sms'
 
   gem.files         = %w{.gitignore Gemfile Rakefile README.md mote_sms.gemspec} + Dir['**/*.{rb,pem}']
-  gem.executables   = gem.files.grep(%r{^bin/}) { |f| File.basename(f) }
+  gem.bindir        = 'exe'
+  gem.executables   = gem.files.grep(%r{^exe/}) { |f| File.basename(f) }
   gem.test_files    = gem.files.grep(%r{^(test|spec|features)/})
   gem.require_paths = %w{lib}
   gem.version       = MoteSMS::VERSION
 
-  gem.required_ruby_version = '>= 1.9'
+  gem.required_ruby_version = '>= 2.0'
 
   gem.add_dependency 'phony', ['>= 1.7', '< 3.0']
+  gem.add_dependency 'activesupport', ['>= 4.2', '< 6']
 
   gem.add_development_dependency 'rake'
-  gem.add_development_dependency 'rspec', ['~> 2.14']
+  gem.add_development_dependency 'rspec', ['~> 2.4']
   gem.add_development_dependency 'webmock', ['~> 1.8.0']
-  gem.add_development_dependency 'actionmailer', ['>= 3.2']
+  gem.add_development_dependency 'actionmailer', ['>= 4.2', '< 6']
+  gem.add_development_dependency 'activejob', ['>= 4.2', '< 6']
 end

data/spec/mote_sms/delivery_job_spec.rb

@@ -0,0 +1,13 @@
+require 'spec_helper'
+require 'mote_sms'
+
+describe MoteSMS::DeliveryJob do
+  subject { described_class.new }
+
+  context '#perform' do
+    it 'creates a new message based on the params and delivers it' do
+      expect_any_instance_of(MoteSMS::Message).to receive(:deliver_now).with(d: 123)
+      subject.perform('SENDER', ['41791231212'], 'This is the SMS content', d: 123)
+    end
+  end
+end

data/spec/mote_sms/message_spec.rb

@@ -1,5 +1,6 @@
 require 'spec_helper'
-require 'mote_sms/message'
+require 'mote_sms'
+require 'active_job'
 
 describe MoteSMS::Message do
   it 'can be constructed using a block' do
@@ -32,29 +33,71 @@ describe MoteSMS::Message do
   end
 
   context "#deliver" do
+    subject { described_class.new }
+
+    it "delegates to deliver_now and deprecates it" do
+      expect(subject).to receive(:deliver_now)
+      expect(Kernel).to receive(:warn).with('Message#deliver is deprecated and will be removed from MoteSMS. Please use #deliver_now')
+      subject.deliver
+    end
+  end
+
+  context "#deliver_now" do
     let(:transport) { double("Some Transport") }
-    subject { described_class.new(transport) }
+    subject {
+      expect(Kernel).to receive(:warn).with('Message#new(transport) is deprecated and will be removed from MoteSMS')
+      described_class.new(transport)
+    }
 
     it "sends messages to transport" do
       expect(transport).to receive(:deliver).with(subject, {})
-      subject.deliver
+      subject.deliver_now
     end
 
     it "can pass additional attributes to transport" do
       expect(transport).to receive(:deliver).with(subject, serviceid: "myapplication")
-      subject.deliver serviceid: "myapplication"
+      subject.deliver_now serviceid: "myapplication"
     end
 
-    it "can override per message transport using :transport option" do
+    it "can override per message transport using :transport option and it deprecates it" do
       expect(transport).to_not receive(:deliver)
-      subject.deliver transport: double(deliver: true)
+      expect(Kernel).to receive(:warn).with('options[:transport] in Message#deliver_now is deprecated and will be removed from MoteSMS')
+      subject.deliver_now transport: double(deliver: true)
     end
 
     it "uses global MoteSMS.transport if no per message transport defined" do
       message = described_class.new
       expect(transport).to receive(:deliver).with(message, {})
       expect(MoteSMS).to receive(:transport) { transport }
-      message.deliver
+      message.deliver_now
+    end
+  end
+
+  context "#deliver_later" do
+    before { MoteSMS.transport = MoteSMS::TestTransport }
+    after { MoteSMS.transport = nil }
+
+    subject do
+      described_class.new do
+        from 'SENDER'
+        to '+41 79 123 12 12'
+        body 'This is the SMS content'
+      end
+    end
+
+    it "can not override per message transport using :transport option and it deprecates it" do
+      expect(Kernel).to receive(:warn).with('options[:transport] is not supported in Message#deliveer_later')
+      expect(MoteSMS::DeliveryJob).to_not receive(:perform_later)
+      subject.deliver_later transport: double(deliver: true)
+    end
+
+    it "queues the delivery in the DeliveryJob" do
+      subject.deliver_later
+      job = ActiveJob::Base.queue_adapter.enqueued_jobs.first
+      expect(job[:job]).to eq MoteSMS::DeliveryJob
+      expect(job[:args]).to include 'SENDER'
+      expect(job[:args]).to include ['41791231212']
+      expect(job[:args]).to include 'This is the SMS content'
     end
   end
 end

data/spec/mote_sms/number_spec.rb

@@ -2,36 +2,38 @@ require 'spec_helper'
 require 'mote_sms/number'
 
 describe MoteSMS::Number do
+  shared_examples_for 'parsed number' do |formatted, e164|
+    it '#to_s returns a formatted number' do
+      expect(subject.to_s).to eq formatted
+    end
+
+    it '#number returns an E164 number' do
+      expect(subject.number).to eq e164
+    end
+
+    it '#to_number returns an E164 number' do
+      expect(subject.to_number).to eq e164
+    end
+  end
+
   context 'normalized number' do
     subject { described_class.new('41443643533') }
-
-    its(:to_s) { should == '+41 44 364 35 33' }
-    its(:number) { should == '41443643533' }
-    its(:to_number) { should == '41443643533' }
+    it_behaves_like 'parsed number', '+41 44 364 35 33', '41443643533'
   end
 
   context 'E164 conforming number' do
     subject { described_class.new('+41 44 3643533') }
-
-    its(:to_s) { should == '+41 44 364 35 33' }
-    its(:number) { should == '41443643533' }
-    its(:to_number) { should == '41443643533' }
+    it_behaves_like 'parsed number', '+41 44 364 35 33', '41443643533'
   end
 
   context 'handles local numbers' do
     subject { described_class.new('079 700 50 90', cc: '41') }
-
-    its(:to_s) { should == '+41 79 700 50 90' }
-    its(:number) { should == '41797005090'}
-    its(:to_number) { should == '41797005090' }
+    it_behaves_like 'parsed number', '+41 79 700 50 90', '41797005090'
   end
 
   context 'handles numbers with NDC regexp' do
     subject { described_class.new('079 700 50 90', cc: '41', ndc: /(44|79)/) }
-
-    its(:to_s) { should == '+41 79 700 50 90' }
-    its(:number) { should == '41797005090' }
-    its(:to_number) { should == '41797005090' }
+    it_behaves_like 'parsed number', '+41 79 700 50 90', '41797005090'
   end
 
   context 'non conforming number' do
@@ -50,11 +52,11 @@ describe MoteSMS::Number do
 
   context 'vanity numbers' do
     subject { described_class.new('0800-vanity', vanity: true) }
+    it_behaves_like 'parsed number', '0800VANITY', '0800VANITY'
 
-    its(:to_s) { should == '0800VANITY' }
-    its(:number) { should == '0800VANITY' }
-    its(:to_number) { should == '0800VANITY' }
-    its(:vanity?) { should be_truthy }
+    it 'is a #vanity? number' do
+      expect(subject.vanity?).to be_truthy
+    end
 
     it 'raises error if more than 11 alpha numeric chars' do
       expect { described_class.new('1234567890AB', vanity: true) }.to raise_error(ArgumentError, /invalid vanity/i)

data/spec/mote_sms/transports/http_client_spec.rb

@@ -0,0 +1,87 @@
+require 'spec_helper'
+require 'mote_sms/transports/http_client'
+
+describe Transports::HttpClient do
+  subject { described_class.new('https://example.org/') }
+
+  context 'Certificate checks', http: true do
+    context 'api.swisscom.com' do
+      subject { described_class.new('https://api.swisscom.com/', enable_fingerprint: true) }
+
+      it 'makes a "successful" request, i.e. no HTTPS issues' do
+        request = Net::HTTP::Get.new('/')
+        response = subject.request(request)
+        expect(response).to be_a Net::HTTPInternalServerError
+      end
+    end
+
+    context 'https://bulk.mobile-gw.com:9012' do
+      subject { described_class.new('https://bulk.mobile-gw.com:9012', enable_fingerprint: true) }
+
+      it 'makes a "successful" request, i.e. no HTTPS issues' do
+        request = Net::HTTP::Get.new('/')
+        response = subject.request(request)
+        expect(response).to be_a Net::HTTPOK
+      end
+    end
+  end
+
+  context '#initialize' do
+    before { ENV.delete('MOTE_SMS_EXAMPLE_ORG_FINGERPRINT') }
+
+    it 'has a default user agent' do
+      expect(subject.user_agent).to eq "Ruby/mote_sms #{MoteSMS::VERSION}"
+    end
+
+    it 'has no proxy by default' do
+      expect(subject.proxy_address).to be_nil
+      expect(subject.proxy_port).to be_nil
+    end
+
+    it 'tries to load a fingerprint via hostname, when ENV is not set' do
+      expect(described_class).to receive(:fingerprint_host).with('example.org') { 'pem-fingerprint' }
+      expect(subject.fingerprint).to eq 'pem-fingerprint'
+    end
+
+    it 'tries to use the ENV for a fingerprint lookup' do
+      ENV['MOTE_SMS_EXAMPLE_ORG_FINGERPRINT'] = 'env-fingerprint'
+      allow(described_class).to receive(:fingerprint_host) { 'pem-fingerprint' }
+      expect(subject.fingerprint).to eq 'env-fingerprint'
+    end
+
+    context 'with enable_fingerprint: false' do
+      subject { described_class.new('https://example.org', enable_fingerprint: false) }
+
+      it 'can skip fingerprinting by setting enable_fingerprint: false' do
+        ENV['MOTE_SMS_EXAMPLE_ORG_FINGERPRINT'] = 'env-fingerprint'
+        allow(described_class).to receive(:fingerprint_host) { 'pem-fingerprint' }
+        expect(subject.fingerprint).to be_nil
+      end
+    end
+  end
+
+  context '#https?' do
+    it 'returns true when it is a HTTPS url' do
+      expect(subject.https?).to be_truthy
+    end
+
+    it 'returns false (of course) when it is just HTTP' do
+      expect(described_class.new('http://foo.example.org').https?).to be_falsey
+    end
+  end
+
+  context '#request' do
+    let(:request) { Net::HTTP::Get.new('/') }
+
+    before {
+      stub_request(:get, "https://example.org/").
+        with(headers: { 'Accept' => '*/*', 'Accept-Encoding' => 'gzip;q=1.0,deflate;q=0.6,identity;q=0.3', 'User-Agent' => 'Ruby/mote_sms 1.2.0' }).
+        to_return(status: 200, body: "", headers: {})
+    }
+
+    it 'submits a request and overrides the UA' do
+      response = subject.request(request)
+      expect(response).to be_a Net::HTTPOK
+    end
+  end
+end