foreman_openscap 0.8.6 → 0.9.0

data/db/migrate/20141118142954_add_constraint_to_scaptimony_policies.rb

@@ -1,4 +1,4 @@
-class AddConstraintToScaptimonyPolicies < ActiveRecord::Migration
+class AddConstraintToScaptimonyPolicies < ActiveRecord::Migration[4.2]
   def change
     change_column :scaptimony_policies, :name, :string, :null => false
   end

data/db/migrate/20141119164918_create_scaptimony_xccdf_results.rb

@@ -1,4 +1,4 @@
-class CreateScaptimonyXccdfResults < ActiveRecord::Migration
+class CreateScaptimonyXccdfResults < ActiveRecord::Migration[4.2]
   def change
     create_table :scaptimony_xccdf_results do |t|
       t.string :name, :limit => 16, :null => false

data/db/migrate/20141119175434_create_scaptimony_xccdf_rules.rb

@@ -1,4 +1,4 @@
-class CreateScaptimonyXccdfRules < ActiveRecord::Migration
+class CreateScaptimonyXccdfRules < ActiveRecord::Migration[4.2]
   def change
     create_table :scaptimony_xccdf_rules do |t|
       t.string :xid, :null => false

data/db/migrate/20141119182606_create_scaptimony_xccdf_rule_results.rb

@@ -1,4 +1,4 @@
-class CreateScaptimonyXccdfRuleResults < ActiveRecord::Migration
+class CreateScaptimonyXccdfRuleResults < ActiveRecord::Migration[4.2]
   def change
     create_table :scaptimony_xccdf_rule_results do |t|
       t.references :arf_report, :index => true, :null => false

data/db/migrate/20141121120326_create_scaptimony_arf_report_breakdowns.rb

@@ -1,4 +1,4 @@
-class CreateScaptimonyArfReportBreakdowns < ActiveRecord::Migration
+class CreateScaptimonyArfReportBreakdowns < ActiveRecord::Migration[4.2]
   def self.up
     execute <<-SQL.strip_heredoc
       CREATE VIEW scaptimony_arf_report_breakdowns AS

data/db/migrate/20141121164042_replace_arf_report_breakdown_view.rb

@@ -1,4 +1,4 @@
-class ReplaceArfReportBreakdownView < ActiveRecord::Migration
+class ReplaceArfReportBreakdownView < ActiveRecord::Migration[4.2]
   def self.up
     execute 'DROP VIEW IF EXISTS scaptimony_arf_report_breakdowns'
     execute <<-SQL.strip_heredoc

data/db/migrate/20141206211151_create_scaptimony_assets_policies.rb

@@ -1,4 +1,4 @@
-class CreateScaptimonyAssetsPolicies < ActiveRecord::Migration
+class CreateScaptimonyAssetsPolicies < ActiveRecord::Migration[4.2]
   def change
     create_table :scaptimony_assets_policies, :id => false do |t|
       t.references :asset, :index => true, :null => false

data/db/migrate/20141214112917_add_scap_file_to_scap_content.rb

@@ -1,4 +1,4 @@
-class AddScapFileToScapContent < ActiveRecord::Migration
+class AddScapFileToScapContent < ActiveRecord::Migration[4.2]
   def change
     add_column :scaptimony_scap_contents, :scap_file, :binary
   end

data/db/migrate/20141216154502_rename_scaptimony_asset_policies.rb

@@ -1,4 +1,4 @@
-class RenameScaptimonyAssetPolicies < ActiveRecord::Migration
+class RenameScaptimonyAssetPolicies < ActiveRecord::Migration[4.2]
   def change
     rename_table(:scaptimony_assets_policies, :scaptimony_asset_policies)
   end

data/db/migrate/20150111085317_polymorph_asset.rb

@@ -1,4 +1,4 @@
-class PolymorphAsset < ActiveRecord::Migration
+class PolymorphAsset < ActiveRecord::Migration[4.2]
   def change
     change_table(:scaptimony_assets) do |t|
       t.references :assetable, :polymorphic => true, :index => true

data/db/migrate/20150112152944_create_scaptimony_arf_report_raws.rb

@@ -1,4 +1,4 @@
-class CreateScaptimonyArfReportRaws < ActiveRecord::Migration
+class CreateScaptimonyArfReportRaws < ActiveRecord::Migration[4.2]
   def change
     create_table :scaptimony_arf_report_raws, :id => false do |t|
       t.references :arf_report, :null => false

