akamai_api 0.1.1 → 0.2.0

data/spec/features/ccu/status_request_spec.rb

@@ -0,0 +1,17 @@
+require 'spec_helper'
+
+describe 'Given I request the current status' do
+  context 'and given my login credentials are correct', vcr: { cassette_name: 'akamai_api_ccu_status/valid_credentials' } do
+    it 'I receive a successful response object' do
+      expect(AkamaiApi::Ccu.status).to be_a AkamaiApi::Ccu::Status::Response
+    end
+  end
+
+  context 'and given my login credentials are wrong', vcr: { cassette_name: 'akamai_api_ccu_status/invalid_credentials' } do
+    before { AkamaiApi::Ccu.stub auth: { username: 'foo', password: 'bar' } }
+
+    it 'an error is raised' do
+      expect { AkamaiApi::Ccu.status }.to raise_error AkamaiApi::Unauthorized
+    end
+  end
+end

data/spec/fixtures/ccu/successful_purge.json

@@ -0,0 +1 @@
+{"estimatedSeconds": 420, "progressUri": "/ccu/v2/purges/6a727459-cc60-11e3-bcb6-55367cb65536", "purgeId": "6a727459-cc60-11e3-bcb6-55367cb65536", "supportId": "17PY1398420021554891-311977056", "httpStatus": 201, "detail": "Request accepted.", "pingAfterSeconds": 420}

data/spec/lib/akamai_api/ccu/purge/request_spec.rb

@@ -0,0 +1,124 @@
+require 'spec_helper'
+
+describe AkamaiApi::Ccu::Purge::Request do
+  it "includes httparty" do
+    expect(subject.class.included_modules).to include HTTParty
+  end
+
+  it "sets a base_uri" do
+    expect(subject.class.base_uri).to eq 'https://api.ccu.akamai.com/ccu/v2/queues/default'
+  end
+
+  it "sets a content type header" do
+    expect(subject.class.headers).to eq 'Content-Type' => 'application/json'
+  end
+
+  describe "#action" do
+    it "is 'remove' by default" do
+      expect(subject.action).to eq 'remove'
+    end
+
+    it "can be changed" do
+      expect { subject.action = 'invalidate' }.to change(subject, :action).to 'invalidate'
+    end
+
+    it "raises an error if an invalid value is set" do
+      expect { subject.action = 'foobar' }.to raise_error AkamaiApi::Ccu::UnrecognizedOption
+    end
+  end
+
+  describe "#type" do
+    it "is 'arl' by default" do
+      expect(subject.type).to eq 'arl'
+    end
+
+    it "can be changed" do
+      expect { subject.type = 'cpcode' }.to change(subject, :type).to eq 'cpcode'
+    end
+
+    it "raises an error if an invalid value is set" do
+      expect { subject.type = 'foobar' }.to raise_error AkamaiApi::Ccu::UnrecognizedOption
+    end
+  end
+
+  describe "#domain" do
+    it "is 'production' by default" do
+      expect(subject.domain).to eq 'production'
+    end
+
+    it "can be changed" do
+      expect { subject.domain = 'staging' }.to change(subject, :domain).to eq 'staging'
+    end
+
+    it "raises an error if an invalid value is set" do
+      expect { subject.domain = 'foobar' }.to raise_error AkamaiApi::Ccu::UnrecognizedOption
+    end
+  end
+
+  describe "#execute" do
+    let(:fake_response) { double code: 201, parsed_response: {} }
+    let(:sample_arl) { 'http://www.foo.bar/t.txt' }
+
+    it "executes a post on the base url" do
+      expect(subject.class).to receive :post do |path, args|
+        expect(path).to eq '/'
+        fake_response
+      end
+      subject.execute sample_arl
+    end
+
+    it "sets the auth in the post" do
+      subject.stub auth: 'foo'
+      expect(subject.class).to receive :post do |path, args|
+        expect(args[:basic_auth]).to eq 'foo'
+        fake_response
+      end
+      subject.execute sample_arl
+    end
+
+    it "accepts a single element" do
+      expect(subject).to receive(:request_body).with([sample_arl])
+      subject.class.stub post: fake_response
+      subject.execute sample_arl
+    end
+
+    it "accepts a collection with a single element" do
+      expect(subject).to receive(:request_body).with([sample_arl])
+      subject.class.stub post: fake_response
+      subject.execute sample_arl
+    end
+
+    it "accepts a collection" do
+      expect(subject).to receive(:request_body).with(['a', 'b'])
+      subject.class.stub post: fake_response
+      subject.execute ['a', 'b']
+    end
+
+    it "works with splatting" do
+      expect(subject).to receive(:request_body).with(['a', 'b'])
+      subject.class.stub post: fake_response
+      subject.execute 'a', 'b'
+    end
+
+    it "sets the request body" do
+      expect(subject).to receive(:request_body).with([sample_arl]).and_return 'foo'
+      expect(subject.class).to receive :post do |path, args|
+        expect(args[:body]).to eq 'foo'
+        fake_response
+      end
+      subject.execute sample_arl
+    end
+
+    it "raises an exception when the response code is 401" do
+      fake_response = double code: 401
+      subject.class.stub post: fake_response
+      expect { subject.execute sample_arl }.to raise_error AkamaiApi::Unauthorized
+    end
+
+    it "returns a response built with the resulted json" do
+      fake_response = double code: 201, parsed_response: { a: 'b' }
+      subject.class.stub post: fake_response
+      expect(subject.execute(sample_arl).raw).to eq a: 'b'
+    end
+  end
+end

