aviator 0.1.0 → 0.2.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: f56372f2f4fea06121bebdef1c2bccfef12c1f56
-  data.tar.gz: a17fe8facfb17ba357ceb772d9087e89d53a59ca
+  metadata.gz: c8e26c692d08f4a3414f7b3ff14b120cee73bad5
+  data.tar.gz: 9d60456a3afe9eed57d27acd626a08c3a987b3be
 SHA512:
-  metadata.gz: 00aa0e3555582535cfac8d3699259d31140cc88f920f2d162c064d64e2b483ccc8bb484aa29f59f99be99cda355d57a450de29c08714d14e07d23978a9c3a116
-  data.tar.gz: 7a0155396260c67ab9c7f1c0d03504f7a1e81f8e36463b347303198375c0486d415acd5e97d265036a0ac7a7be9878c9d304ee2743d98dd40b6733f594611cef
+  metadata.gz: ffae1b956dc180c585cfa847889a9e0e8267843600e9bac5471a25e8781797ebf6eccb3cedeb05501375372aeedc15348d7b963790bd18682eddce2387cc5a95
+  data.tar.gz: f9ebba7e141c1f886d17573b04b0dd60b1708e7c814b3cbbaf6493096be8d78dd4bbc76a52a83c308df63d0fe9e0e8eaacbffc76f907d617b1e9ea7d68cefceb

data/CONTRIBUTORS.txt

@@ -0,0 +1,11 @@
+Adrian Peterson Co <adrianpco@gmail.com>
+Alfonso Juan Dillera <aj.dillera@gmail.com>
+Alvin Garcia <agarcia1337@gmail.com>
+Colleen Murphy <colleen@puppetlabs.com>
+Lewis Marshall <lewis@technoplusit.co.uk>
+Mark Maglana <mmaglana@gmail.com>
+Nate West <natescott.west@gmail.com>
+Ricardo Rocha <rocha.porto@gmail.com>
+Stephen Paul Suarez <ssuarez@exist.com>
+Weto Olaguer <wenceslaoolaguer@gmail.com>
+Behrooz Shabani <everplays@gmail.com>

data/README.md