data/db/migrate/20150114210634_rename_scaptimony_arf_report_raw_raw.rb

@@ -1,4 +1,4 @@
-class RenameScaptimonyArfReportRawRaw < ActiveRecord::Migration
+class RenameScaptimonyArfReportRawRaw < ActiveRecord::Migration[4.2]
   def change
     rename_column :scaptimony_arf_report_raws, :raw, :bzip_data
   end

data/db/migrate/20150115155947_add_scaptimony_scap_content_digest.rb

@@ -1,6 +1,6 @@
 require 'digest/sha2'
 
-class AddScaptimonyScapContentDigest < ActiveRecord::Migration
+class AddScaptimonyScapContentDigest < ActiveRecord::Migration[4.2]
   def change
     unless column_exists?(:scaptimony_scap_contents, :digest)
       add_column :scaptimony_scap_contents, :digest, :string, :limit => 128

data/db/migrate/20150116083129_add_day_of_month_and_cron_line_to_scaptimony_policy.rb

@@ -1,4 +1,4 @@
-class AddDayOfMonthAndCronLineToScaptimonyPolicy < ActiveRecord::Migration
+class AddDayOfMonthAndCronLineToScaptimonyPolicy < ActiveRecord::Migration[4.2]
   def change
     add_column :scaptimony_policies, :day_of_month, :integer
     add_column :scaptimony_policies, :cron_line, :string

data/db/migrate/20150821100137_migrate_from_scaptimony.rb

@@ -1,4 +1,4 @@
-class MigrateFromScaptimony < ActiveRecord::Migration
+class MigrateFromScaptimony < ActiveRecord::Migration[4.2]
   def up
     ActiveRecord::ConnectionAdapters::SchemaStatements.module_eval do
       # rename_tables renames the indexes, and their new names overflow, we cancel out the renaming of the indexes

data/db/migrate/20150827123826_remove_scaptimony_permissions.rb

@@ -1,4 +1,4 @@
-class RemoveScaptimonyPermissions < ActiveRecord::Migration
+class RemoveScaptimonyPermissions < ActiveRecord::Migration[4.2]
   def up
     permissions = Permission.where(:resource_type => ["Scaptimony::Policy", "Scaptimony::ScapContent"])
     new_type = "ForemanOpenscap"

data/db/migrate/20150925124959_create_policy_arf_reports.rb

@@ -1,4 +1,4 @@
-class CreatePolicyArfReports < ActiveRecord::Migration
+class CreatePolicyArfReports < ActiveRecord::Migration[4.2]
   def up
     create_table :foreman_openscap_policy_arf_reports do |t|
       t.integer :policy_id

data/db/migrate/20150929124853_add_result_to_logs.rb

@@ -1,4 +1,4 @@
-class AddResultToLogs < ActiveRecord::Migration
+class AddResultToLogs < ActiveRecord::Migration[4.2]
   def up
     add_column :logs, :result, :string
   end

data/db/migrate/20150929152345_move_arf_reports_to_reports_table.rb

@@ -1,8 +1,8 @@
-class MoveArfReportsToReportsTable < ActiveRecord::Migration
+class MoveArfReportsToReportsTable < ActiveRecord::Migration[4.2]
   # rubocop:disable Metrics/MethodLength
   # rubocop:disable Metrics/AbcSize
   def up
-    execute 'DROP VIEW foreman_openscap_arf_report_breakdowns' if table_exists? 'foreman_openscap_arf_report_breakdowns'
+    execute 'DROP VIEW foreman_openscap_arf_report_breakdowns' if view_exists? 'foreman_openscap_arf_report_breakdowns'
     drop_table :foreman_openscap_xccdf_results
     drop_table :foreman_openscap_xccdf_rules
     drop_table :foreman_openscap_xccdf_rule_results

data/db/migrate/20151118165125_add_size_to_scap_content.rb

@@ -1,4 +1,4 @@
-class AddSizeToScapContent < ActiveRecord::Migration
+class AddSizeToScapContent < ActiveRecord::Migration[4.2]
   def change
     change_column :foreman_openscap_scap_contents, :scap_file, :binary, :limit => 16.megabyte
   end

data/db/migrate/20151119155419_add_arf_fields_to_message.rb