data/spec/lib/akamai_api/ccu/purge/response_spec.rb

@@ -0,0 +1,54 @@
+require 'spec_helper'
+
+describe AkamaiApi::Ccu::Purge::Response do
+  let(:raw) do
+    {
+      'describedBy' => 'foo',
+      'title' => 'bar',
+      'pingAfterSeconds' => 100,
+      'purgeId' => '120',
+      'supportId' => '130',
+      'detail' => 'baz',
+      'httpStatus' => 201,
+      'estimatedSeconds' => 90,
+      'progressUri' => 'http://asd.com'
+    }
+  end
+  subject { AkamaiApi::Ccu::Purge::Response.new raw }
+
+  it '#described_by returns the describedBy attribute' do
+    expect(subject.described_by).to eq 'foo'
+  end
+
+  it '#title returns the title attribute' do
+    expect(subject.title).to eq 'bar'
+  end
+
+  it '#time_to_wait returns the pingAfterSeconds attribute' do
+    expect(subject.time_to_wait).to eq 100
+  end
+
+  it '#purge_id returns the purgeId attribute' do
+    expect(subject.purge_id).to eq '120'
+  end
+
+  it '#support_id returns the supportId attribute' do
+    expect(subject.support_id).to eq '130'
+  end
+
+  it '#message returns the detail attribute' do
+    expect(subject.message).to eq 'baz'
+  end
+
+  it '#code returns the httpStatus attribute' do
+    expect(subject.code).to eq 201
+  end
+
+  it '#estimated_time returns the estimatedSeconds attribute' do
+    expect(subject.estimated_time).to eq 90
+  end
+
+  it '#uri returns the progressUri attribute' do
+    expect(subject.uri).to eq 'http://asd.com'
+  end
+end

data/spec/lib/akamai_api/ccu/purge_status/not_found_response_spec.rb

@@ -0,0 +1,41 @@
+require 'spec_helper'
+
+describe AkamaiApi::Ccu::PurgeStatus::NotFoundResponse do
+  let(:raw) do
+    {
+      'progressUri'              => '/ccu/v2/purges/12345678-1234-5678-1234-123456789012',
+      'purgeId'                  => '12345678-1234-5678-1234-123456789012',
+      'supportId'                => '12345678901234567890-123456789',
+      'httpStatus'               => 200,
+      'purgeStatus'              => 'Unknown',
+      'pingAfterSeconds'         => 60,
+      'detail'                   => 'foobarbaz'
+    }
+  end
+  subject { AkamaiApi::Ccu::PurgeStatus::NotFoundResponse.new raw }
+
+  it '#progress_uri returns progressUri attribute' do
+    expect(subject.progress_uri).to eq '/ccu/v2/purges/12345678-1234-5678-1234-123456789012'
+  end
+
+  it '#purge_id returns purgeId attribute' do
+    expect(subject.purge_id).to eq '12345678-1234-5678-1234-123456789012'
+  end
+
+  it '#support_id returns supportId attribute' do
+    expect(subject.support_id).to eq '12345678901234567890-123456789'
+  end
+
+  it '#code returns httpStatus attribute' do
+    expect(subject.code).to eq 200
+  end
+
+  it '#status returns purgeStatus attribute' do
+    expect(subject.status).to eq 'Unknown'
+  end
+
+  it '#message returns detail attribute' do
+    raw['detail'] = 'asd'
+    expect(subject.message).to eq 'asd'
+  end
+end

