Added changes to CHANGELOG.md

CHANGELOG.md

@@ -1,5 +1,9 @@
 ## HEAD
 
+## v1.6.0
+* 11.02.2021: Add a new endpoint (/sidekiq/api/statistic_by_last_job_status.json) *Rhian Moraes*
+* 11.02.2021: /sidekiq/statistic: add two checkboxes to show/hide workers based on their last job status (#113) *Rhian Moraes*
+
 ## v1.5.1
 
 * 06.02.2021: Generate new TAG to fix "version.rb" not updated in the previous one (#170) *Wender Freese*

lib/sidekiq/statistic/statistic/workers.rb

@@ -18,14 +18,12 @@ def display
       end
 
       def display_by_last_status
-        {
-          passed: filter_last_job_status('passed'),
-          failed: filter_last_job_status('failed')
-        }
-      end
+        filtered_workers = display.group_by {|worker| worker[:last_job_status] }
+
+        filtered_workers['passed'] ||= []
+        filtered_workers['failed'] ||= []
 
-      def filter_last_job_status(status)
-        display.select { |worker| worker[:last_job_status] == status }
+        filtered_workers.transform_keys(&:to_sym)
       end
 
       def display_per_day(worker_name)

test/test_sidekiq/statistic_test.rb

@@ -123,17 +123,6 @@ module Statistic
         end
       end
 
-      describe "#filter_last_job_status" do
-        it 'return array with worker when filtering by passed workers' do
-          middlewared {}
-
-          subject = statistic.filter_last_job_status('passed')
-
-          _(subject).must_be_instance_of Array
-          assert_equal subject[0][:name], worker
-        end
-      end
-
       describe '#display_per_day' do
         it 'return workers job per day' do
           middlewared {}