@@ -1,4 +1,4 @@
-class AddArfFieldsToMessage < ActiveRecord::Migration
+class AddArfFieldsToMessage < ActiveRecord::Migration[4.2]
   def change
     add_column :messages, :description, :text
     add_column :messages, :rationale, :text

data/db/migrate/20151120090851_add_openscap_proxy_to_host_and_hostgroup.rb

@@ -1,4 +1,4 @@
-class AddOpenscapProxyToHostAndHostgroup < ActiveRecord::Migration
+class AddOpenscapProxyToHostAndHostgroup < ActiveRecord::Migration[4.2]
   def up
     add_column :hostgroups, :openscap_proxy_id, :integer
     add_column :hosts, :openscap_proxy_id, :integer

data/db/migrate/20160830113437_remove_deleted_policy.rb

@@ -1,4 +1,4 @@
-class RemoveDeletedPolicy < ActiveRecord::Migration
+class RemoveDeletedPolicy < ActiveRecord::Migration[4.2]
   def up
     ForemanOpenscap::AssetPolicy.all.collect(&:policy_id).uniq.each do |policy_id|
       execute("DELETE FROM foreman_openscap_asset_policies WHERE policy_id = '#{policy_id}';") if ForemanOpenscap::Policy.unscoped.find_by(id: policy_id).nil?

data/db/migrate/20160925213031_change_scap_widget_names.rb

@@ -1,4 +1,4 @@
-class ChangeScapWidgetNames < ActiveRecord::Migration
+class ChangeScapWidgetNames < ActiveRecord::Migration[4.2]
   def up
     Widget.where(:name => 'OpenSCAP Host reports widget')\
           .update_all(:name => 'Latest Compliance Reports')

data/db/migrate/20161109155255_create_tailoring_files.rb

@@ -1,4 +1,4 @@
-class CreateTailoringFiles < ActiveRecord::Migration
+class CreateTailoringFiles < ActiveRecord::Migration[4.2]
   def up
     create_table :foreman_openscap_tailoring_files do |t|
       t.string :name, :unique => true, :null => false

data/db/migrate/20161223153249_add_permissions_to_arf_report.rb

@@ -1,4 +1,4 @@
-class AddPermissionsToArfReport < ActiveRecord::Migration
+class AddPermissionsToArfReport < ActiveRecord::Migration[4.2]
   def up
     Permission.where(:name => %w[view_arf_reports destroy_arf_reports])
               .update_all(:resource_type => 'ForemanOpenscap::ArfReport')

data/db/migrate/20170821081205_rename_mail_notification.foreman_openscap.rb

@@ -1,4 +1,4 @@
-class RenameMailNotification < ActiveRecord::Migration
+class RenameMailNotification < ActiveRecord::Migration[4.2]
   def up
     notification = MailNotification.where(:name => 'openscap_policy_summary').first
     if notification

data/db/migrate/20170830221751_add_index_to_logs_result.rb

@@ -1,4 +1,4 @@
-class AddIndexToLogsResult < ActiveRecord::Migration
+class AddIndexToLogsResult < ActiveRecord::Migration[4.2]
   def up
     add_index :logs, :result
   end

data/db/migrate/20171011134112_remove_arf_reports_without_policy.rb

@@ -1,15 +1,5 @@
-class RemoveArfReportsWithoutPolicy < ActiveRecord::Migration
+class RemoveArfReportsWithoutPolicy < ActiveRecord::Migration[4.2]
   def up
-    if User.unscoped.find_by(:login => User::ANONYMOUS_ADMIN)
-      User.as_anonymous_admin do
-        delete_reports
-      end
-    else
-      delete_reports
-    end
-  end
-
-  def delete_reports
     ids_to_keep = ForemanOpenscap::ArfReport.unscoped.all.joins(:policy_arf_report).pluck(:id)
     ForemanOpenscap::ArfReport.unscoped.where.not(:id => ids_to_keep).find_in_batches do |batch|
       batch.map(&:destroy!)

data/db/migrate/20171016125613_add_content_title_unique_constraint.foreman_openscap.rb

@@ -1,4 +1,4 @@
-class AddContentTitleUniqueConstraint < ActiveRecord::Migration
+class AddContentTitleUniqueConstraint < ActiveRecord::Migration[4.2]
   def change
     remove_index :foreman_openscap_scap_contents, :name => 'index_scaptimony_scap_contents_on_title'
     add_index :foreman_openscap_scap_contents, :title, :unique => true

