azure 0.7.0.pre → 0.7.0.pre2

data/test/unit/cloud_service_management/cloud_service_management_service_test.rb

@@ -34,7 +34,7 @@ describe Azure::CloudServiceManagementService do
 
   describe "#list_cloud_services" do
     before do
-      Azure::BaseManagement::ManagementHttpRequest.stubs(:new).with(verb, request_path, nil).returns(mock_request)
+      Azure::BaseManagement::ManagementHttpRequest.stubs(:new).with(verb, request_path, anything).returns(mock_request)
       mock_request.expects(:call).returns(response_body)
     end
 
@@ -57,7 +57,7 @@ describe Azure::CloudServiceManagementService do
 
   describe "#get_cloud_service" do
     before do
-      Azure::BaseManagement::ManagementHttpRequest.stubs(:new).with(verb, request_path, nil).returns(mock_request)
+      Azure::BaseManagement::ManagementHttpRequest.stubs(:new).with(verb, request_path, anything).returns(mock_request)
       mock_request.expects(:call).returns(response_body)
     end
 

data/test/unit/config/azure_test.rb

@@ -0,0 +1,112 @@
+#-------------------------------------------------------------------------
+# # Copyright (c) Microsoft and contributors. All rights reserved.
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#   http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+#--------------------------------------------------------------------------
+require 'test_helper'
+
+describe Azure do
+
+  describe 'default configuration for the azure module' do
+    subject { Azure }
+
+    it 'should provide itself as the base configuration through the config method' do
+      subject.config.must_equal Azure
+    end
+
+    it 'should provide access to the configuration through the config method' do
+      subject.config.management_endpoint.must_equal Azure.management_endpoint
+    end
+
+    it 'should start with the default management endpoint' do
+      Azure.management_endpoint.must_equal Azure::Default.management_endpoint
+    end
+
+    describe 'configuration of management certificates with files and strings' do
+      let(:private_key) {Azure.http_private_key}
+      let(:public_key) {Azure.http_certificate_key}
+
+      describe 'using a pfx' do
+
+        describe 'loaded as a string' do
+          before do
+            Azure.management_certificate = File.binread(Fixtures['management_certificate.pfx'])
+          end
+
+          it 'should populate the private key' do
+            refute_nil(private_key)
+            private_key.must_be_kind_of(OpenSSL::PKey::RSA)
+          end
+
+          it 'should populate the public key' do
+            refute_nil(public_key)
+            public_key.must_be_kind_of(OpenSSL::X509::Certificate)
+          end
+
+        end
+
+        describe 'loaded as a file' do
+          before do
+            Azure.management_certificate = Fixtures['management_certificate.pfx']
+          end
+
+          it 'should populate the private key' do
+            refute_nil(private_key)
+            private_key.must_be_kind_of(OpenSSL::PKey::RSA)
+          end
+
+          it 'should populate the public key' do
+            refute_nil(public_key)
+            public_key.must_be_kind_of(OpenSSL::X509::Certificate)
+          end
+        end
+      end
+
+      describe 'using a pem' do
+
+        describe 'loaded as a string' do
+          before do
+            Azure.management_certificate = File.read(Fixtures['management_certificate.pem'])
+          end
+
+          it 'should populate the private key' do
+            refute_nil(private_key)
+            private_key.must_be_kind_of(OpenSSL::PKey::RSA)
+          end
+
+          it 'should populate the public key' do
+            refute_nil(public_key)
+            public_key.must_be_kind_of(OpenSSL::X509::Certificate)
+          end
+
+        end
+
+        describe 'loaded as a file' do
+          before do
+            Azure.management_certificate = Fixtures['management_certificate.pem']
+          end
+
+          it 'should populate the private key' do
+            refute_nil(private_key)
+            private_key.must_be_kind_of(OpenSSL::PKey::RSA)
+          end
+
+          it 'should populate the public key' do
+            refute_nil(public_key)
+            public_key.must_be_kind_of(OpenSSL::X509::Certificate)
+          end
+        end
+      end
+    end
+  end
+
+end