data/spec/lib/akamai_api/ccu/purge_status/request_spec.rb

@@ -0,0 +1,64 @@
+require 'spec_helper'
+
+describe AkamaiApi::Ccu::PurgeStatus::Request do
+  it "includes httparty" do
+    expect(subject.class.included_modules).to include HTTParty
+  end
+
+  it "sets a base_uri" do
+    expect(subject.class.base_uri).to eq 'https://api.ccu.akamai.com'
+  end
+
+  describe "#execute" do
+    let(:fake_response) { double code: 200, parsed_response: {} }
+
+    it "executes a GET request on the given progress_uri" do
+      expect(AkamaiApi::Ccu::PurgeStatus::Request).to receive :get do |uri, args|
+        expect(uri).to eq '/ccu/v2/purges/foo'
+        fake_response
+      end
+      subject.execute '/ccu/v2/purges/foo'
+    end
+
+    it "sets the auth in the request" do
+      AkamaiApi::Ccu.stub auth: 'foo'
+      expect(AkamaiApi::Ccu::PurgeStatus::Request).to receive :get do |uri, args|
+        expect(args).to eq basic_auth: 'foo'
+        fake_response
+      end
+      subject.execute '/ccu/v2/purges/foo'
+    end
+
+    it "raises an exception when the response code is 401" do
+      fake_response.stub code: 401
+      expect(AkamaiApi::Ccu::PurgeStatus::Request).to receive(:get).and_return fake_response
+      expect { subject.execute '/ccu/v2/purges/foo' }.to raise_error AkamaiApi::Unauthorized
+    end
+
+    it "returns a response built with the received json" do
+      fake_response.stub parsed_response: {a: 'b'}
+      expect(AkamaiApi::Ccu::PurgeStatus::Request).to receive(:get).and_return fake_response
+      expect(subject.execute '/ccu/v2/purges/foo' ).to be_a AkamaiApi::Ccu::PurgeStatus::Response
+    end
+
+    context "when the given progressUri is the purgeId" do
+      it "the progressUri prefix is appended" do
+        expect(AkamaiApi::Ccu::PurgeStatus::Request).to receive :get do |uri, args|
+          expect(uri).to eq '/ccu/v2/purges/foo'
+          fake_response
+        end
+        subject.execute 'foo'
+      end
+    end
+
+    context "when the given progressUri isn't prefixed by a slash" do
+      it "a slash is prefixed" do
+        expect(AkamaiApi::Ccu::PurgeStatus::Request).to receive :get do |uri, args|
+          expect(uri).to eq '/ccu/v2/purges/foo'
+          fake_response
+        end
+        subject.execute 'ccu/v2/purges/foo'
+      end
+    end
+  end
+end

data/spec/lib/akamai_api/ccu/purge_status/successful_response_spec.rb