data/lib/foreman_openscap/engine.rb

@@ -42,7 +42,7 @@ module ForemanOpenscap
 
     initializer 'foreman_openscap.register_plugin', :before => :finisher_hook do |app|
       Foreman::Plugin.register :foreman_openscap do
-        requires_foreman '>= 1.16'
+        requires_foreman '>= 1.17'
 
         apipie_documented_controllers ["#{ForemanOpenscap::Engine.root}/app/controllers/api/v2/compliance/*.rb"]
 
@@ -199,16 +199,16 @@ module ForemanOpenscap
     config.to_prepare do
       Host::Managed.send(:include, ForemanOpenscap::OpenscapProxyExtensions)
       Host::Managed.send(:include, ForemanOpenscap::OpenscapProxyCoreExtensions)
-      Host::Managed.send(:include, ForemanOpenscap::HostExtensions)
-      HostsHelper.send(:include, ForemanOpenscap::HostsHelperExtensions)
+      Host::Managed.send(:prepend, ForemanOpenscap::HostExtensions)
+      HostsHelper.send(:prepend, ForemanOpenscap::HostsHelperExtensions)
       Hostgroup.send(:include, ForemanOpenscap::OpenscapProxyExtensions)
       Hostgroup.send(:include, ForemanOpenscap::OpenscapProxyCoreExtensions)
       Hostgroup.send(:include, ForemanOpenscap::HostgroupExtensions)
       ForemanOpenscap::ArfReport.send(:include, ForemanOpenscap::ComplianceStatusScopedSearch)
       SmartProxy.send(:include, ForemanOpenscap::SmartProxyExtensions)
-      HostsController.send(:include, ForemanOpenscap::HostsControllerExtensions)
+      HostsController.send(:prepend, ForemanOpenscap::HostsControllerExtensions)
       Log.send(:include, ForemanOpenscap::LogExtensions)
-      LookupKeysHelper.send(:include, ForemanOpenscap::LookupKeysHelperExtensions)
+      LookupKeysHelper.send(:prepend, ForemanOpenscap::LookupKeysHelperExtensions)
     end
 
     rake_tasks do

data/lib/foreman_openscap/version.rb

@@ -1,3 +1,3 @@
 module ForemanOpenscap
-  VERSION = "0.8.6".freeze
+  VERSION = "0.9.0".freeze
 end

data/test/factories/arf_report_factory.rb

@@ -7,7 +7,7 @@ FactoryBot.define do
     sequence(:created_at) { |n| Time.new(1490 + n, 0o1, 13, 15, 24, 0o0) }
     logs []
     status 0
-    metrics {}
+    metrics({})
     openscap_proxy nil
   end
 end

data/test/functional/api/v2/compliance/arf_reports_controller_test.rb

@@ -18,7 +18,7 @@ class Api::V2::Compliance::ArfReportsControllerTest < ActionController::TestCase
 
   test "should get index" do
     create_arf_report
-    get :index, {}, set_session_user
+    get :index, :session => set_session_user
     response = ActiveSupport::JSON.decode(@response.body)
     assert_not response['results'].empty?
     assert_response :success
@@ -26,7 +26,7 @@ class Api::V2::Compliance::ArfReportsControllerTest < ActionController::TestCase
 
   test "should get show" do
     report = create_arf_report
-    get :show, { :id => report.to_param }, set_session_user
+    get :show, :params => { :id => report.to_param }, :session => set_session_user
     response = ActiveSupport::JSON.decode(@response.body)
     refute response['passed'].blank?
     refute response['failed'].blank?
@@ -38,7 +38,7 @@ class Api::V2::Compliance::ArfReportsControllerTest < ActionController::TestCase
     report = create_arf_report
     bzipped_report = File.read "#{ForemanOpenscap::Engine.root}/test/files/arf_report/arf_report.bz2"
     ForemanOpenscap::ArfReport.any_instance.stubs(:to_bzip).returns(bzipped_report)
-    get :download, { :id => report.to_param }, set_session_user
+    get :download, :params => { :id => report.to_param }, :session => set_session_user
     t = Tempfile.new('tmp_report')
     t.write @response.body
     t.close
@@ -50,10 +50,10 @@ class Api::V2::Compliance::ArfReportsControllerTest < ActionController::TestCase
     date = Time.new(1984, 9, 15)
     ForemanOpenscap::Helper.stubs(:get_asset).returns(@asset)
     post :create,
