quiz_api_client 4.14.0 → 4.14.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 450aa47af30bde7b962ac76248bde2fa992b4168875f1e8847c48539813576e5
-  data.tar.gz: 73da155fb2da7763ec9a07f557250842e62bd38ddfe570418df8a7eeee7608aa
+  metadata.gz: c7df5e01555ce19ad84445f0287c640e0c2eb9873af3c88a6ad1432162ce228a
+  data.tar.gz: 04d758bf5007880147d0e4316b6d151821b904b0a84db0401b7b53b426d6791f
 SHA512:
-  metadata.gz: bd2bf29f5d1a2b3d512e8cb75b40479789138551d7fce2438e3a215158b423f174bb19bb5f854f48373f6ea685815ca73dfa0431cbde2e0839c8ae191aeff1f0
-  data.tar.gz: 3a95461f774db17ccb8ed636d6a552c3ea4c50291d648989d44d2f621434c196ef21468de4c8e81d0f3ccf215e6dff132dcd8cfd5c404bceba484316319062a7
+  metadata.gz: f4e95a71d9e8853ae5a3ef251b6df743ee36ce1838be1bcdf7ab6675c6e85a800a5cb4a48a7f394b6094d6b9747ea90c5a1ff2942059991bd6a5f69306185043
+  data.tar.gz: 72c177b35fc78f784c0b26fa49d90c92bfc305539528b357e104864cf339cd870202c97710df4ca672e8d9505b96217be73039b313a4bec7bd244bffbd6c3c15

data/lib/quiz_api_client/http_client.rb

@@ -101,7 +101,7 @@ module QuizApiClient
     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)
+      record_metrics(method, resp, start_time, Time.now)
     end
 
     def make_paginated_request(method, url, options)
@@ -166,9 +166,9 @@ module QuizApiClient
       failure.raise_error(method, url, response: response, current_error: current_error)
     end
 
-    def record_metrics(method, url, resp, start_time, end_time)
+    def record_metrics(method, resp, start_time, end_time)
       code = resp&.code || 0
-      metrics = QuizApiClient::HttpRequest::Metrics.new(config, method, url, code)
+      metrics = QuizApiClient::HttpRequest::Metrics.new(config, method, code)
 
       metrics.increment
       metrics.duration(start_time, end_time)

data/lib/quiz_api_client/http_request/metrics.rb

@@ -3,13 +3,12 @@ module QuizApiClient
     class Metrics
       extend Forwardable
 
-      attr_reader :config, :method, :url, :code
+      attr_reader :config, :method, :code
       def_delegators :config, :metrics_handler, :metrics_namespace
 
-      def initialize(config, method, url, code)
+      def initialize(config, method, code)
         @config = config
         @method = method
-        @url = url
         @code = code
       end
 
@@ -58,8 +57,7 @@ module QuizApiClient
       def tags
         {
           method: method,
-          status: code,
-          url: url
+          status: code
         }
       end
     end

data/lib/quiz_api_client/version.rb

@@ -1,3 +1,3 @@
 module QuizApiClient
-  VERSION = '4.14.0'.freeze
+  VERSION = '4.14.1'.freeze
 end

data/spec/http_client_spec.rb

@@ -41,9 +41,9 @@ describe QuizApiClient::HttpClient do
     mock
   end
 
-  def expect_metrics_calls(config, method, url, code)
+  def expect_metrics_calls(config, method, code)
     expect(QuizApiClient::HttpRequest::Metrics).to receive(:new)
-      .with(config, method, url, code)
+      .with(config, method, code)
       .and_return(mock_metrics(mock_time(10)))
   end
 
@@ -65,7 +65,7 @@ describe QuizApiClient::HttpClient do
     it 'makes a get request' do
       path = '/api/quizzes'
       stub_quiz_api path, query: { sort: 'alpha' }
-      expect_metrics_calls(client.config, :get, url_for_path(path), 200)
+      expect_metrics_calls(client.config, :get, 200)
       response = client.get(path, all: false, query: { sort: 'alpha' })
       expect(response.parsed_response).to eq [1]
     end
@@ -138,7 +138,7 @@ describe QuizApiClient::HttpClient do
         body: { title: 'ohai' }.to_json,
         **default_request_data
       ).and_return(instance_double('HTTParty::Response', success?: true, code: 200))
-      expect_metrics_calls(client.config, :post, url, 200)
+      expect_metrics_calls(client.config, :post, 200)
       client.post('/api/quizzes', title: 'ohai')
     end
   end
@@ -151,7 +151,7 @@ describe QuizApiClient::HttpClient do
         body: { title: 'new title' }.to_json,
         **default_request_data
       ).and_return(instance_double('HTTParty::Response', success?: true, code: 200))
-      expect_metrics_calls(client.config, :patch, url, 200)
+      expect_metrics_calls(client.config, :patch, 200)
       client.patch('/api/quizzes/1', title: 'new title')
     end
   end
@@ -164,7 +164,7 @@ describe QuizApiClient::HttpClient do
         body: { title: 'new title' }.to_json,
         **default_request_data
       ).and_return(instance_double('HTTParty::Response', success?: true, code: 200))
-      expect_metrics_calls(client.config, :put, url, 200)
+      expect_metrics_calls(client.config, :put, 200)
       client.put('/api/quizzes/1', title: 'new title')
     end
   end
@@ -176,7 +176,7 @@ describe QuizApiClient::HttpClient do
         url,
         **default_request_data
       ).and_return(instance_double('HTTParty::Response', success?: true, code: 200))
-      expect_metrics_calls(client.config, :delete, url, 200)
+      expect_metrics_calls(client.config, :delete, 200)
       client.delete('/api/quizzes/1')
     end
   end
