Refs: #4629@1h; Implementing CR + missing worker spec

5 files changed, 24 insertions, 6 deletions

diff --git a/spec/lib/af83/cloning/compliance_control_set_copier_spec.rb b/spec/lib/compliance_control_set_copier_spec.rb
index aacd9d7c9..b620d0657 100644
--- a/spec/lib/af83/cloning/compliance_control_set_copier_spec.rb
+++ b/spec/lib/compliance_control_set_copier_spec.rb
@@ -1,4 +1,4 @@
-RSpec.describe AF83::ComplianceControlSetCopier do
+RSpec.describe ComplianceControlSetCopier do
 
   subject{ described_class.new }
 
diff --git a/spec/models/compliance_check_spec.rb b/spec/models/compliance_check_spec.rb
index acdcc3ebf..710cd4b4f 100644
--- a/spec/models/compliance_check_spec.rb
+++ b/spec/models/compliance_check_spec.rb
@@ -1,5 +1,3 @@
-require 'rails_helper'
-
 RSpec.describe ComplianceCheck, type: :model do
   it 'should have a valid factory' do
     expect(FactoryGirl.build(:compliance_check)).to be_valid
diff --git a/spec/models/compliance_control_spec.rb b/spec/models/compliance_control_spec.rb
index 7c58eaf3e..db73dab21 100644
--- a/spec/models/compliance_control_spec.rb
+++ b/spec/models/compliance_control_spec.rb
@@ -43,9 +43,17 @@ RSpec.describe ComplianceControl, type: :model do
         compliance_control_block_id: compliance_control_block.id,
         compliance_control_set_id: create( :compliance_control_set ).id
       expect(compliance_control).to_not be_valid
+
+      direct_name      = compliance_control.compliance_control_set.name
+      indirect_name    = compliance_control_block.compliance_control_set.name
+      expected_message = "Le contrôle ne peut pas être associé à un jeu de contrôle (id: #{direct_name}) différent de celui de son groupe (id: #{indirect_name})"
+
       selected_error_message =
-        compliance_control.errors.messages[:coherent_control_set].grep(%r{ControlSet associé})
-      expect( selected_error_message ).to_not be_empty
+        compliance_control
+          .errors
+          .messages[:coherent_control_set]
+          .first
+      expect( selected_error_message ).to eq(expected_message)
     end
   end
 end
diff --git a/spec/workers/compliance_control_set_copy_worker_spec.rb b/spec/workers/compliance_control_set_copy_worker_spec.rb
new file mode 100644
index 000000000..3456ad398
--- /dev/null
+++ b/spec/workers/compliance_control_set_copy_worker_spec.rb
@@ -0,0 +1,13 @@
+RSpec.describe ComplianceControlSetCopyWorker do
+  let( :copier ){ ComplianceControlSetCopier }
+
+  let( :compliance_control_set_id ){ double('compliance_control_set_id') }
+  let( :referential_id ){ double('referential_id') }
+  
+  before do
+    expect_any_instance_of( copier ).to receive(:copy).with(compliance_control_set_id, referential_id)
+  end
+  it 'delegates to ComplianceControlSetCopier' do
+    described_class.new.perform(compliance_control_set_id, referential_id)
+  end
+end
diff --git a/spec/workers/line_referential_sync_worker_spec.rb b/spec/workers/line_referential_sync_worker_spec.rb
index f1a63c9db..f8d7eed91 100644
--- a/spec/workers/line_referential_sync_worker_spec.rb
+++ b/spec/workers/line_referential_sync_worker_spec.rb
@@ -1,4 +1,3 @@
-require 'rails_helper'
 RSpec.describe LineReferentialSyncWorker, type: :worker do
   let!(:line_referential_sync) { create :line_referential_sync }