@@ -0,0 +1,86 @@
+require 'spec_helper'
+
+describe AkamaiApi::Ccu::PurgeStatus::SuccessfulResponse do
+  let(:raw) do
+    {
+      'originalEstimatedSeconds' => 480,
+      'progressUri'              => '/ccu/v2/purges/12345678-1234-5678-1234-123456789012',
+      'originalQueueLength'      => 6,
+      'purgeId'                  => '12345678-1234-5678-1234-123456789012',
+      'supportId'                => '12345678901234567890-123456789',
+      'httpStatus'               => 200,
+      'completionTime'           => '2014-02-19T22:16:20Z',
+      'submittedBy'              => 'test1',
+      'purgeStatus'              => 'In-Progress',
+      'submissionTime'           => '2014-02-19T21:16:20Z',
+      'pingAfterSeconds'         => 60
+    }
+  end
+  subject { AkamaiApi::Ccu::PurgeStatus::SuccessfulResponse.new raw }
+
+  it '#estimated_time returns estimatedSeconds attribute' do
+    expect(subject.estimated_time).to eq 480
+  end
+
+  it '#progress_uri returns progressUri attribute' do
+    expect(subject.progress_uri).to eq '/ccu/v2/purges/12345678-1234-5678-1234-123456789012'
+  end
+
+  it '#queue_length returns originalQueueLength attribute' do
+    expect(subject.queue_length).to eq 6
+  end
+
+  it '#purge_id returns purgeId attribute' do
+    expect(subject.purge_id).to eq '12345678-1234-5678-1234-123456789012'
+  end
+
+  it '#support_id returns supportId attribute' do
+    expect(subject.support_id).to eq '12345678901234567890-123456789'
+  end
+
+  it '#code returns httpStatus attribute' do
+    expect(subject.code).to eq 200
+  end
+
+  describe '#completed_at' do
+    it 'returns a Time object' do
+      expect(subject.completed_at).to be_a Time
+    end
+
+    it 'returns completionTime attribute' do
+      expect(subject.completed_at.iso8601).to eq '2014-02-19T22:16:20Z'
+    end
+
+    it 'can return nil' do
+      raw['completionTime'] = nil
+      expect(subject.completed_at).to be_nil
+    end
+  end
+
+  it '#submitted_by returns submittedBy attribute' do
+    expect(subject.submitted_by).to eq 'test1'
+  end
+
+  it '#status returns purgeStatus attribute' do
+    expect(subject.status).to eq 'In-Progress'
+  end
+
+  describe '#submitted_at' do
+    it 'returns a Time object' do
+      expect(subject.submitted_at).to be_a Time
+    end
+
+    it 'can return nil' do
+      raw['submissionTime'] = nil
+      expect(subject.submitted_at).to be_nil
+    end
+
+    it 'returns submissionTime attribute' do
+      expect(subject.submitted_at.iso8601).to eq '2014-02-19T21:16:20Z'
+    end
+  end
+
+  it '#time_to_wait returns pingAfterSeconds attribute' do
+    expect(subject.time_to_wait).to eq 60
+  end
+end

data/spec/lib/akamai_api/ccu/response_spec.rb

@@ -0,0 +1,15 @@
+require 'spec_helper'
+
+describe AkamaiApi::Ccu::Response do
+  subject { AkamaiApi::Ccu::Response.new 'supportId' => 'foo', 'httpStatus' => 201 }
+
+  it '#support_id returns the supportId attribute' do
+    expect(subject.support_id).to eq 'foo'
+  end
+
+  %w(code http_status).each do |reader|
+    it "##{reader} returns the httpStatus attribute" do
+      expect(subject.send reader).to eq 201
+    end
+  end
+end

data/spec/lib/akamai_api/ccu/status/request_spec.rb

@@ -0,0 +1,22 @@
+require 'spec_helper'
+
+describe AkamaiApi::Ccu::Status::Request do
+  it 'executes a get upon base url passing the basic auth' do
+    fake_response = double code: 200, parsed_response: {}
+    AkamaiApi::Ccu.stub auth: 'foo'
+    expect(AkamaiApi::Ccu::Status::Request).to receive(:get).with('/', basic_auth: 'foo').and_return fake_response
+    subject.execute
+  end
+
+  it 'raises an unauthorized error if code is 401' do
+    fake_response = double code: 401, parsed_response: {}
+    expect(AkamaiApi::Ccu::Status::Request).to receive(:get).and_return fake_response
+    expect { subject.execute }.to raise_error AkamaiApi::Unauthorized
+  end
+
+  it 'returns a StatusResponse' do
+    fake_response = double code: 200, parsed_response: {}
+    expect(AkamaiApi::Ccu::Status::Request).to receive(:get).and_return fake_response
+    expect(subject.execute).to be_a AkamaiApi::Ccu::Status::Response
+  end
+end

data/spec/lib/akamai_api/ccu/status/response_spec.rb

@@ -0,0 +1,29 @@
+require 'spec_helper'
+
+describe AkamaiApi::Ccu::Status::Response do
+  let(:raw) do
+    {
+      'httpStatus' => 201,
+      'queueLength' => 1,
+      'detail' => 'asd',
+      'supportId' => '21'
+    }
+  end
+  subject { AkamaiApi::Ccu::Status::Response.new raw }
+
+  it '#code returns the httpStatus property' do
+    expect(subject.code).to eq 201
+  end
+
+  it '#queue_length returns the queueLength property' do
+    expect(subject.queue_length).to eq 1
+  end
+
+  it '#message returns the detail property' do
+    expect(subject.message).to eq 'asd'
+  end
+
+  it '#support_id returns the supportId property' do
+    expect(subject.support_id).to eq '21'
+  end
+end

