foreman_statistics 2.0.0 → 2.0.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: ebe6e6bb3aad64251f7d191f3bc2cfcd6adca51bdffcb85901957a9074cff7e3
-  data.tar.gz: d4f425260d58b12cc0888a1b20cea07a4294f1a19cdcd0979ca6a7f2b4927797
+  metadata.gz: 755640806f3a97794bb4b955ebb99ed37c522ad48683772b7ab457d45cba3b3b
+  data.tar.gz: 7360f71bb358f1c13706bcdacd16e9fb7f36e62dbb6e9314e8cb85de55e034fc
 SHA512:
-  metadata.gz: 379d5362616632051dc1e6ac7e9b0919898cf59e5d90ddc4f832876469a6c52ae25a2b4e8aa8f71c409d969b873739342c517b53ad28b286fe6568beeae041af
-  data.tar.gz: 255b4e780195c26a59bdf7fff704df68a6ee0bdea94b0859857d91f5147562490eefa97032724bc14a3c6e2ba862574970ae71360573aa7a422e0c1f8c73d26f
+  metadata.gz: 83f1c16448f74158231ec49156f31c1dcb9f595485b938ae0986e1930e554b7ec7a7832f2e52a8661e80d7834beccb5c282626497ff2ee99266379b9b615f19b
+  data.tar.gz: 920cdba7dbdf1df8204485f3b2b5519d98816ac8468efa587aa307618ec49460b5dbdad7c70a6673c738fc48b1743e1d3cefc40097050f9982646090cd390e25

data/app/services/foreman_statistics/statistics/count_hosts.rb

@@ -20,11 +20,12 @@ module ForemanStatistics
 
         output = []
         data = scope.reorder('').group(grouping).count
-        associations = klass.find_by(id: data.keys)
+        associations = klass.where(id: data.keys).to_a
         data.each do |k, v|
           output << { label: associations.detect { |a| a.id == k }.to_label, data: v } unless v.zero?
-        rescue StandardError
-          logger.info "skipped #{k} as it has has no label"
+        rescue StandardError => e
+          Rails.logger.info "skipped '#{association} - #{k}' as it has has no label"
+          Rails.logger.debug e
         end
         output
       end

data/lib/foreman_statistics/version.rb

@@ -1,3 +1,3 @@
 module ForemanStatistics
-  VERSION = '2.0.0'.freeze
+  VERSION = '2.0.1'.freeze
 end

data/test/unit/foreman_statistics/statistics_test.rb

@@ -50,9 +50,19 @@ module ForemanStatistics
       assert_kind_of Array, stat.calculate
     end
 
-    test 'it should initialize a host counter statistics object' do
-      stat = Statistics::CountHosts.new(:count_by => :compute_resource)
-      assert_kind_of Array, stat.calculate
+    describe 'CountHosts' do
+      test 'initializes a host counter statistics object' do
+        stat = Statistics::CountHosts.new(:count_by => :compute_resource)
+        assert_kind_of Array, stat.calculate
+      end
+
+      test 'calculates stats by given metric' do
+        host = FactoryBot.create(:host, :with_operatingsystem)
+        stat = Statistics::CountHosts.new(count_by: :operatingsystem, title: 'OS Distribution').calculate
+        _(stat).must_be_instance_of(Array)
+        _(stat.first[:data]).must_equal(1)
+        _(stat.first[:label]).must_equal(host.operatingsystem.to_label)
+      end
     end
 
     context 'with puppet plugin' do

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: foreman_statistics
 version: !ruby/object:Gem::Version
-  version: 2.0.0
+  version: 2.0.1
 platform: ruby
 authors:
 - Ondrej Ezr
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-10-28 00:00:00.000000000 Z
+date: 2022-02-14 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rdoc
@@ -221,7 +221,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.1.6
+rubygems_version: 3.3.4
 signing_key:
 specification_version: 4
 summary: Add Statistics and Trends.