pew_pew 0.0.2 → 0.1.0

data/spec/pew_pew/config_spec.rb

@@ -1,38 +1,38 @@
-require 'spec_helper'
+module PewPew
+  RSpec.describe Config do
+    {
+      BASE_URI: 'https://api.mailgun.net/v2',
+      USER_AGENT: "PewPew Ruby Gem #{VERSION}",
+      USERNAME: 'api'
+    }.each do |constant, value|
+      context "::#{constant}" do
+        subject { described_class.const_get(constant) }
 
-describe PewPew::Config do
-  {
-    BASE_URI: 'https://api.mailgun.net/v2',
-    USER_AGENT: "PewPew Ruby Gem #{PewPew::VERSION}",
-    USERNAME: 'api'
-  }.each do |constant, value|
-    context constant do
-      subject { described_class.const_get(constant) }
-
-      it { should == value }
+        it { should == value }
+      end
     end
-  end
 
-  subject { Class.new { include PewPew::Config }.new }
+    subject(:config) { Class.new { include Config }.new }
 
-  context 'defaults' do
-    its(:api_key) { should be_nil }
-    its(:domain) { should be_nil }
-  end
+    context 'defaults' do
+      its(:api_key) { should be_nil }
+      its(:domain) { should be_nil }
+    end
 
-  context '#api_key=' do
-    specify do
-      expect {
-        subject.api_key = 'key-02n9f3ijl9sm9u97-8p7r-d7-15q-ui1'
-      }.to change(subject, :api_key).to('key-02n9f3ijl9sm9u97-8p7r-d7-15q-ui1')
+    context '#api_key=' do
+      specify do
+        expect { config.api_key = 'key-02n9f3ijl9sm9u97-8p7r-d7-15q-ui1' }
+          .to change(config, :api_key)
+          .to('key-02n9f3ijl9sm9u97-8p7r-d7-15q-ui1')
+      end
     end
-  end
 
-  context '#domain=' do
-    specify do
-      expect {
-        subject.domain = 'pewpew.mailgun.org'
-      }.to change(subject, :domain).to('pewpew.mailgun.org')
+    context '#domain=' do
+      specify do
+        expect {
+          config.domain = 'pewpew.mailgun.org'
+        }.to change(config, :domain).to('pewpew.mailgun.org')
+      end
     end
   end
 end

data/spec/pew_pew/resource_spec.rb

@@ -1,46 +1,52 @@
-require 'spec_helper'
-
-describe PewPew::Resource do
-  let(:client) { PewPew::Client.new }
-  let(:resource) { Class.new { include PewPew::Resource }.new(client) }
-
-  context '#connection' do
-    subject { resource.send(:connection) }
-
-    context 'middleware' do
-      let(:handlers) { subject.builder.handlers }
-
-      [
-        PewPew::Resource::ResponseDecorator,
-        FaradayMiddleware::Mashify,
-        FaradayMiddleware::ParseJson,
-        Faraday::Request::Multipart,
-        Faraday::Request::UrlEncoded,
-        Faraday::Adapter::NetHttp
-      ].each.with_index do |middleware, index|
-        it "uses #{middleware}" do
-          handlers.index(middleware).should == index
+module PewPew
+  RSpec.describe Resource do
+    let(:client) { Client.new }
+
+    subject(:resource) { Class.new { include Resource }.new(client) }
+
+    context '#connection' do
+      let(:connection) { resource.send(:connection) }
+
+      context 'middleware' do
+        let(:handlers) { connection.builder.handlers }
+
+        [
+          Resource::ResponseDecorator,
+          Faraday::Mashify::Middleware,
+          Faraday::Response::Json,
+          Faraday::Multipart::Middleware,
+          Faraday::Request::UrlEncoded
+        ].each.with_index do |middleware, index|
+          it "uses #{middleware}" do
+            expect(handlers.index(middleware)).to eq index
+          end
         end
       end
     end
   end
-end
 
-describe PewPew::Resource::ResponseDecorator do
-  it { should be_a(Faraday::Response::Middleware) }
+  RSpec.describe Resource::ResponseDecorator do
+    subject(:middleware) { described_class.new }
 
-  it 'converts an array response into a mash' do
-    env = { body: [:response] }
-    subject.on_complete(env)
-    env[:body].total_count.should == 1
-    env[:body].items.should == [:response]
-  end
+    it { is_expected.to be_a Faraday::Middleware }
 
-  it 'stores the request status on the response body' do
-    env = { body: PewPew::Response.new, status: 200 }
+    it 'converts an array response into a mash' do
+      env = { body: [:response] }
 