data/spec/lib/akamai_api/ccu_spec.rb

@@ -1,87 +1,75 @@
 require 'spec_helper'
 
-module AkamaiApi
-  describe Ccu do
-    include Savon::SpecHelper
+describe AkamaiApi::Ccu do
+  subject { AkamaiApi::Ccu }
 
-    before(:all) { savon.mock! }
-    after(:all)  { savon.unmock! }
-
-    let(:fixture) { File.read 'spec/fixtures/ccu/successful_purge.xml' }
-
-    def soap_body method, *uris
-      method_parts = method.split('_')
-      options = ["action=#{method_parts.first}", "type=#{method_parts.last}"]
-      SoapBody.new do
-        string :name,    AkamaiApi.config[:auth].first
-        string :pwd,     AkamaiApi.config[:auth].last
-        string :network, ''
-        array  :opt,     options
-        array  :uri,     uris
-      end
-    end
-
-    shared_examples 'purge helper' do
-      before do
-        body = soap_body(method, '12345').to_s
-        savon.expects(:purge_request).with(:message => body).returns(fixture)
+  %w(invalidate remove).each do |action|
+    describe "##{action}" do
+      it 'raises error if less than 2 arguments are given' do
+        expect { subject.send action, 'foo' }.to raise_error ArgumentError
       end
 
-      it 'responds with a CcuResponse object' do
-        Ccu.send(method, '12345').should be_a CcuResponse
+      it 'delegates to #purge with the given type' do
+        expect(subject).to receive(:purge).with(action.to_sym, :cpcode, 'foo', domain: 'asd').and_return 'bar'
+        expect(subject.send action, :cpcode, 'foo', domain: 'asd').to eq 'bar'
       end
+    end
 
-      context 'when data are correct' do
-        it 'returns the expected response code' do
-          Ccu.send(method, '12345').code.should == 100
-        end
-
-        it 'returns a successful message' do
-          Ccu.send(method, '12345').message.should == 'Success.'
-        end
+    %w(arl cpcode).each do |type|
+      describe "##{action}_#{type}" do
+        let(:method) { "#{action}_#{type}" }
 
-        it 'returns a unique session id' do
-          Ccu.send(method, '12345').session_id.should == '97870328-018c-11e2-aabc-489beabc489b'
+        it 'raises error if less than 1 argument is given' do
+          expect { subject.send method }.to raise_error ArgumentError
         end
 
-        it 'returns the estimated time in seconds' do
-          Ccu.send(method, '12345').estimated_time.should == 420
+        it 'delegates to #purge with the given items' do
+          expect(subject).to receive(:purge).with(action.to_sym, type.to_sym, 'foo', domain: 'asd').and_return 'bar'
+          expect(subject.send method, 'foo', domain: 'asd').to eq 'bar'
         end
       end
     end
+  end
 
-    context '#invalidate_arl' do
-      let(:method) { 'invalidate_arl' }
-      it_should_behave_like 'purge helper'
-    end
-
-    context '#invalidate_cpcode' do
-      let(:method) { 'invalidate_cpcode' }
-      it_should_behave_like 'purge helper'
-    end
-
-    context '#remove_arl' do
-      let(:method) { 'remove_arl' }
-      it_should_behave_like 'purge helper'
-    end
-
-    context '#remove_cpcode' do
-      let(:method) { 'remove_cpcode' }
-      it_should_behave_like 'purge helper'
+  describe '#purge' do
+    it 'delegates to Purge::Request' do
+      fake_request = double
+      expect(fake_request).to receive(:execute).with('baz').and_return 'quiz'
+      expect(AkamaiApi::Ccu::Purge::Request).to receive(:new).with('foo', 'bar', domain: 'asd').
+                                               and_return fake_request
+      expect(subject.purge 'foo', 'bar', 'baz', domain: 'asd').to eq 'quiz'
     end
 
-    describe '#purge raises an error when' do
+    describe 'raises an error when' do
       it 'action is not allowed' do
-        expect { Ccu.purge :sss, :cpcode, '12345' }.to raise_error Ccu::UnrecognizedOption
+        expect { subject.purge :sss, :cpcode, '12345' }.to raise_error AkamaiApi::Ccu::UnrecognizedOption
       end
 
       it 'type is not allowed' do