-         @from_json.merge(:cname => @cname,
-                          :policy_id => @policy.id,
-                          :date => date.to_i),
-         set_session_user
+         :params => @from_json.merge(:cname => @cname,
+                                     :policy_id => @policy.id,
+                                     :date => date.to_i),
+         :session => set_session_user
     report = ForemanOpenscap::ArfReport.unscoped.last
     assert_equal date, report.reported_at
     report_logs = report.logs
@@ -68,10 +68,10 @@ class Api::V2::Compliance::ArfReportsControllerTest < ActionController::TestCase
     date = Time.new(1944, 6, 6)
     ForemanOpenscap::Helper.stubs(:get_asset).returns(asset)
     post :create,
-         @from_json.merge(:cname => @cname,
-                          :policy_id => @policy.id,
-                          :date => date.to_i),
-         set_session_user
+         :params => @from_json.merge(:cname => @cname,
+                                     :policy_id => @policy.id,
+                                     :date => date.to_i),
+         :session => set_session_user
     assert_response :unprocessable_entity
     res = JSON.parse(@response.body)
     assert_equal "Failed to upload Arf Report, no OpenSCAP proxy set for host #{asset.host.name}", res["result"]
@@ -86,10 +86,10 @@ class Api::V2::Compliance::ArfReportsControllerTest < ActionController::TestCase
 
     ForemanOpenscap::Helper.stubs(:get_asset).returns(@asset)
     post :create,
-         @from_json.merge(:cname => @cname,
-                          :policy_id => @policy.id,
-                          :date => dates[1].to_i),
-         set_session_user
+         :params => @from_json.merge(:cname => @cname,
+                                     :policy_id => @policy.id,
+                                     :date => dates[1].to_i),
+         :session => set_session_user
     assert_equal Message.where(:digest => ForemanOpenscap::ArfReport.unscoped.last.logs.first.message.digest).count, 1
   end
 
@@ -102,10 +102,10 @@ class Api::V2::Compliance::ArfReportsControllerTest < ActionController::TestCase
     ForemanOpenscap::Helper.stubs(:get_asset).returns(@asset)
     changed_from_json = arf_from_json "#{ForemanOpenscap::Engine.root}/test/files/arf_report/arf_report_msg_desc_changed.json"
     post :create,
-         changed_from_json.merge(:cname => @cname,
-                                 :policy_id => @policy.id,
-                                 :date => Time.new(2017, 6, 6).to_i),
-         set_session_user
+         :params => changed_from_json.merge(:cname => @cname,
+                                            :policy_id => @policy.id,
+                                            :date => Time.new(2017, 6, 6).to_i),
+         :session => set_session_user
 
     assert_response :success
 
@@ -127,10 +127,10 @@ class Api::V2::Compliance::ArfReportsControllerTest < ActionController::TestCase
     ForemanOpenscap::Helper.stubs(:get_asset).returns(@asset)
     changed_from_json = arf_from_json "#{ForemanOpenscap::Engine.root}/test/files/arf_report/arf_report_msg_value_changed.json"
     post :create,
