Don't use `raise_error` without a parameter

4 files changed, 7 insertions, 6 deletions

diff --git a/spec/models/variable_spec.rb b/spec/models/variable_spec.rb
index 1ecb960..51ae2d2 100644
--- a/spec/models/variable_spec.rb
+++ b/spec/models/variable_spec.rb
@@ -38,7 +38,8 @@ describe Variable do
     it 'fails to decrypt if iv is incorrect' do
       subject.encrypted_value_iv = nil
       subject.instance_variable_set(:@value, nil)
-      expect { subject.value }.to raise_error
+      expect { subject.value }.
+        to raise_error(OpenSSL::Cipher::CipherError, 'bad decrypt')
     end
   end
 end
diff --git a/spec/models/web_hook_spec.rb b/spec/models/web_hook_spec.rb
index 0f0f175..e3fa31c 100644
--- a/spec/models/web_hook_spec.rb
+++ b/spec/models/web_hook_spec.rb
@@ -56,9 +56,8 @@ describe WebHook do
     it "catches exceptions" do
       WebHook.should_receive(:post).and_raise("Some HTTP Post error")
 
-      lambda {
-        @web_hook.execute(@data)
-      }.should raise_error
+      expect { @web_hook.execute(@data) }.
+        to raise_error(RuntimeError, 'Some HTTP Post error')
     end
   end
 end
diff --git a/spec/requests/api/projects_spec.rb b/spec/requests/api/projects_spec.rb
index c165aec..8a744c1 100644
--- a/spec/requests/api/projects_spec.rb
+++ b/spec/requests/api/projects_spec.rb
@@ -147,7 +147,7 @@ describe API::API do
       delete api("/projects/#{project.id}"), options
       response.status.should == 200
 
-      expect { project.reload }.to raise_error
+      expect { project.reload }.to raise_error(ActiveRecord::RecordNotFound)
     end
 
     it "non-manager is not authorized" do
diff --git a/spec/services/create_project_service_spec.rb b/spec/services/create_project_service_spec.rb
index 7520e24..140f44e 100644
--- a/spec/services/create_project_service_spec.rb
+++ b/spec/services/create_project_service_spec.rb
@@ -17,7 +17,8 @@ describe CreateProjectService do
 
     context 'without project dump' do
       it 'should raise exception' do
-        expect { service.execute(current_user, '', '') }.to raise_error
+        expect { service.execute(current_user, '', '') }.
+          to raise_error(NoMethodError)
       end
     end