fix specs

app/models/identity_provider.rb

@@ -1,6 +1,9 @@
 # frozen_string_literal: true
 
 class IdentityProvider < ApplicationRecord
+  validates :name, presence: true
+  validates :provider, presence: true
+
   mount_uploader :logo_image, LogoUploader
 
   def parsed_options(rack_env = nil)

spec/models/identity_provider_spec.rb

@@ -15,6 +15,11 @@
       expect(subject).to be_valid
     end
 
+    it 'is not valid without a name' do
+      subject.name = nil
+      expect(subject).not_to be_valid
+    end
+
     it 'is not valid without a provider' do
       subject.provider = nil
       expect(subject).not_to be_valid

spec/requests/single_signon_request_spec.rb

@@ -4,9 +4,22 @@
 
 RSpec.describe "SingleSignons", type: :request do
   describe "GET single_signon#index" do
-    it "returns http success" do
-      get "/single_signon"
-      expect(response).to have_http_status(:success)
+    describe "with no IdentityProviders" do
+      it "redirects to sign in" do
+        get "/single_signon"
+        expect(response).to have_http_status(:redirect)
+      end
+    end
+
+    describe "with an IdentityProvider" do
+      before do
+        IdentityProvider.create(name: 'fake', provider: 'saml')
+      end
+
+      it "renders succes" do
+        get "/single_signon"
+        expect(response).to have_http_status(:success)
+      end
     end
   end
 end