-    expect {
-      subject.on_complete(env)
-    }.to change(env[:body], :status).from(nil).to(200)
+      middleware.on_complete env
+
+      expect(env).to match a_hash_including(
+        body: an_object_having_attributes(total_count: 1, items: [:response])
+      )
+    end
+
+    it 'stores the request status on the response body' do
+      env = { body: Response.new, status: 200 }
+
+      expect { middleware.on_complete env }
+        .to change(env[:body], :status)
+        .from(nil)
+        .to(200)
+    end
   end
 end

data/spec/pew_pew/resources/bounces_spec.rb

@@ -1,78 +1,82 @@
-require 'spec_helper'
+module PewPew
+  module Resources
+    RSpec.describe Bounces, :resource, :domain do
+      let(:resource) { described_class.new(client) }
 
-describe PewPew::Resources::Bounces, :resource, :domain do
-  let(:resource) { described_class.new(client) }
+      context '#all' do
+        let(:response) { resource.all }
 
-  context '#all' do
-    let(:response) { resource.all }
+        subject { response }
 
-    subject { response }
+        specify { should be_success }
 
-    specify { should be_success }
+        its(:status) { should == 200 }
+        its(:total_count) { should == 1 }
 
-    its(:status) { should == 200 }
-    its(:total_count) { should == 1 }
+        context 'item' do
+          subject { response.items.first }
 
-    context 'item' do
-      subject { response.items.first }
+          its(:address) { should == 'test@example.com' }
+          its(:code) { should == '554' }
+          its(:error) { should == 'Relay access denied' }
+          its(:created_at) { should == 'Sat, 02 Jun 2012 07:24:08 GMT' }
+        end
+      end
 
-      its(:address) { should == 'test@example.com' }
-      its(:code) { should == '554' }
-      its(:error) { should == 'Relay access denied' }
-      its(:created_at) { should == 'Sat, 02 Jun 2012 07:24:08 GMT' }
-    end
-  end
+      context '#find' do
+        let(:response) { resource.find('test@example.com') }
 
-  context '#find' do
-    let(:response) { resource.find('test@example.com') }
+        subject { response }
 
-    subject { response }
+        specify { should be_success }
 
-    specify { should be_success }
+        its(:status) { should == 200 }
 
-    its(:status) { should == 200 }
+        context 'bounce' do
+          subject { response.bounce }
 
-    context 'bounce' do
-      subject { response.bounce }
-
-      its(:address) { should == 'test@example.com' }
-      its(:code) { should == '554' }
-      its(:error) { should == 'Relay access denied' }
-      its(:created_at) { should == 'Sat, 02 Jun 2012 07:24:08 GMT' }
-    end
-  end
+          its(:address) { should == 'test@example.com' }
+          its(:code) { should == '554' }
+          its(:error) { should == 'Relay access denied' }
+          its(:created_at) { should == 'Sat, 02 Jun 2012 07:24:08 GMT' }
+        end
+      end
 
-  context '#create' do
-    let(:params) do
-      {
-        address: 'test@example.com',
-        code: 554,
-        error: 'Relay access denied',
-      }
-    end
+      context '#create' do
+        let(:params) do
+          {
+            address: 'test@example.com',
+            code: 554,
+            error: 'Relay access denied',
+          }
+        end
 
-    let(:response) { resource.create(params) }
+        let(:response) { resource.create(params) }
 
-    subject { response }
+        subject { response }
 
-    specify { should be_success }
+        specify { should be_success }
 
-    its(:status) { should == 200 }
-    its(:message) { should == 'Address has been added to the bounces table' }
+        its(:status) { should == 200 }
+        its(:message) {
+          should == 'Address has been added to the bounces table'
+        }
 
-    its(:address) { should == 'test@example.com' }
-  end
+        its(:address) { should == 'test@example.com' }
+      end
 
-  context '#remove' do
-    subject { response }
+      context '#remove' do
+        subject { response }
 
-    let(:response) { resource.remove('test@example.com') }
+        let(:response) { resource.remove('test@example.com') }
 
-    specify { should be_success }
+        specify { should be_success }
 
-    its(:status) { should == 200 }
-    its(:message) { should == 'Bounced address has been removed' }
+        its(:status) { should == 200 }
+        its(:message) { should == 'Bounced address has been removed' }
 
-    its(:address) { should == 'test@example.com' }
+        its(:address) { should == 'test@example.com' }
+      end
+    end
   end
 end