-        expect { Ccu.purge :remove, :foo, '12345' }.to raise_error Ccu::UnrecognizedOption
+        expect { subject.purge :remove, :foo, '12345' }.to raise_error AkamaiApi::Ccu::UnrecognizedOption
       end
 
       it 'domain is specified and not allowed' do
-        expect { Ccu.purge :remove, :arl, 'foo', :domain => :foo }.to raise_error
+        expect { subject.purge :remove, :arl, 'foo', :domain => :foo }.to raise_error
+      end
+    end
+  end
+
+  describe '#status' do
+    context "when no argument is given" do
+      it "delegates to Status request instance" do
+        expect(AkamaiApi::Ccu::Status::Request).to receive(:new).and_return double execute: 'foo'
+        expect(subject.status).to eq 'foo'
+      end
+    end
+
+    context "when a progress uri is passed as argument" do
+      it "delegates to PurgeStatus request instance" do
+        fake_instance = double
+        expect(AkamaiApi::Ccu::PurgeStatus::Request).to receive(:new).and_return fake_instance
+        expect(fake_instance).to receive(:execute).with('foo').and_return 'asd'
+        expect(subject.status 'foo').to eq 'asd'
       end
     end
   end
-end
+end

data/spec/lib/akamai_api/cli/ccu/status_renderer_spec.rb

@@ -0,0 +1,36 @@
+require 'spec_helper'
+
+module AkamaiApi
+  describe Cli::Ccu::StatusRenderer do
+    subject { Cli::Ccu::StatusRenderer.new Ccu::Status::Response.new({}) }
+
+    describe '#render' do
+      it 'returns a string' do
+        expect(subject.render).to be_a String
+      end
+
+      it 'delegates to #render_status if response is a Status::Response' do
+        expect(subject).to receive(:render_status)
+        subject.render
+      end
+
+      it 'delegates to #render_purge_status if response is a PurgeStatus::Response' do
+        subject = Cli::Ccu::StatusRenderer.new Ccu::PurgeStatus::SuccessfulResponse.new({})
+        expect(subject).to receive(:render_purge_status)
+        subject.render
+      end
+    end
+
+    it '#render_purge_status delegates to #render_successful_purge_status for success response' do
+      subject = Cli::Ccu::StatusRenderer.new Ccu::PurgeStatus::SuccessfulResponse.new({})
+      expect(subject).to receive :render_successful_purge_status
+      subject.render_purge_status
+    end
+
+    it '#render_purge_status delegates to #render_not_found_purge_status for not foundresponse' do
+      subject = Cli::Ccu::StatusRenderer.new Ccu::PurgeStatus::NotFoundResponse.new({})
+      expect(subject).to receive :render_not_found_purge_status
+      subject.render_purge_status
+    end
+  end
+end

data/spec/lib/akamai_api/eccu_request_spec.rb

@@ -52,7 +52,7 @@ module AkamaiApi
 
     describe '::last' do
       it 'find the most recent entry' do
-        EccuRequest.stub! :all_ids => %w(a b)
+        EccuRequest.stub :all_ids => %w(a b)
         EccuRequest.should_receive(:find).with('b', anything()).and_return :a
         EccuRequest.last.should == :a
       end
@@ -60,7 +60,7 @@ module AkamaiApi
 
     describe '::first' do
       it 'find the oldest entry' do
-        EccuRequest.stub! :all_ids => %w(a b)
+        EccuRequest.stub :all_ids => %w(a b)
         EccuRequest.should_receive(:find).with('a', anything()).and_return :a
         EccuRequest.first.should == :a
       end
@@ -68,13 +68,13 @@ module AkamaiApi
 
     describe '::all' do
       it 'returns the detail for each enlisted id' do
-        EccuRequest.stub! :all_ids => %w(a b)
+        EccuRequest.stub :all_ids => %w(a b)
         EccuRequest.should_receive(:find).with('a', anything()).and_return(:a)
         EccuRequest.should_receive(:find).with('b', anything()).and_return(:b)
         EccuRequest.all.should =~ [:a, :b]
       end
       it 'returns the detail for each enlisted id even if only one id is returned' do
-        EccuRequest.stub! :all_ids => "a"
+        EccuRequest.stub :all_ids => "a"
         EccuRequest.should_receive(:find).with('a', anything()).and_return(:a)
         EccuRequest.all.should =~ [:a]
       end
@@ -256,4 +256,4 @@ module AkamaiApi
       end
     end
   end
-end
+end