topological_inventory-providers-common 2.0.0 → 2.1.4

data/spec/topological_inventory/providers/common/operations/processor_spec.rb

@@ -0,0 +1,71 @@
+require "topological_inventory/providers/common/operations/processor"
+
+RSpec.describe TopologicalInventory::Providers::Common::Operations::Processor do
+  let(:operation_name) { 'Source.availability_check' }
+  let(:params) { {'source_id' => 1, 'external_tenant' => '12345'} }
+  let(:payload) { {"params" => params, "request_context" => double('request_context')} }
+  let(:message) { double("ManageIQ::Messaging::ReceivedMessage", :message => operation_name, :payload => payload) }
+
+  subject { described_class.new(message, nil) }
+
+  describe "#process" do
+    it "starts the operation if class and method exists" do
+      result = double('result')
+
+      klass = TopologicalInventory::Providers::Common::Operations::Source
+      allow(subject).to receive(:operation_class).and_return(klass)
+
+      source = klass.new(params, payload['request_context'], nil)
+      expect(klass).to receive(:new).with(params, payload['request_context'], nil).and_return(source)
+      expect(source).to receive(:availability_check).and_return(result)
+
+      expect(subject.process).to eq(result)
+    end
+
+    it "returns 'not_implemented' if class of method doesn't exist" do
+      allow(subject).to receive(:operation_class).and_return(nil)
+      allow(subject).to receive(:method).and_return('awesome')
+
+      expect(subject.process).to eq(subject.operation_status[:not_implemented])
+    end
+
+    it "updates task if not_implemented" do
+      allow(subject).to receive(:operation_class).and_return(nil)
+      allow(subject).to receive(:method).and_return('awesome')
+
+      subject.send(:params)['task_id'] = '1'
+      expect(subject).to(receive(:update_task).with('1',
+                                                    :state   => "completed",
+                                                    :status  => "error",
+                                                    :context => anything))
+      subject.process
+    end
+
+    it "updates task if exception raised" do
+      subject.send(:params)['task_id'] = '1'
+      expect(subject).to(receive(:update_task).with('1',
+                                                    :state   => "completed",
+                                                    :status  => "error",
+                                                    :context => anything))
+      expect { subject.process }.to raise_exception(NotImplementedError)
+    end
+  end
+
+  describe "#with_time_measure" do
+    let(:metrics) { double("Metrics") }
+
+    it "records time and yields if metrics provided" do
+      allow(subject).to receive(:metrics).and_return(metrics)
+
+      expect(metrics).to receive(:record_operation_time).with(operation_name).and_yield
+
+      expect(subject.send(:with_time_measure) { 42 }).to eq(42)
+    end
+
+    it "only yields if metrics not present" do
+      expect(metrics).not_to receive(:record_operation_time)
+
+      expect(subject.send(:with_time_measure) { 42 }).to eq(42)
+    end
+  end
+end

data/topological_inventory-providers-common.gemspec

@@ -26,16 +26,20 @@ Gem::Specification.new do |spec|
   spec.add_runtime_dependency 'activesupport', '~> 5.2.4.3'
   spec.add_runtime_dependency 'config', '~> 1.7', '>= 1.7.2'
   spec.add_runtime_dependency 'json', '~> 2.3'
-  spec.add_runtime_dependency "manageiq-loggers", ">= 0.4.2"
-  spec.add_runtime_dependency "sources-api-client", "~> 3.0"
-  spec.add_runtime_dependency "topological_inventory-api-client", "~> 3.0", ">= 3.0.1"
-  spec.add_runtime_dependency "topological_inventory-ingress_api-client", "~> 1.0", ">= 1.0.3"
+  spec.add_runtime_dependency 'manageiq-loggers', '>= 0.4.2'
+  spec.add_runtime_dependency 'manageiq-messaging', '~> 1.0.0'
+  spec.add_runtime_dependency 'more_core_extensions'
+  spec.add_runtime_dependency 'prometheus_exporter', '~> 0.4.17'
+  spec.add_runtime_dependency 'sources-api-client', '~> 3.0'
+  spec.add_runtime_dependency 'topological_inventory-api-client', '~> 3.0', '>= 3.0.1'
+  spec.add_runtime_dependency 'topological_inventory-ingress_api-client', '~> 1.0', '>= 1.0.3'
 