data/test/unit/config/client_test.rb

@@ -0,0 +1,37 @@
+#-------------------------------------------------------------------------
+# # Copyright (c) Microsoft and contributors. All rights reserved.
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#   http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+#--------------------------------------------------------------------------
+require 'test_helper'
+
+describe Azure::Client do
+
+  describe 'default configuration for a client instance' do
+    subject { Azure.client }
+
+    Azure::Configurable.keys.each do |key|
+      it "should have the same value as its parent for #{key}" do
+        subject.send(key.to_sym).must_equal Azure.send(key.to_sym)
+      end
+
+      unless key == :management_certificate ## TODO: generate another pem and pfx for file / string tests then remove this
+        it "should have a different value as its parent after changing the client for #{key}" do
+          subject.send("#{key}=".to_sym, 'blah')
+          subject.send(key.to_sym).wont_equal Azure.send(key.to_sym)
+        end
+      end
+    end
+
+  end
+
+end

data/test/unit/core/auth/shared_key_lite_test.rb

@@ -12,37 +12,37 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 #--------------------------------------------------------------------------
-require "test_helper"
-require "azure/core/auth/shared_key_lite"
+require 'test_helper'
+require 'azure/core/auth/shared_key_lite'
 
 describe Azure::Core::Auth::SharedKeyLite do
-  subject { Azure::Core::Auth::SharedKeyLite.new "account-name", "YWNjZXNzLWtleQ==" }
+  subject { Azure::Core::Auth::SharedKeyLite.new 'account-name', 'YWNjZXNzLWtleQ==' }
   
-  let(:verb) { "POST" }
-  let(:uri) { URI.parse "http://dummy.uri/resource" }
+  let(:verb) { 'POST' }
+  let(:uri) { URI.parse 'http://dummy.uri/resource' }
   let(:headers) do
     {
-      "Content-MD5"=> "foo",
-      "Content-Type"=> "foo",
-      "Date"=> "foo"
+      'Content-MD5' => 'foo',
+      'Content-Type' => 'foo',
+      'Date' => 'foo'
     }
   end
   let(:headers_without_date) { 
     headers_without_date = headers.clone
-    headers_without_date.delete "Date"
+    headers_without_date.delete 'Date'
     headers_without_date
   }
   
-  describe "sign" do
-    it "creates a signature from the provided HTTP method, uri, and reduced set of standard headers" do
-      subject.sign(verb, uri, headers).must_equal "account-name:vVFnj/+27JFABZgpt5H8g/JVU2HuWFnjv5aeUIxQvBE="
+  describe 'sign' do
+    it 'creates a signature from the provided HTTP method, uri, and reduced set of standard headers' do
+      subject.sign(verb, uri, headers).must_equal 'account-name:vVFnj/+27JFABZgpt5H8g/JVU2HuWFnjv5aeUIxQvBE='
     end
 
-    it "ignores standard headers other than Content-MD5, Content-Type, and Date" do
-      subject.sign(verb, uri, headers.merge({"Content-Encoding"=> "foo"})).must_equal "account-name:vVFnj/+27JFABZgpt5H8g/JVU2HuWFnjv5aeUIxQvBE="
+    it 'ignores standard headers other than Content-MD5, Content-Type, and Date' do
+      subject.sign(verb, uri, headers.merge({'Content-Encoding' => 'foo'})).must_equal 'account-name:vVFnj/+27JFABZgpt5H8g/JVU2HuWFnjv5aeUIxQvBE='
     end
 
-    it "throws IndexError when there is no Date header" do
+    it 'throws IndexError when there is no Date header' do
       assert_raises IndexError do
         subject.sign(verb, uri, headers_without_date)
       end

data/test/unit/core/auth/shared_key_test.rb

