address code review

Author: kmalyjur

app/controllers/api/v2/preupgrade_report_entries_controller.rb

@@ -32,7 +32,7 @@ def index
       param :excluded_ids, Array, required: false, desc: N_('Array of excluded entry IDs')
       def bulk_remediate
         entries = filtered_remediation_entries
-        remediation_ids = entries.pluck(:id).sort
+        remediation_ids = entries.pluck(:id)
 
         if remediation_ids.empty?
           return render json: { error: _('No fixable entries found matching the selection.') },
@@ -91,9 +91,7 @@ def filtered_remediation_entries
 
       def target_host_ids(entries)
         host_ids = entries.pluck(:host_id).uniq.compact
-
         host_ids = [@preupgrade_report.host_id].compact if host_ids.empty? && @preupgrade_report
-
         host_ids
       end
 

test/functional/api/v2/preupgrade_report_entries_controller_test.rb

@@ -214,7 +214,7 @@ class PreupgradeReportEntriesControllerTest < ActionController::TestCase
         end
 
         test 'triggers job invocation for all fixable entries' do
-          expected_ids = [@entry1.id, @entry2.id, @entry3.id].sort.join(',')
+          expected_ids = [@entry1.id, @entry2.id, @entry3.id].join(',')
 
           JobInvocationComposer.expects(:for_feature)
                                .with('leapp_remediation_plan', [@host.id], { 'remediation_ids' => expected_ids })
@@ -227,7 +227,7 @@ class PreupgradeReportEntriesControllerTest < ActionController::TestCase
         end
 
         test 'respects the excluded_ids parameter' do
-          expected_ids = [@entry1.id, @entry3.id].sort.join(',')
+          expected_ids = [@entry1.id, @entry3.id].join(',')
 
           JobInvocationComposer.expects(:for_feature)
                                .with('leapp_remediation_plan', [@host.id], { 'remediation_ids' => expected_ids })