@@ -6,5 +6,6 @@
 [![Code Climate](https://codeclimate.com/github/aviator/aviator.png)](https://codeclimate.com/github/aviator/aviator)
 [![Gem Version](https://badge.fury.io/rb/aviator.png)](http://badge.fury.io/rb/aviator)
 
+Aviator complies with <a href="http://semver.org/">Semantic Versioning</a>.
 
 <a href="http://aviator.github.io/www/">Usage and Installation</a>

data/lib/aviator/core/cli/describer.rb

@@ -36,7 +36,7 @@ module Aviator
       request_class = "Aviator::#{ provider_name.camelize }::#{ service_name.camelize }::Requests::"\
                       "#{ api_version.camelize }::#{ endpoint_type.camelize }::#{ request_name.camelize }".constantize
 
-      display = ":Request => #{ request_name }\n"
+      display = "Request: #{ request_name }\n"
 
 
       # Build the parameters
@@ -83,7 +83,7 @@ module Aviator
       # Build the sample code
       display << "\nSample Code:\n"
 
-      display << "  session.#{ service_name }_service.request(:#{ request_name })"
+      display << "  session.request(:#{ service_name }_service, :#{ request_name })"
 
       if params && params.length > 0
         display << " do |params|\n"

data/lib/aviator/core/response.rb

@@ -14,7 +14,7 @@ module Aviator
 
 
     def body
-      if raw_body.length > 0
+      @body ||= if raw_body.length > 0
         if Aviator::Compatibility::RUBY_1_8_MODE
           clean_body = raw_body.gsub(/\\ /, ' ')
         else
@@ -29,14 +29,22 @@ module Aviator
 
 
     def headers
-      Hashish.new(@response.headers)
+      @headers ||= Hashish.new(@response.headers)
     end
 
 
+    def to_hash
+      Hashish.new({
+        :status  => status,
+        :headers => headers,
+        :body    => body
+      })
+    end
+
     private
 
     def raw_body
-      @response.body
+      @raw_body ||= @response.body
     end
 
   end

data/lib/aviator/core/service.rb

@@ -1,5 +1,8 @@
 module Aviator
 
+  #
+  # Manages a service
+  #
   class Service
 
     class AccessDetailsNotDefinedError < StandardError
@@ -35,7 +38,6 @@ module Aviator
       end
     end
 
-
     class UnknownRequestError < StandardError
       def initialize(request_name, options)
         super "Unknown request #{ request_name } #{ options }."
@@ -75,7 +77,9 @@ module Aviator
       load_requests
     end
 
-
+    #
+    # No longer recommended for public use. Use Aviator::Session#request instead
+    #
     def request(request_name, options={}, &params)
       if options[:api_version].nil? && @default_options[:api_version]
         options[:api_version] = @default_options[:api_version]
@@ -93,6 +97,19 @@ module Aviator
 
       raise UnknownRequestError.new(request_name, options) unless request_class
 
+      # Always use :params over &params if provided
+      if options[:params]
+        params = lambda do |params|
+          options[:params].each do |key, value|
+            begin
+              params[key] = value
+            rescue NameError => e
+              raise NameError.new("Unknown param name '#{key}'")
+            end
+          end
+        end
+      end
+
       request  = request_class.new(session_data, &params)
 
       response = http_connection.send(request.http_method) do |r|

data/lib/aviator/core/session.rb

@@ -1,12 +1,14 @@
+#
+# Author::    Mark Maglana (mmaglana@gmail.com)
+# Copyright:: Copyright (c) 2014 Mark Maglana
+# License::   Distributed under the MIT license
+# Homepage::  http://aviator.github.io/www/
+#
 module Aviator
 
   #
-  # Manages a provider (e.g. OpenStack) session.
-  #
-  # Author::    Mark Maglana (mmaglana@gmail.com)
-  # Copyright:: Copyright (c) 2014 Mark Maglana
-  # License::   Distributed under the MIT license
-  # Homepage::  http://aviator.github.io/www/
+  # Manages a provider (e.g. OpenStack) session and serves as the entry point
+  # for a consumer class/object. See Session::new for notes on usage.
   #
   class Session
 
@@ -51,8 +53,7 @@ module Aviator
     end
 
     #
-    # Create a new Session instance with options provided in <tt>opts</tt> which can
-    # have many forms discussed below.
+    # Create a new Session instance.
     #
     # <b>Initialize with a config file</b>
     #
@@ -73,17 +74,20 @@ module Aviator
     #      password: mypassword
     #      tenant_name: myproject
     #
+    # <b>SIDENOTE:</b> For more information about the <tt>validator</tt> member, see Session#validate.
+    #
     # Once the session has been instantiated, you may authenticate against the
     # provider as follows:
     #
     #  session.authenticate
     #
-    # Note that the required items under <tt>auth_credentials</tt> in the config
-    # file depends on the required parameters of the request class declared under
-    # <tt>auth_service</tt>. If writing the <tt>auth_credentials</tt> in the config
-    # file is not acceptable, you may omit it and just supply the credentials at
-    # runtime. For instance, assume that the <tt>auth_credentials</tt> section in the
-    # config file above is missing. You would then authenticate as follows:
+    # The members you put under <tt>auth_credentials</tt> will depend on the request
+    # class you declare under <tt>auth_service:request</tt> and what parameters it
+    # accepts. To know more about a request class and its parameters, you can use
+    # the CLI tool <tt>aviator describe</tt> or view the request definition file directly.
+    #
+    # If writing the <tt>auth_credentials</tt> in the config file is not acceptable,
+    # you may omit it and just supply the credentials at runtime. For example:
     #
     #  session.authenticate do |params|
     #    params.username    = ARGV[0]
@@ -91,7 +95,7 @@ module Aviator
     #    params.tenant_name = ARGV[2]
     #  end
     #
-    # Please see Session#authenticate for more info.
+    # See Session#authenticate for more info.
     #
     # Note that while the example config file above only has one environment (production),
     # you can declare an arbitrary number of environments in your config file. Shifting
@@ -100,8 +104,8 @@ module Aviator
     #
     # <b>Initialize with an in-memory hash</b>
     #
-    # You can create an in-memory hash which is similar in structure to the config file except
-    # that you don't need to specify an environment name. For example:
+    # You can create an in-memory hash with a structure similar to the config file but without
+    # the environment name. For example:
     #
     #  configuration = {
     #    :provider => 'openstack',
@@ -150,12 +154,14 @@ module Aviator
     end
 
     #
-    # Authenticates against the auth_service request class declared in the session's
-    # configuration during initialization. Please see Session.new for more information
+    # Authenticates against the backend provider using the auth_service request class
+    # declared in the session's configuration. Please see Session.new for more information
     # on declaring the request class to use for authentication.
     #
-    # If the auth_service request class accepts a parameter block, you may also supply that
-    # when calling this method and it will be directly passed to the request. For example:
+    # <b>Request params block</b>
+    #
+    # If the auth_service request class accepts parameters, you may supply that
+    # as a block and it will be directly passed to the request. For example:
     #
     #  session = Aviator::Session.new(:config => config)
     #  session.authenticate do |params|
@@ -164,9 +170,12 @@ module Aviator
     #    params.tenant_name = project
     #  end
     #
-    # Expects an HTTP status 200 or 201. Any other status is treated as a failure.
+    # If your configuration happens to have an <tt>auth_credentials</tt> in it, those
+    # will be overridden by this block.
     #
-    # Note that you can also treat the block's argument like a hash with the attribute
+    # <b>Treat parameters as a hash</b>
+    #
+    # You can also treat the params struct like a hash with the attribute
     # names as the keys. For example, we can rewrite the above as:
     #
     #  session = Aviator::Session.new(:config => config)
@@ -178,7 +187,29 @@ module Aviator
     #
     # Keys can be symbols or strings.
     #
-    def authenticate(&block)
+    # <b>Use a hash argument instead of a block</b>
+    #
+    # You may also provide request params as an argument instead of a block. This is
+    # especially useful if you want to mock Aviator as it's easier to specify ordinary
+    # argument expectations over blocks. Further rewriting the example above,
+    # we end up with:
+    #
+    #  session = Aviator::Session.new(:config => config)
+    #  session.authenticate :params => {
+    #    :username    => username,
+    #    :password    => password,
+    #    :tenant_name => project
+    #  }
+    #
+    # If both <tt>:params</tt> and a block are provided, the <tt>:params</tt>
+    # values will be used and the block ignored.
+    #
+    # <b>Success requirements</b>
+    #
+    # Expects an HTTP status 200 or 201 response from the backend. Any other
+    # status is treated as a failure.
+    #
+    def authenticate(opts={}, &block)
       block ||= lambda do |params|
         config[:auth_credentials].each do |key, value|
           begin
@@ -189,7 +220,7 @@ module Aviator
         end
       end
 
-      response = auth_service.request config[:auth_service][:request].to_sym, &block
+      response = auth_service.request(config[:auth_service][:request].to_sym, opts, &block)
 
       if [200, 201].include? response.status
         @auth_response = Hashish.new({
@@ -204,7 +235,10 @@ module Aviator
     end
 
     #
-    # Returns true if the session has been authenticated.
+    # Returns true if the session has been authenticated. Note that this relies on
+    # cached response from a previous run of Session#authenticate if one was made.
+    # If you want to check against the backend provider if the session is still valid,
+    # use Session#validate instead.
     #
     def authenticated?
       !auth_response.nil?
@@ -247,7 +281,7 @@ module Aviator
     end
 
 
-    def method_missing(name, *args, &block)
+    def method_missing(name, *args, &block) # :nodoc:
       service_name_parts = name.to_s.match(/^(\w+)_service$/)
 
       if service_name_parts
@@ -262,7 +296,8 @@ module Aviator
     # Creates a new Session object from a previous session's dump. See Session#dump for
     # more information.
     #
-    # If you want the newly deserialized session to log its output, make sure to indicate it on load
+    # If you want the newly deserialized session to log its output, add a <tt>:log_file</tt>
+    # option.
     #
     #  Aviator::Session.load(session_dump_str, :log_file => 'path/to/aviator.log')
     #
@@ -301,37 +336,77 @@ module Aviator
     #
     # Keys can be symbols or strings.
     #
+    # You may also provide parameters as an argument instead of a block. This is
+    # especially useful when mocking Aviator as it's easier to specify ordinary
+    # argument expectations over blocks. Further rewriting the example above,
+    # we end up with:
+    #
+    #  session.request :compute_service, :create_server, :params => {
+    #    :name       => "My Server",
+    #    :image_ref  => "7cae8c8e-fb01-4a88-bba3-ae0fcb1dbe29",
+    #    :flavor_ref => "fa283da1-59a5-4245-8569-b6eadf69f10b"
+    #  }
+    #
+    # If both <tt>:params</tt> and a block are provided, the values in <tt>:params</tt>
+    # will be used and the block ignored.
+    #
+    # <b>Return Value</b>
+    #
+    # The return value will be an instance of Hashish, a lightweight replacement for
+    # activesupport's HashWithIndifferentAccess, with the following structure:
+    #
+    #   {
+    #     :status => 200,
+    #     :headers => {
+    #       'X-Auth-Token' => 'd9186f45ce5446eaa0adc9def1c46f5f',
+    #       'Content-Type' => 'application/json'
+    #     },
+    #     :body => {
+    #       :some_key => :some_value
+    #     }
+    #   }
+    #
+    # Note that the members in <tt>:headers</tt> and <tt>:body</tt> will vary depending
+    # on the provider and the request that was made.
+    #
     # ---
     #
     # <b>Request Options</b>
     #
-    # You can further customize how the request is fulfilled by providing one or more
-    # options to the method call. For example, the following ensures that the request
-    # will call the :create_server request for the v1 API.
+    # You can further customize how the method behaves by providing one or more
+    # options to the call. For example, assuming you are using the <tt>openstack</tt>
+    # provider, the following will call the <tt>:create_server</tt> request of the
+    # v1 API of <tt>:compute_service</tt>.
     #
-    #  session.request :compute_service, :create_server, :api_version => v1
+    #  session.request :compute_service, :create_server, :api_version => v1, :params => params
     #
     # The available options vary depending on the provider. See the documentation
-    # on the provider's Provider class for more information (e.g. Aviator::OpenStack::Provider)
+    # on the provider's Provider class for more information (e.g. Aviator::Openstack::Provider)
     #
-    def request(service_name, request_name, opts={}, &params)
+    def request(service_name, request_name, opts={}, &block)
       service = send("#{service_name.to_s}_service")
-      service.request(request_name, opts, &params)
+      response = service.request(request_name, opts, &block)
+      response.to_hash
     end
 
 
     #
-    # Returns true if the session is still valid in the underlying provider. This method does this
-    # by calling the validator request class declared declared under <tt>auth_service</tt> in the
+    # Returns true if the session is still valid in the underlying provider. This method calls
+    # the <tt>validator</tt> request class declared under <tt>auth_service</tt> in the
     # configuration. The validator can be any request class as long as:
     #
     # * The request class exists!
+    # * Is not an anonymous request. Otherwise it will always return true.
     # * Does not require any parameters
     # * It returns an HTTP status 200 or 203 to indicate auth info validity.
     # * It returns any other HTTP status to indicate that the auth info is invalid.
     #
     # See Session::new for an example on how to specify the request class to use for session validation.
     #
+    # Note that this method requires the session to be previously authenticated otherwise a
+    # NotAuthenticatedError will be raised. If you just want to check if the session was previously
+    # authenticated, use Session#authenticated? instead.
+    #
     def validate
       raise NotAuthenticatedError.new unless authenticated?
       raise ValidatorNotDefinedError.new unless config[:auth_service][:validator]

data/lib/aviator/openstack/identity/requests/v3/public/create_token.rb

@@ -1,7 +1,6 @@
-# Original work by Stephen Paul Suarez
-# https://github.com/musashi-dev/aviator/blob/develop/lib/aviator/openstack/identity/v3/public/create_token.rb
-
 module Aviator
+  # Original work by Stephen Paul Suarez
+  # https://github.com/musashi-dev/aviator/blob/develop/lib/aviator/openstack/identity/v3/public/create_token.rb
 
   define_request :create_token, :inherit => [:openstack, :common, :v3, :public, :base] do
 

data/lib/aviator/openstack/provider.rb

@@ -1,13 +1,6 @@
 module Aviator
 module Openstack
 
-  #
-  # Manages a provider (e.g. OpenStack) session.
-  #
-  # Author::    Mark Maglana (mmaglana@gmail.com)
-  # Copyright:: Copyright (c) 2014 Mark Maglana
-  # License::   Distributed under the MIT license
-  # Homepage::  http://aviator.github.io/www/
   #
   # <b>Request Options</b>
   #
@@ -16,10 +9,10 @@ module Openstack
   #
   # :api_version => :v2::
   #     Forces Aviator to use the request class for the v2 API. For any other
-  #     version, replace Note that this may throw an error if no such request
-  #     class exists. If you want to globally specify the API version to use for
-  #     a specific service, declare it in your config file under the correct
-  #     environment. For example:
+  #     version, replace :v2 with the desired one. Note that this may throw an
+  #     error if no such request class for the given api version exists. If you
+  #     want to globally specify the API version to use for a specific service,
+  #     declare it in your config file under the correct environment. For example:
   #
   #  production:
   #    provider: openstack

data/lib/aviator/version.rb

@@ -1,3 +1,3 @@
 module Aviator
-  VERSION = "0.1.0"
+  VERSION = "0.2.1"
 end

data/test/aviator/core/cli/describer_test.rb

@@ -110,10 +110,10 @@ class Aviator::Test
         request_class = build_request(provider, service, request_name)
 
         expected = <<-EOF
-:Request => #{ request_name }
+Request: #{ request_name }
 
 Sample Code:
-  session.#{ service }_service.request(:#{ request_name })
+  session.request(:#{ service }_service, :#{ request_name })
 EOF
 
         output = klass.describe_request(
@@ -139,7 +139,7 @@ EOF
                         end
 
         expected = <<-EOF
-:Request => #{ request_name }
+Request: #{ request_name }
 
 Parameters:
  +----------+-----------+
@@ -150,7 +150,7 @@ Parameters:
  +----------+-----------+
 
 Sample Code:
-  session.#{ service }_service.request(:#{ request_name }) do |params|
+  session.request(:#{ service }_service, :#{ request_name }) do |params|
     params.another = value
     params.theParam = value
   end
@@ -179,7 +179,7 @@ EOF
                         end
 
         expected = <<-EOF
-:Request => #{ request_name }
+Request: #{ request_name }
 
 Parameters:
  +--------------+-----------+---------------+
@@ -190,7 +190,7 @@ Parameters:
  +--------------+-----------+---------------+
 
 Sample Code:
-  session.#{ service }_service.request(:#{ request_name }) do |params|
+  session.request(:#{ service }_service, :#{ request_name }) do |params|
     params.another_param = value
     params.the_param = value
   end
@@ -221,7 +221,7 @@ EOF
                         end
 
         expected = <<-EOF
-:Request => #{ request_name }
+Request: #{ request_name }
 
 Parameters:
  +--------------+-----------+---------------+
@@ -232,7 +232,7 @@ Parameters:
  +--------------+-----------+---------------+
 
 Sample Code:
-  session.#{ service }_service.request(:#{ request_name }) do |params|
+  session.request(:#{ service }_service, :#{ request_name }) do |params|
     params.another_param = value
     params.the_param = value
   end

data/test/aviator/core/response_test.rb

@@ -0,0 +1,41 @@
+require 'test_helper'
+require 'mocha/setup'
+
+
+class Aviator::Test
+
+  describe 'aviator/core/response' do
+
+    describe '#to_hash' do
+
+      it 'returns a Hashish instance representing the response' do
+        mock_http_response = mock('Faraday::Response')
+        mock_request = mock('Aviator::Request')
+
+        hashed_response = {
+          :status => 200,
+          :headers => {
+            :header1 => 'value1',
+            :header2 => 'value2'
+          },
+          :body => {
+            :body1 => 'bodyvalue1',
+            :body2 => 'bodyvalue2'
+          }
+        }
+
+        mock_http_response.expects(:status).returns(hashed_response[:status])
+        mock_http_response.expects(:headers).returns(hashed_response[:headers])
+        mock_http_response.expects(:body).returns(hashed_response[:body].to_json)
+
+        response = Aviator::Response.new(mock_http_response, mock_request)
+
+        response.to_hash.must_equal Hashish.new(hashed_response)
+      end
+
+    end
+
+
+  end
+
+end

data/test/aviator/core/session_test.rb

@@ -112,6 +112,33 @@ describe 'Aviator::Session' do
     end
 
 
+    it 'authenticates against the auth service using the credentials in a params arg' do
+      # Keys below are dependent on whatever is declared in
+      # the request classes referenced by auth_service:request
+      # in the config file
+      params = {
+        'username' => 'someuser',
+        'password' => 'password'
+      }
+
+      mock_conn = mock('Faraday::Connection')
+      mock_http_req = mock('Faraday::Request')
+      mock_response = mock('Faraday::Response')
+      Faraday.expects(:new).returns(mock_conn)
+      mock_conn.expects(:post).yields(mock_http_req).returns(mock_response)
+      mock_http_req.expects(:url)
+      mock_http_req.expects(:body=).with(){|json| JSON.load(json) == params }
+      mock_response.expects(:status).returns(200)
+      mock_response.expects(:headers).returns({})
+      mock_response.expects(:body).returns({})
+
+      session = Aviator::Session.new(:config => valid_config[valid_env])
+      session.authenticate :params => params
+
+      session.authenticated?.must_equal true
+    end
+
+
     it 'raises an AuthenticationError when authentication fails' do
       Faraday.expects(:new).returns(mock_conn = mock('Faraday::Connection'))
       mock_conn.expects(:post).returns(mock_response = mock('Faraday::Response'))
@@ -353,14 +380,48 @@ describe 'Aviator::Session' do
       mock_response.stubs(:status).returns(200)
       mock_response.stubs(:headers).returns({})
       mock_response.stubs(:body).returns('{}')
+      hashed_response = {:response1 => :value1}
+      mock_response.stubs(:to_hash).returns(hashed_response)
       mock_auth.expects(:request).returns(mock_response)
       mock_compute.expects(:request).with(request_name, request_opts).yields(yielded_params).returns(mock_response)
 
       session = Aviator::Session.new(:config => valid_config[valid_env])
       session.authenticate
 
-      session.request :compute, request_name, request_opts, &request_params
+      response = session.request :compute, request_name, request_opts, &request_params
       yielded_params.wont_be_empty
+      response.must_equal hashed_response
+    end
+
+
+    it 'passes the params arg to Service#request' do
+      request_name   = :create_server
+      request_opts   = {
+        :params => {
+           :key1 => :value1,
+           :key2 => :value2
+        }
+      }
+
+      mock_auth     = mock('Aviator::Service')
+      mock_compute  = mock('Aviator::Service')
+      mock_response = mock('Aviator::Response')
+
+      Aviator::Service.expects(:new).twice.returns(mock_auth, mock_compute)
+      mock_response.stubs(:status).returns(200)
+      mock_response.stubs(:headers).returns({:header1 => :headervalue1})
+      mock_response.stubs(:body).returns('{":bodykey1": "bodyvalue1"}')
+      hashed_response = {:responsekey1 => :responsevalue1}
+      mock_response.stubs(:to_hash).returns(hashed_response)
+      mock_auth.expects(:request).returns(mock_response)
+      mock_compute.expects(:request).with(request_name, request_opts).returns(mock_response)
+
+      session = Aviator::Session.new(:config => valid_config[valid_env])
+      session.authenticate
+
+      response = session.request :compute, request_name, request_opts
+
+      response.must_equal hashed_response
     end
 
   end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: aviator
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 0.2.1
 platform: ruby
 authors:
 - Mark Maglana
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-12-08 00:00:00.000000000 Z
+date: 2014-12-14 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: faraday
@@ -255,6 +255,7 @@ files:
 - ".coveralls.yml"
 - ".gitignore"
 - ".travis.yml"
+- CONTRIBUTORS.txt
 - Gemfile
 - Guardfile
 - LICENSE.txt
@@ -358,6 +359,7 @@ files:
 - test/aviator/core/cli/describer_test.rb
 - test/aviator/core/request_builder_test.rb
 - test/aviator/core/request_test.rb
+- test/aviator/core/response_test.rb
 - test/aviator/core/service_test.rb
 - test/aviator/core/session_test.rb
 - test/aviator/openstack/common/requests/v0/public/base_test.rb
@@ -940,6 +942,7 @@ test_files:
 - test/aviator/core/cli/describer_test.rb
 - test/aviator/core/request_builder_test.rb
 - test/aviator/core/request_test.rb
+- test/aviator/core/response_test.rb
 - test/aviator/core/service_test.rb
 - test/aviator/core/session_test.rb
 - test/aviator/openstack/common/requests/v0/public/base_test.rb