quiz_api_client 4.1.0 → 4.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: f901bb687e405ab11127e8cbbe476f38c3578d122d838b25438cc106287b4d83
-  data.tar.gz: 6bae389bad7c912f709747a00a562e42cd10e4b85d1d72f5a7c5ffff6564ae61
+  metadata.gz: 11e55f74ad6fd90e5f09e214394d3d44103e7027d83ce03299f5670cbc79111d
+  data.tar.gz: 836308804ec45f08806c57bf63c6d7b9167e03eee52d955e7bca38b310f75305
 SHA512:
-  metadata.gz: 519030cead9c1d820cf40bb489029433f751ce41e6488dcddfd7a8520ea2c47c5fa7a8a96e0aaea59640793f0efcc7fcd825c3d5e2fbe026533cb9d18208fd18
-  data.tar.gz: b906985f27f6e78e734f248685ae071efde9414cc3198d5a214caf9d54d8c1d2f29e6cc7d9e53677eac2c1c1d4d9bbd3016f3cea210173c1cf00628784d18507
+  metadata.gz: 9ed1d073d41458ff170b5767eec5d37e560af8211f6fa5da0906d7d644411a9a4e589c7d644b807300dd76400374e0e9bcd58d4595bd2e4004d65d228e4afb9f
+  data.tar.gz: 26d1ce56705ad6a1ba18c93175ae572e4ab54e95438513095c663c6a3d6efda3a57cce29ff432ac6885b5bf139305f954bab53890360b89945fdc0c48878b123

data/README.md

