Merge pull request #6495 from krauselukas/repo_controller_spec

Adjust repositories controller rspec for bootstrap DoD migration
openSUSE · Dec 6, 2018 · 614acd8 · 614acd8
2 parents 39b6bea + e90726f
commit 614acd8
Showing 1 changed file with 8 additions and 4 deletions.
diff --git a/src/api/spec/controllers/webui/repositories_controller_spec.rb b/src/api/spec/controllers/webui/repositories_controller_spec.rb
@@ -192,7 +192,8 @@
       end
 
       it { expect(assigns(:error)).to start_with('Repository with name') }
-      it { expect(response).to have_http_status(:success) }
+      # FIXME: Remove 'have_http_status(:success)' after old webui got dropped
+      it { expect(response).to have_http_status(:success).or(have_http_status(:redirect)) }
     end
 
     context 'with no valid repository type' do
@@ -204,7 +205,8 @@
       end
 
       it { expect(assigns(:error)).to start_with("Couldn't add repository:") }
-      it { expect(response).to have_http_status(:success) }
+      # FIXME: Remove 'have_http_status(:success)' after old webui got dropped
+      it { expect(response).to have_http_status(:success).or(have_http_status(:redirect)) }
     end
 
     context 'with no valid repository Architecture' do
@@ -216,7 +218,8 @@
       end
 
       it { expect(assigns(:error)).to start_with("Couldn't add repository:") }
-      it { expect(response).to have_http_status(:success) }
+      # FIXME: Remove 'have_http_status(:success)' after old webui got dropped
+      it { expect(response).to have_http_status(:success).or(have_http_status(:redirect)) }
     end
 
     context 'with valid repository data' do
@@ -228,7 +231,8 @@
       end
 
       it { expect(assigns(:error)).to be_nil }
-      it { expect(response).to have_http_status(:success) }
+      # FIXME: Remove 'have_http_status(:success)' after old webui got dropped
+      it { expect(response).to have_http_status(:success).or(have_http_status(:redirect)) }
     end
   end