@@ -12,47 +12,47 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 #--------------------------------------------------------------------------
-require "test_helper"
-require "azure/core/auth/shared_key"
+require 'test_helper'
+require 'azure/core/auth/shared_key'
 
 describe Azure::Core::Auth::SharedKey do
-  subject { Azure::Core::Auth::SharedKey.new "account-name", "YWNjZXNzLWtleQ==" }
+  subject { Azure::Core::Auth::SharedKey.new 'account-name', 'YWNjZXNzLWtleQ==' }
   
-  let(:verb) { "POST" }
-  let(:uri) { URI.parse "http://dummy.uri/resource" }
+  let(:verb) { 'POST' }
+  let(:uri) { URI.parse 'http://dummy.uri/resource' }
   let(:headers) do
     {
-      "Content-Encoding"=> "foo",
-      "Content-Language"=> "foo",
-      "Content-Length"=> "foo",
-      "Content-MD5"=> "foo",
-      "Content-Type"=> "foo",
-      "Date"=> "foo",
-      "If-Modified-Since"=> "foo",
-      "If-Match"=> "foo",
-      "If-None-Match"=> "foo",
-      "If-Unmodified-Since"=> "foo",
-      "Range"=> "foo", 
-      "x-ms-ImATeapot"=> "teapot",
-      "x-ms-ShortAndStout"=>"True"
+      'Content-Encoding' => 'foo',
+      'Content-Language' => 'foo',
+      'Content-Length' => 'foo',
+      'Content-MD5' => 'foo',
+      'Content-Type' => 'foo',
+      'Date' => 'foo',
+      'If-Modified-Since' => 'foo',
+      'If-Match' => 'foo',
+      'If-None-Match' => 'foo',
+      'If-Unmodified-Since' => 'foo',
+      'Range' => 'foo',
+      'x-ms-ImATeapot' => 'teapot',
+      'x-ms-ShortAndStout' => 'True'
     }
   end
 
-  describe "sign" do
-    it "creates a signature from the provided HTTP method, uri, and a specific set of standard headers" do
-      subject.sign(verb, uri, headers).must_equal "account-name:vcdxlDVoE1QvJerkg0ci3Wlnj2Qq8yzlsrkRf5dEU/I="
+  describe 'sign' do
+    it 'creates a signature from the provided HTTP method, uri, and a specific set of standard headers' do
+      subject.sign(verb, uri, headers).must_equal 'account-name:vcdxlDVoE1QvJerkg0ci3Wlnj2Qq8yzlsrkRf5dEU/I='
     end
   end
   
-  describe "canonicalized_headers" do
-    it "creates a canonicalized header string" do
+  describe 'canonicalized_headers' do
+    it 'creates a canonicalized header string' do
       subject.canonicalized_headers(headers).must_equal "x-ms-imateapot:teapot\nx-ms-shortandstout:True"
     end
   end
 
-  describe "canonicalized_resource" do
-    it "creates a canonicalized resource string" do
-      subject.canonicalized_resource(uri).must_equal "/account-name/resource"
+  describe 'canonicalized_resource' do
+    it 'creates a canonicalized resource string' do
+      subject.canonicalized_resource(uri).must_equal '/account-name/resource'
     end
   end
 end

data/test/unit/core/http/http_error_test.rb

@@ -12,23 +12,23 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 #--------------------------------------------------------------------------
-require "test_helper"
-require "azure/core/http/http_error"
+require 'test_helper'
+require 'azure/core/http/http_error'
 
 describe Azure::Core::Http::HTTPError do
   let :http_response do
-    stub(:body => Fixtures[:http_error], :status_code => 409, :uri => 'http://dummy.uri')
+    stub(body: Fixtures[:http_error], status_code: 409, uri: 'http://dummy.uri', headers: {})
   end
 
   subject do
     Azure::Core::Http::HTTPError.new(http_response)
   end
 