@@ -41,6 +41,10 @@ client = QuizApiClient::Client.new(
 
 In order to set addition error information in your error handling system, you can set the `error_hander` property of the config.  The list of valid values can be found in the `QuizApiClient::Config::ERROR_HANDLERS` constant.
 
+### Metrics
+
+HTTP Request metrics will now be provided if the `setup_metrics` method is called.  The list of valid values can be found in the `QuizApiClient::Config::METRICS_HANDLERS` constant.  The count and duration metrics will be provided under the namespace provided.
+
 ### Creation of Tokens
 
 JWTs are created without hitting quiz_api and and they are validated on quiz_api. Tokens are created for a given scope, expiration, and an optional resource_id.

data/lib/quiz_api_client/config.rb

@@ -3,10 +3,13 @@ module QuizApiClient
     DEFAULT_ALLOWABLE_RESPONSE_CODES = [401, 422].freeze
     DEFAULT_PROTOCOL = 'https'.freeze
     ERROR_HANDLERS = %i[sentry_raven].freeze
+    METRICS_HANDLERS = %i[inststatsd].freeze
 
     class InvalidErrorHandler < StandardError; end
+    class InvalidMetricsHandler < StandardError; end
+    class InvalidMetricsNamespace < StandardError; end
 
-    attr_reader :error_handler
+    attr_reader :error_handler, :metrics_handler, :metrics_namespace
     attr_writer :protocol, :allowable_response_codes
     attr_accessor :consumer_key, :consumer_request_id, :host, :shared_secret
 
@@ -19,9 +22,17 @@ module QuizApiClient
     end
 
     def error_handler=(handler)
-      validate_error_hander!(handler)
+      validate_error_handler!(handler)
 
-      @error_handler = handler
+      @error_handler = handler.to_sym
+    end
+
+    def setup_metrics(handler, namespace)
+      validate_metrics_handler!(handler)
+      validate_metrics_namespace!(namespace)
+
+      @metrics_handler = handler.to_sym
+      @metrics_namespace = namespace.to_s.strip
     end
 
     def allowable_response_codes
@@ -30,14 +41,24 @@ module QuizApiClient
 
     private
 
-    def validate_error_hander!(handler)
+    def validate_error_handler!(handler)
       return unless handler
+      return if ERROR_HANDLERS.include?(handler)
 
-      unless ERROR_HANDLERS.include?(handler)
-        raise InvalidErrorHandler, "It must be one of the following: #{ERROR_HANDLERS.inspect}"
-      end
+      raise InvalidErrorHandler, "It must be one of the following: #{ERROR_HANDLERS.inspect}"
+    end
 
-      @error_handler = handler.to_sym
+    def validate_metrics_handler!(handler)
+      return unless handler
+      return if METRICS_HANDLERS.include?(handler)
+
+      raise InvalidMetricsHandler, "It must be one of the following: #{METRICS_HANDLERS.inspect}"
+    end
+
+    def validate_metrics_namespace!(namespace)
+      return unless namespace.nil? || namespace.to_s.strip == ''
+
+      raise InvalidMetricsNamespace, 'It must be present'
     end
   end
 end

data/lib/quiz_api_client/http_client.rb

@@ -82,6 +82,7 @@ module QuizApiClient
     end
 
     def make_request(method, url, request_options = {})
+      start_time = Time.now
       resp = self.class.send(
         method,
         url,
@@ -91,6 +92,8 @@ module QuizApiClient
       resp
     rescue HTTParty::Error, Errno::ECONNREFUSED, Net::ReadTimeout => e
       raise_error(method, url, current_error: e)
+    ensure
+      record_metrics(method, url, resp, start_time, Time.now)
     end
 
     def make_paginated_request(method, url, options)
@@ -146,50 +149,18 @@ module QuizApiClient
       [url, options.merge(query: query)]
     end
 
-    def generate_error_context(method, url, response = nil)
-      context = {
-        quiz_api_client: {
-          request: {
-            method: method,
-            url: url
-          }
-        }
-      }
-
-      if response
-        context[:quiz_api_client][:response] = {
-          body: response.body,
-          code: response.code
-        }
-      end
-
-      context
-    end
-
-    def record_error_context(context)
-      case config.error_handler
-      when :sentry_raven
-        require 'sentry-raven'
-        Raven.extra_context(context)
-      end
-    end
+    def raise_error(method, url, response: nil, current_error: nil)
+      failure = QuizApiClient::HttpRequest::Failure.new(config)
 
-    def error_message(current_error, url, response)
-      if current_error && response
-        "#{current_error.message}: #{url} responded #{response.body} (#{response.code})"
-      elsif current_error
-        current_error.message
-      elsif response
-        "#{url} responded #{response.body} (#{response.code})"
-      end
+      failure.raise_error(method, url, response: response, current_error: current_error)
     end
 
-    def raise_error(method, url, response: nil, current_error: nil, error_class: RequestFailed)
-      context = generate_error_context(method, url, response)
-
-      record_error_context(context)
+    def record_metrics(method, url, resp, start_time, end_time)
+      code = resp&.code || 0
+      metrics = QuizApiClient::HttpRequest::Metrics.new(config, method, url, code)
 
-      raise error_class.new(context), error_message(current_error, url, response)
+      metrics.increment
+      metrics.duration(start_time, end_time)
     end
 
     def successful_response?(resp)

data/lib/quiz_api_client/http_request/failure.rb

@@ -0,0 +1,65 @@
+module QuizApiClient
+  module HttpRequest
+    class Failure
+      attr_reader :config
+
+      def initialize(config)
+        @config = config
+      end
+
+      def raise_error(
+        method,
+        url,
+        response: nil,
+        current_error: nil,
+        error_class: QuizApiClient::HttpClient::RequestFailed
+      )
+        context = generate_error_context(method, url, response)
+
+        record_error_context(context)
+
+        raise error_class.new(context), error_message(current_error, url, response)
+      end
+
+      private
+
+      def generate_error_context(method, url, response = nil)
+        context = {
+          quiz_api_client: {
+            request: {
+              method: method,
+              url: url
+            }
+          }
+        }
+
+        if response
+          context[:quiz_api_client][:response] = {
+            body: response.body,
+            code: response.code
+          }
+        end
+
+        context
+      end
+
+      def record_error_context(context)
+        case config.error_handler
+        when :sentry_raven
+          require 'sentry-raven'
+          Raven.extra_context(context)
+        end
+      end
+
+      def error_message(current_error, url, response)
+        if current_error && response
+          "#{current_error.message}: #{url} responded #{response.body} (#{response.code})"
+        elsif current_error
+          current_error.message
+        elsif response
+          "#{url} responded #{response.body} (#{response.code})"
+        end
+      end
+    end
+  end
+end

data/lib/quiz_api_client/http_request/metrics.rb

@@ -0,0 +1,67 @@
+module QuizApiClient
+  module HttpRequest
+    class Metrics
+      extend Forwardable
+
+      attr_reader :config, :method, :url, :code
+      def_delegators :config, :metrics_handler, :metrics_namespace
+
+      def initialize(config, method, url, code)
+        @config = config
+        @method = method
+        @url = url
+        @code = code
+      end
+
+      def increment
+        return unless configured?
+
+        case metrics_handler
+        when :inststatsd
+          InstStatsd::Statsd.increment count_metric_name, tags: tags
+        end
+      end
+
+      def duration(start_time, end_time)
+        return unless configured?
+
+        duration_ms = ((end_time - start_time) * 1_000).round
+
+        case metrics_handler
+        when :inststatsd
+          InstStatsd::Statsd.timing duration_metric_name, duration_ms, tags: tags
+        end
+      end
+
+      private
+
+      def configured?
+        metrics_handler_present? && metrics_namespace_present?
+      end
+
+      def count_metric_name
+        "#{metrics_namespace}.quiz_api_client.request.count"
+      end
+
+      def duration_metric_name
+        "#{metrics_namespace}.quiz_api_client.request.duration_ms"
+      end
+
+      def metrics_handler_present?
+        !metrics_handler.nil?
+      end
+
+      def metrics_namespace_present?
+        !metrics_namespace.nil?
+      end
+
+      def tags
+        {
+          method: method,
+          status: code,
+          url: url
+        }
+      end
+    end
+  end
+end

data/lib/quiz_api_client/services/items_service.rb

@@ -1,5 +1,11 @@
 module QuizApiClient::Services
   class ItemsService < BaseApiService
+    def list(params:, token: nil)
+      raise 'Quiz Id Required' unless params && params[:quiz_id]
+
+      list_from_quiz_api(params: params, token: token)
+    end
+
     def create(params:, token: nil)
       raise 'Quiz Id Required' unless params && params[:quiz_id]
 
@@ -8,6 +14,11 @@ module QuizApiClient::Services
 
     private
 
+    def list_from_quiz_api(params:, token:)
+      quiz_id = params.delete(:quiz_id)
+      client(token: token).get("/api/quizzes/#{quiz_id}/items")
+    end
+
     def post_to_quiz_api(params:, token:)
       quiz_id = params.delete(:quiz_id)
       client(token: token).post(

data/lib/quiz_api_client/services/quiz_clone_jobs_service.rb

@@ -4,6 +4,10 @@ module QuizApiClient::Services
       post_to_quiz_api(params: params, token: token)
     end
 
+    def create_batch(body:, token: nil)
+      batch_post_to_quiz_api(body: body, token: token)
+    end
+
     private
 
     def post_to_quiz_api(params:, token:)
@@ -11,5 +15,12 @@ module QuizApiClient::Services
         "/api/quizzes/#{params.fetch(:quiz_id)}/quiz_clone_jobs"
       )
     end
+
+    def batch_post_to_quiz_api(body:, token:)
+      client(token: token).post(
+        '/api/quiz_clone_jobs/create_batch',
+        body
+      )
+    end
   end
 end

data/lib/quiz_api_client/services/quiz_sync_job_service.rb

@@ -0,0 +1,17 @@
+module QuizApiClient::Services
+  class QuizSyncJobService < BaseApiService
+    def show(params:, token: nil)
+      raise 'Quiz Sync Job Id Required' unless params && params[:id]
+
+      get_from_quiz_api(params: params, token: token)
+    end
+
+    private
+
+    def get_from_quiz_api(params:, token:)
+      client(token: token).get(
+        "/api/quiz_sync_jobs/#{params.fetch(:id)}"
+      )
+    end
+  end
+end

data/lib/quiz_api_client/services/quiz_sync_jobs_service.rb

@@ -0,0 +1,16 @@
+module QuizApiClient::Services
+  class QuizSyncJobsService < BaseApiService
+    def create_batch(body:, token: nil)
+      batch_post_to_quiz_api(body: body, token: token)
+    end
+
+    private
+
+    def batch_post_to_quiz_api(body:, token:)
+      client(token: token).post(
+        '/api/quiz_sync_jobs/create_batch',
+        body
+      )
+    end
+  end
+end

data/lib/quiz_api_client/version.rb

@@ -1,3 +1,3 @@
 module QuizApiClient
-  VERSION = '4.1.0'.freeze
+  VERSION = '4.3.0'.freeze
 end

data/lib/quiz_api_client.rb

@@ -50,6 +50,14 @@ module QuizApiClient
       @_quiz_clone_jobs_service ||= Services::QuizCloneJobsService.new(config)
     end
 
+    def quiz_sync_job_service
+      @_quiz_sync_job_service ||= Services::QuizSyncJobService.new(config)
+    end
+
+    def quiz_sync_jobs_service
+      @_quiz_sync_jobs_service ||= Services::QuizSyncJobsService.new(config)
+    end
+
     def qti_imports_service
       @_qti_imports_service ||= Services::QtiImportsService.new(config)
     end
@@ -101,6 +109,8 @@ require 'quiz_api_client/version'
 require 'quiz_api_client/config'
 require 'quiz_api_client/http_client'
 require 'quiz_api_client/json_formatter'
+require 'quiz_api_client/http_request/failure'
+require 'quiz_api_client/http_request/metrics'
 require 'quiz_api_client/services/jwt_service'
 require 'quiz_api_client/services/base_api_service'
 
@@ -118,6 +128,8 @@ require 'quiz_api_client/services/quizzes_service'
 require 'quiz_api_client/services/quiz_session_result_service'
 require 'quiz_api_client/services/quiz_clone_job_service'
 require 'quiz_api_client/services/quiz_clone_jobs_service'
+require 'quiz_api_client/services/quiz_sync_job_service'
+require 'quiz_api_client/services/quiz_sync_jobs_service'
 require 'quiz_api_client/services/session_items_service'
 require 'quiz_api_client/services/session_item_results_service'
 require 'quiz_api_client/services/shared_banks'

data/spec/config_spec.rb

@@ -0,0 +1,66 @@
+describe QuizApiClient::Config do
+  describe 'initialize with block' do
+    it 'sets the config value' do
+      config = described_class.new do |c|
+        c.host = 'a host'
+      end
+      expect(config.host).to eq 'a host'
+    end
+  end
+
+  describe '#protocol' do
+    it 'returns the default when nil' do
+      config = described_class.new
+      expect(config.protocol).to eq QuizApiClient::Config::DEFAULT_PROTOCOL
+    end
+  end
+
+  describe '#error_handler=' do
+    it 'raises an error with an invalid error handler' do
+      config = described_class.new
+      expect { config.error_handler = :invalid_error_handler }
+        .to raise_error(QuizApiClient::Config::InvalidErrorHandler)
+    end
+  end
+
+  describe '#allowable_response_codes' do
+    it 'returns the default when nil' do
+      config = described_class.new
+      expect(config.allowable_response_codes).to eq QuizApiClient::Config::DEFAULT_ALLOWABLE_RESPONSE_CODES
+    end
+  end
+
+  describe '#setup_metrics' do
+    let(:valid_handler) { :inststatsd }
+    let(:invalid_handler) { :invalid_handler }
+
+    let(:valid_namespace) { :quiz_lti }
+    let(:invalid_namespace) { ' ' }
+
+    it 'raises an error with an invalid metrics handler' do
+      config = described_class.new
+      expect { config.setup_metrics(invalid_handler, valid_namespace) }
+        .to raise_error(QuizApiClient::Config::InvalidMetricsHandler)
+    end
+
+    it 'raises an error with an invalid namespace' do
+      config = described_class.new
+      expect { config.setup_metrics(valid_handler, invalid_namespace) }
+        .to raise_error(QuizApiClient::Config::InvalidMetricsNamespace)
+    end
+
+    it 'raises an error with an nil namespace' do
+      config = described_class.new
+      expect { config.setup_metrics(valid_handler, nil) }
+        .to raise_error(QuizApiClient::Config::InvalidMetricsNamespace)
+    end
+
+    it 'sets the handler and namespace successfully' do
+      config = described_class.new
+      config.setup_metrics(valid_handler, valid_namespace)
+
+      expect(config.metrics_handler).to eq valid_handler
+      expect(config.metrics_namespace).to eq valid_namespace.to_s
+    end
+  end
+end

data/spec/contracts/interaction_types_service_spec.rb

@@ -0,0 +1,22 @@
+describe QuizApiClient::Services::InteractionTypesService, :pact do
+  include PactHelper
+
+  it_behaves_like 'a http get request to quiz_api collection endpoint' do
+    let(:provider_state) { 'a set of interaction types' }
+    let(:request_description) { 'a request to retrieve a list of interaction types' }
+    let(:quizzes_api_path) { '/api/interaction_types' }
+    let(:scope) { 'quiz.build' }
+    let(:params) { nil }
+    let(:response_body) do
+      Pact.each_like(
+        id: '1',
+        slug: 'choice',
+        name: 'Multiple Choice',
+        properties_schema: {},
+        scoring_algorithm_default: 'Equivalence',
+        user_response_type_options: ['Uuid']
+      )
+    end
+    let(:service_name) { :interaction_types_service }
+  end
+end

data/spec/contracts/item_analyses_service_spec.rb

@@ -0,0 +1,59 @@
+describe QuizApiClient::Services::ItemAnalysesService, :pact do
+  include PactHelper
+
+  it_behaves_like 'a http get request to quiz_api collection endpoint', testrail_id: %w[3402546] do
+    let(:request_description) { 'a request to retrieve item analyses all quiz_id items' }
+    let(:quiz_id) { 1 }
+    let(:quizzes_api_path) { "/api/quizzes/#{quiz_id}/stats/item_analyses" }
+    let(:provider_state) { 'quiz with item analysis' }
+    let(:resource_id) { '1' }
+    let(:scope) { 'quiz.analysis' }
+    let(:params) { { quiz_id: '1' } }
+    let(:response_body) do
+      [
+        {
+          item_id: Pact.like('1'),
+          points_possible: Pact.like(1.0),
+          item_body: Pact.like('item body'),
+          interaction_type: 'choice',
+          item_analysis: {},
+          batch_timestamp: Pact.like('2017-06-15T22:14:51.215Z'),
+          outcome_alignment_set_guid: Pact.like('cbedd9b0-d1ab-43a9-a3b2-89c3134a722a')
+        },
+        {
+          item_id: Pact.like('1'),
+          points_possible: Pact.like(1.0),
+          item_body: Pact.like('item body'),
+          interaction_type: 'choice',
+          item_analysis: {},
+          batch_timestamp: Pact.like('2017-06-15T22:14:51.215Z'),
+          outcome_alignment_set_guid: Pact.like('cbedd9b0-d1ab-43a9-a3b2-89c3134a722a')
+        }
+      ]
+    end
+    let(:service_name) { :item_analyses_service }
+  end
+
+  it_behaves_like 'a http get only request to quiz_api', testrail_id: %w[3402547] do
+    let(:request_description) { 'a request to retrieve item analyses for a quiz item' }
+    let(:quiz_id) { 1 }
+    let(:id) { 1 }
+    let(:quizzes_api_path) { "/api/quizzes/#{quiz_id}/stats/item_analyses/#{id}" }
+    let(:provider_state) { 'quiz with item analysis' }
+    let(:resource_id) { '1' }
+    let(:scope) { 'quiz.analysis' }
+    let(:params) { { quiz_id: '1', id: '1' } }
+    let(:response_body) do
+      {
+        item_id: Pact.like('1'),
+        points_possible: Pact.like(1.0),
+        item_body: Pact.like('item body'),
+        interaction_type: 'choice',
+        item_analysis: {},
+        batch_timestamp: Pact.like('2017-06-15T22:14:51.215Z'),
+        outcome_alignment_set_guid: Pact.like('cbedd9b0-d1ab-43a9-a3b2-89c3134a722a')
+      }
+    end
+    let(:service_name) { :item_analyses_service }
+  end
+end

data/spec/contracts/items_service_spec.rb

@@ -0,0 +1,59 @@
+describe QuizApiClient::Services::ItemsService, :pact do
+  include PactHelper
+
+  it_behaves_like 'a http post request to quiz_api' do
+    let(:provider_state) { 'a quiz and a multiple choice interaction type' }
+    let(:request_description) { 'a request to create an item' }
+    let(:quizzes_api_path) { "/api/quizzes/#{quiz_id}/items" }
+    let(:resource_id) { '1' }
+    let(:scope) { 'quiz.build' }
+    let(:quiz_id) { 1 }
+    let(:params) do
+      {
+        quiz_id: quiz_id,
+        interaction_data: {
+          choices: [{
+            id: 'guid',
+            itemBody: '<p>Choice</p>',
+            position: 1
+          }]
+        },
+        interaction_type_id: 1, # we'll seed InteractionType::MultipleChoice
+        item_body: 'Some kind of stem',
+        properties: {},
+        scoring_algorithm: 'Equivalence',
+        scoring_data: {
+          value: 'another guid'
+        },
+        title: 'A perfectly cromulent title',
+        user_response_type: 'Uuid'
+      }
+    end
+    let(:item_params) { params.clone.tap { |p| p.delete(:quiz_id) } }
+    let(:body) { { quiz_id: quiz_id, item: item_params } }
+    let(:response_body) do
+      {
+        id: Pact.like('1'),
+        interaction_data: {
+          choices: Pact.each_like(
+            id: 'a guid',
+            item_body: 'choice description',
+            position: 1
+          )
+        },
+        item_body: Pact.like(''),
+        properties: Pact.like({}),
+        status: Pact.like(''),
+        title: Pact.like(''),
+        user_response_type: Pact.like(''),
+        stimulus_id: Pact.like(''),
+        interaction_type: Pact.like({}),
+        scoring_data: {
+          value: Pact.like('')
+        },
+        scoring_algorithm: Pact.like('')
+      }
+    end
+    let(:service_name) { :items_service }
+  end
+end

data/spec/contracts/qti_imports_service_spec.rb

@@ -0,0 +1,34 @@
+describe QuizApiClient::Services::QtiImportsService, :pact do
+  include PactHelper
+
+  it_behaves_like 'a http post request to quiz_api', testrail_id: %w[3402548] do
+    let(:request_description) { 'a request to import a QTI' }
+    let(:provider_state) { 'a quiz' }
+    let(:quiz_id) { 1 }
+    let(:url) { 'http://fake.url' }
+    let(:quizzes_api_path) { "/api/quizzes/#{quiz_id}/qti_imports" }
+    let(:scope) { 'qti_import.create' }
+    let(:resource_id) { quiz_id }
+    let(:params) do
+      {
+        quiz_id: quiz_id,
+        url: url
+      }
+    end
+    let(:body) { { qti_import: params } }
+    let(:response_body) do
+      {
+        id: Pact.like('1'),
+        parent_id: quiz_id.to_s,
+        parent_type: Pact.like('Quiz'),
+        status: 'scheduled',
+        url: url,
+        message: Pact.like(''),
+        created_at: Pact.like(''),
+        updated_at: Pact.like(''),
+        progress: Pact.like(0.0)
+      }
+    end
+    let(:service_name) { :qti_imports_service }
+  end
+end