@@ -277,7 +277,7 @@ describe QuizApiClient::HttpClient do
           it 'raises error' do
             expect(client).to receive(:successful_response?).with(mock_response).and_return(false)
             expect(client.class).to receive(:get).and_return(mock_response)
-            expect_metrics_calls(client.config, method, url_for_path(path), code)
+            expect_metrics_calls(client.config, method, code)
             expect_raise_error_call(client.config, method, url_for_path(path), mock_response, nil)
             expect { client.get(path, all: false, query: { sort: 'alpha' }) }
               .to raise_error(QuizApiClient::HttpClient::RequestFailed)
@@ -290,7 +290,7 @@ describe QuizApiClient::HttpClient do
           it 'raises error' do
             expect(client).to receive(:successful_response?).with(mock_response).and_return(false)
             expect(client.class).to receive(:post).and_return(mock_response)
-            expect_metrics_calls(client.config, method, url_for_path(path), code)
+            expect_metrics_calls(client.config, method, code)
             expect_raise_error_call(client.config, method, url_for_path(path), mock_response, nil)
             expect { client.post(path) }.to raise_error(QuizApiClient::HttpClient::RequestFailed)
           end
@@ -302,7 +302,7 @@ describe QuizApiClient::HttpClient do
           it 'raises error' do
             expect(client).to receive(:successful_response?).with(mock_response).and_return(false)
             expect(client.class).to receive(:put).and_return(mock_response)
-            expect_metrics_calls(client.config, method, url_for_path(path), code)
+            expect_metrics_calls(client.config, method, code)
             expect_raise_error_call(client.config, method, url_for_path(path), mock_response, nil)
             expect { client.put(path) }.to raise_error(QuizApiClient::HttpClient::RequestFailed)
           end
@@ -314,7 +314,7 @@ describe QuizApiClient::HttpClient do
           it 'raises error' do
             expect(client).to receive(:successful_response?).with(mock_response).and_return(false)
             expect(client.class).to receive(:patch).and_return(mock_response)
-            expect_metrics_calls(client.config, method, url_for_path(path), code)
+            expect_metrics_calls(client.config, method, code)
             expect_raise_error_call(client.config, method, url_for_path(path), mock_response, nil)
             expect { client.patch(path) }.to raise_error(QuizApiClient::HttpClient::RequestFailed)
           end
@@ -326,7 +326,7 @@ describe QuizApiClient::HttpClient do
           it 'raises error' do
             expect(client).to receive(:successful_response?).with(mock_response).and_return(false)
             expect(client.class).to receive(:delete).and_return(mock_response)
-            expect_metrics_calls(client.config, method, url_for_path(path), code)
+            expect_metrics_calls(client.config, method, code)
             expect_raise_error_call(client.config, method, url_for_path(path), mock_response, nil)
             expect { client.delete(path) }.to raise_error(QuizApiClient::HttpClient::RequestFailed)
           end
@@ -348,21 +348,21 @@ describe QuizApiClient::HttpClient do
 
       it 'handles an HTTParty::Error error' do
         stub_request(:get, url).to_raise(HTTParty::Error)
-        expect_metrics_calls(client.config, :get, url_for_path(path), 0)
+        expect_metrics_calls(client.config, :get, 0)
         expect_raise_error_call(client.config, :get, url_for_path(path), nil, kind_of(HTTParty::Error))
         expect { client.get('/') }.to raise_error(QuizApiClient::HttpClient::RequestFailed)
       end
 
       it 'handles an Errno::ECONNREFUSED error' do
         stub_request(:get, url).to_raise(Errno::ECONNREFUSED)
-        expect_metrics_calls(client.config, :get, url_for_path(path), 0)
+        expect_metrics_calls(client.config, :get, 0)
         expect_raise_error_call(client.config, :get, url_for_path(path), nil, kind_of(Errno::ECONNREFUSED))
         expect { client.get('/') }.to raise_error(QuizApiClient::HttpClient::RequestFailed)
       end
 
       it 'handles an Net::ReadTimeout error' do
         stub_request(:get, url).to_raise(Net::ReadTimeout)
-        expect_metrics_calls(client.config, :get, url_for_path(path), 0)
+        expect_metrics_calls(client.config, :get, 0)
         expect_raise_error_call(client.config, :get, url_for_path(path), nil, kind_of(Net::ReadTimeout))
         expect { client.get('/') }.to raise_error(QuizApiClient::HttpClient::RequestFailed)
       end

data/spec/quiz_api_client/http_request/metrics_spec.rb

@@ -3,10 +3,9 @@ require 'inst_statsd'
 describe QuizApiClient::HttpRequest::Metrics do
   let(:config) { QuizApiClient::Config.new }
   let(:method) { 'GET' }
-  let(:url) { 'https://something.com' }
   let(:code) { 200 }
 
-  subject { described_class.new(config, method, url, code) }
+  subject { described_class.new(config, method, code) }
 
   describe '#increment' do
     it 'does nothing when the metrics handler is nil' do
@@ -29,8 +28,7 @@ describe QuizApiClient::HttpRequest::Metrics do
         'fake-namespace.quiz_api_client.request.count',
         tags: {
           method: method,
-          status: code,
-          url: url
+          status: code
         }
       )
 
@@ -64,8 +62,7 @@ describe QuizApiClient::HttpRequest::Metrics do
         duration_in_secs * 1_000,
         tags: {
           method: method,
-          status: code,
-          url: url
+          status: code
         }
       )
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: quiz_api_client
 version: !ruby/object:Gem::Version
-  version: 4.14.0
+  version: 4.14.1
 platform: ruby
 authors:
 - Alex Slaughter
@@ -15,7 +15,7 @@ authors:
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2023-10-23 00:00:00.000000000 Z
+date: 2024-02-22 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: httparty