-  spec.add_development_dependency "bundler", "~> 2.0"
-  spec.add_development_dependency "rake", ">= 12.3.3"
-  spec.add_development_dependency "rspec", "~> 3.0"
-  spec.add_development_dependency 'rubocop', '~>0.69.0'
-  spec.add_development_dependency 'rubocop-performance', '~>1.3'
-  spec.add_development_dependency "simplecov", "~> 0.17.1"
+  spec.add_development_dependency 'bundler', '~> 2.0'
+  spec.add_development_dependency 'rake', '>= 12.3.3'
+  spec.add_development_dependency 'rspec', '~> 3.0'
+  spec.add_development_dependency 'rubocop', '~> 1.0.0'
+  spec.add_development_dependency 'rubocop-performance', '~> 1.8'
+  spec.add_development_dependency 'rubocop-rails', '~> 2.8'
+  spec.add_development_dependency 'simplecov', '~> 0.17.1'
   spec.add_development_dependency 'webmock'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: topological_inventory-providers-common
 version: !ruby/object:Gem::Version
-  version: 2.0.0
+  version: 2.1.4
 platform: ruby
 authors:
 - Martin Slemr
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2020-10-20 00:00:00.000000000 Z
+date: 2020-12-14 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -72,6 +72,48 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: 0.4.2
+- !ruby/object:Gem::Dependency
+  name: manageiq-messaging
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 1.0.0
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 1.0.0
+- !ruby/object:Gem::Dependency
+  name: more_core_extensions
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: prometheus_exporter
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 0.4.17
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 0.4.17
 - !ruby/object:Gem::Dependency
   name: sources-api-client
   requirement: !ruby/object:Gem::Requirement
@@ -174,28 +216,42 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.69.0
+        version: 1.0.0
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.69.0
+        version: 1.0.0
 - !ruby/object:Gem::Dependency
   name: rubocop-performance
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.3'
+        version: '1.8'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.8'
+- !ruby/object:Gem::Dependency
+  name: rubocop-rails
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '2.8'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.3'
+        version: '2.8'
 - !ruby/object:Gem::Dependency
   name: simplecov
   requirement: !ruby/object:Gem::Requirement
@@ -231,6 +287,7 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
+- ".codeclimate.yml"
 - ".github/workflows/gem-push.yml"
 - ".gitignore"
 - ".rspec"
@@ -238,6 +295,7 @@ files:
 - ".rubocop_cc.yml"
 - ".rubocop_local.yml"
 - ".travis.yml"
+- ".yamllint"
 - CHANGELOG.md
 - Gemfile
 - LICENSE.txt
@@ -252,11 +310,15 @@ files:
 - lib/topological_inventory/providers/common/collector/parser.rb
 - lib/topological_inventory/providers/common/collectors_pool.rb
 - lib/topological_inventory/providers/common/logging.rb
+- lib/topological_inventory/providers/common/messaging_client.rb
+- lib/topological_inventory/providers/common/metrics.rb
 - lib/topological_inventory/providers/common/mixins/sources_api.rb
+- lib/topological_inventory/providers/common/mixins/statuses.rb
 - lib/topological_inventory/providers/common/mixins/topology_api.rb
 - lib/topological_inventory/providers/common/mixins/x_rh_headers.rb
 - lib/topological_inventory/providers/common/operations/async_worker.rb
 - lib/topological_inventory/providers/common/operations/health_check.rb
+- lib/topological_inventory/providers/common/operations/processor.rb
 - lib/topological_inventory/providers/common/operations/source.rb
 - lib/topological_inventory/providers/common/save_inventory/exception.rb
 - lib/topological_inventory/providers/common/save_inventory/saver.rb
@@ -272,6 +334,7 @@ files:
 - spec/topological_inventory/providers/common/collectors_pool_spec.rb
 - spec/topological_inventory/providers/common/logger_spec.rb
 - spec/topological_inventory/providers/common/operations/async_worker_spec.rb
+- spec/topological_inventory/providers/common/operations/processor_spec.rb
 - spec/topological_inventory/providers/common/operations/source_spec.rb
 - spec/topological_inventory/providers/common/save_inventory/saver_spec.rb
 - spec/topological_inventory/providers/common_spec.rb