-         changed_from_json.merge(:cname => @cname,
+         :params => changed_from_json.merge(:cname => @cname,
                                  :policy_id => @policy.id,
                                  :date => Time.new(2017, 8, 6).to_i),
-         set_session_user
+         :session => set_session_user
 
     assert_response :success
 

data/test/functional/api/v2/compliance/policies_controller_test.rb

@@ -13,7 +13,7 @@ class Api::V2::Compliance::PoliciesControllerTest < ActionController::TestCase
 
   test "should get index" do
     FactoryBot.create(:policy)
-    get :index, {}, set_session_user
+    get :index, :session => set_session_user
     response = ActiveSupport::JSON.decode(@response.body)
     assert !response['results'].empty?
     assert_response :success
@@ -21,7 +21,7 @@ class Api::V2::Compliance::PoliciesControllerTest < ActionController::TestCase
 
   test "should show a policy" do
     policy = FactoryBot.create(:policy)
-    get :show, { :id => policy.to_param }, set_session_user
+    get :show, :params => { :id => policy.to_param }, :session => set_session_user
     response = ActiveSupport::JSON.decode(@response.body)
     assert response['name'], policy.name
     assert_response :success
@@ -29,7 +29,7 @@ class Api::V2::Compliance::PoliciesControllerTest < ActionController::TestCase
 
   test "should update a policy" do
     policy = FactoryBot.create(:policy)
-    put :update, { :id => policy.id, :policy => { :period => 'monthly', :day_of_month => 15 } }
+    put :update, :params => { :id => policy.id, :policy => { :period => 'monthly', :day_of_month => 15 } }
     updated_policy = ActiveSupport::JSON.decode(@response.body)
     assert(updated_policy['period'], 'monthly')
     assert_response :ok
@@ -37,12 +37,12 @@ class Api::V2::Compliance::PoliciesControllerTest < ActionController::TestCase
 
   test "should not update invalid" do
     policy = FactoryBot.create(:policy)
-    put :update, { :id => policy.id, :policy => { :name => '' } }
+    put :update, :params => { :id => policy.id, :policy => { :name => '' } }
     assert_response :unprocessable_entity
   end
 
   test "should create a policy" do
-    post :create, @attributes, set_session_user
+    post :create, :params => @attributes, :session => set_session_user
     response = ActiveSupport::JSON.decode(@response.body)
     assert response['scap_content_profile_id'], @scap_content_profile.to_param
     assert_response :created
@@ -51,7 +51,7 @@ class Api::V2::Compliance::PoliciesControllerTest < ActionController::TestCase
   test "should not create a policy with tailoring file profile and without the actual file" do
     tailoring_profile = FactoryBot.create(:scap_content_profile, :profile_id => 'xccdf_org.test.tailoring_profile')
     @attributes[:policy][:tailoring_file_profile_id] = tailoring_profile.id
-    post :create, @attributes, set_session_user
+    post :create, :params => @attributes, :session => set_session_user
     response = ActiveSupport::JSON.decode(@response.body)
     assert_not_nil response['error']['errors']['tailoring_file_id']
     assert_response :unprocessable_entity
@@ -60,27 +60,27 @@ class Api::V2::Compliance::PoliciesControllerTest < ActionController::TestCase
   test "should not create a policy with tailoring file and without tailoring profile" do
     tailoring_file = FactoryBot.create(:tailoring_file)
     @attributes[:policy][:tailoring_file_id] = tailoring_file.id
-    post :create, @attributes, set_session_user
+    post :create, :params => @attributes, :session => set_session_user
     response = ActiveSupport::JSON.decode(@response.body)
     assert_not_nil response['error']['errors']['tailoring_file_profile_id']
     assert_response :unprocessable_entity
   end
 
   test "should not create invalid policy" do
-    post :create, {}, set_session_user
+    post :create, :session => set_session_user
     assert_response :unprocessable_entity
   end
 
   test "should destroy" do
     policy = FactoryBot.create(:policy)
-    delete :destroy, { :id => policy.id }, set_session_user
+    delete :destroy, :params => { :id => policy.id }, :session => set_session_user
     assert_response :ok
     refute ForemanOpenscap::Policy.exists?(policy.id)
   end
 
   test "should return xml of scap content" do
     policy = FactoryBot.create(:policy)
-    get :content, { :id => policy.id }, set_session_user
+    get :content, :params => { :id => policy.id }, :session => set_session_user
     assert(@response.header['Content-Type'], 'application/xml')
     assert_response :success
   end
@@ -89,14 +89,14 @@ class Api::V2::Compliance::PoliciesControllerTest < ActionController::TestCase
     tailoring_profile = FactoryBot.create(:scap_content_profile)
     policy = FactoryBot.create(:policy, :tailoring_file => FactoryBot.create(:tailoring_file, :scap_content_profiles => [tailoring_profile]),
                                         :tailoring_file_profile => tailoring_profile)
-    get :tailoring, { :id => policy.id }, set_session_user
+    get :tailoring, :params => { :id => policy.id }, :session => set_session_user
     assert(@response.header['Content-Type'], 'application/xml')
     assert_response :success
   end
 
   test "should return meaningufull error when no tailioring file assigned" do
     policy = FactoryBot.create(:policy)
-    get :tailoring, { :id => policy.id }, set_session_user
+    get :tailoring, :params => { :id => policy.id }, :session => set_session_user
     assert_response :not_found
     response = ActiveSupport::JSON.decode(@response.body)
     assert_equal "No Tailoring file assigned for policy with id #{policy.id}", response['error']['message']