-  it "is an instance of Azure::Core::Error" do
+  it 'is an instance of Azure::Core::Error' do
     subject.must_be_kind_of Azure::Core::Error
   end
 
-  it "lets us see the original uri" do
+  it 'lets us see the original uri' do
     subject.uri.must_equal 'http://dummy.uri'
   end
 
@@ -37,26 +37,25 @@ describe Azure::Core::Http::HTTPError do
   end
 
   it "lets us see the error's type" do
-    subject.type.must_equal "TableAlreadyExists"
+    subject.type.must_equal 'TableAlreadyExists'
   end
 
   it "lets us see the error's description" do
-    subject.description.must_equal "The table specified already exists."
+    subject.description.must_equal 'The table specified already exists.'
   end
 
-  it "generates an error message that wraps both the type and description" do
-    subject.message.must_equal "TableAlreadyExists (409): The table specified already exists."
+  it 'generates an error message that wraps both the type and description' do
+    subject.message.must_equal 'TableAlreadyExists (409): The table specified already exists.'
   end
 
   describe 'with invalid http_response body' do
-
     let :http_response do
-      stub(:body => "\r\nInvalid request\r\n", :status_code => 409, :uri => 'http://dummy.uri')
+      stub(:body => "\r\nInvalid request\r\n", :status_code => 409, :uri => 'http://dummy.uri', headers: {})
     end
 
-    it "sets the type to unknown if the response body is not an XML" do
-      subject.type.must_equal "Unknown"
-      subject.description.must_equal "Invalid request"
+    it 'sets the type to unknown if the response body is not an XML' do
+      subject.type.must_equal 'Unknown'
+      subject.description.must_equal 'Invalid request'
     end
   end
 end

data/test/unit/core/http/http_request_test.rb

@@ -16,9 +16,11 @@ require 'test_helper'
 require 'azure/core/http/http_request'
 
 describe Azure::Core::Http::HttpRequest do
+  let(:uri) { URI('http://example.com') }
+
   describe ' default_headers ' do
-    subject do 
-      Azure::Core::Http::HttpRequest.new(:get, URI('/'), nil, 'Thu, 04 Oct 2012 06:38:27 GMT')
+    subject do
+      Azure::Core::Http::HttpRequest.new(:get, uri, body: nil, current_time: 'Thu, 04 Oct 2012 06:38:27 GMT')
     end
 
     it 'sets the x-ms-date header to the current_time' do
@@ -36,77 +38,100 @@ describe Azure::Core::Http::HttpRequest do
     it 'sets the MaxDataServiceVersion header to the current max` API version' do
       subject.headers['MaxDataServiceVersion'] = '2.0;NetFx'
     end
+  end
 
-    let :http do
-      mock = Minitest::Mock.new
-      mock.expect(:use_ssl=, true, [true])
-      mock.expect(:verify_mode=, OpenSSL::SSL::VERIFY_PEER, [1])
-      mock.expect(:request, Net::HTTPResponse.new(1.1, 200, '<body/>'), [Net::HTTP::Get])
-      mock
+  describe 'when passed custom headers' do
+    subject do
+      Azure::Core::Http::HttpRequest.new(:get,
+                                         uri,
+                                         body: nil,
+                                         headers: {
+                                             'blah' => 'something',
+                                             'x-ms-version' => '123'
+                                         })
     end
 
-    describe 'when not using a ca_file' do
-      subject do
-        Azure::Core::Http::HttpRequest.new(:get, URI('https://manage.windowsazure.com'), nil, 'Thu, 04 Oct 2012 06:38:27 GMT')
-      end
+    it 'should have overridden the value of x-ms-version' do
+      subject.headers['x-ms-version'].must_equal '123'
+    end
 
-      it 'should set the ca_file on the http request' do
-        Net::HTTP.stub(:new, http) do
-          subject.call
-        end
-        http.verify
-      end
+    it 'should have added in the blah = something header' do
+      subject.headers['blah'].must_equal 'something'
     end
 
-    describe 'when using a ca_file' do
-      before do
-        Azure.config.ca_file = './blah.pem'
-      end
+  end
 
-      after do
-        Azure.config.ca_file = nil
-      end
+  describe ' when passed a body ' do
+    subject do
+      Azure::Core::Http::HttpRequest.new(:post, uri, body: '<body/>')
+    end
 
-      subject do
-        Azure::Core::Http::HttpRequest.new(:get, URI('https://manage.windowsazure.com'), nil, 'Thu, 04 Oct 2012 06:38:27 GMT')
-      end
+    it 'sets the default Content-Type header' do
+      subject.headers['Content-Type'].must_equal 'application/atom+xml; charset=utf-8'
+    end
 
-      let :ca_http do
-        http.expect(:ca_file=, Azure.config.ca_file, [String])
-      end
+    it 'sets the Content-Length header' do
+      subject.headers['Content-Length'].must_equal '7'
+    end
 
-      it 'should set the ca_file on the http request' do
-        Net::HTTP.stub(:new, ca_http) do
-          subject.call
-        end
-        ca_http.verify
-      end
+    it 'sets the Content-MD5 header to a Base64 encoded representation of the MD5 hash of the body' do
+      subject.headers['Content-MD5'].must_equal 'PNeJy7qyzV4XUoBBHkVu0g=='
+    end
+  end
 
+  describe ' when the body is nil ' do
+    subject do
+      Azure::Core::Http::HttpRequest.new(:get, uri)
     end
 
-    describe ' when passed a body ' do
-      subject do 
-        Azure::Core::Http::HttpRequest.new(:get, URI('/'), '<body/>')
-      end
+    it 'leaves the Content-Type, Content-Length, and Content-MD5 headers blank' do
+      subject.headers['Content-Length'].must_equal '0'
+      subject.headers['Content-MD5'].must_be_nil
+    end
+  end
 
-      it 'sets the default Content-Type header' do
-        subject.headers['Content-Type'].must_equal 'application/atom+xml; charset=utf-8'
-      end
+  describe '#call' do
+
+    let(:mock_conn) do
+      conn = mock
+      conn.expects(:run_request, [uri, nil, nil]).returns(mock_res)
+      conn
+    end
+
+    subject do
+      sub = Azure::Core::Http::HttpRequest.new(:get, uri)
+      sub.expects(:http_setup).returns(mock_conn)
+      sub
+    end
+
+    describe 'on success' do
+      let(:body) { '</body>' }
 
-      it 'sets the Content-Length header' do
-        subject.headers['Content-Length'].must_equal '7'
+      let(:mock_res) do
+        res = mock
+        res.expects(:success?).returns(true)
+        res.expects(:body).returns(body)
+        res
       end
 
-      it 'sets the Content-MD5 header to a Base64 encoded representation of the MD5 hash of the body' do
-        subject.headers['Content-MD5'].must_equal 'PNeJy7qyzV4XUoBBHkVu0g=='
+      it 'should return a response' do
+        subject.call.body.must_equal(body)
       end
     end
 
-    describe ' when the body is nil ' do
-      it 'leaves the Content-Type, Content-Length, and Content-MD5 headers blank' do
-        subject.headers['Content-Type'].must_equal ''
-        subject.headers['Content-Length'].must_equal '0'
-        subject.headers['Content-MD5'].must_be_nil
+    describe 'on failure' do
+      let(:body) { 'OH NO!!' }
+
+      let(:mock_res) do
+        res = mock
+        res.expects(:success?).returns(false).at_least_once
+        res.expects(:status).returns(401).at_least_once
+        res.expects(:body).returns(body).at_least_once
+        res
+      end
+
+      it 'should return a response' do
+       -> { subject.call }.must_raise(Azure::Core::Http::HTTPError)
       end
     end
   end