Convert `admin/site_uploads` spec controller->system (#34108)

spec/controllers/admin/site_uploads_controller_spec.rb

@@ -1,23 +0,0 @@
-# frozen_string_literal: true
-
-require 'rails_helper'
-
-RSpec.describe Admin::SiteUploadsController do
-  render_views
-
-  let(:user) { Fabricate(:admin_user) }
-
-  before do
-    sign_in user, scope: :user
-  end
-
-  describe 'DELETE #destroy' do
-    let(:site_upload) { Fabricate(:site_upload, var: 'thumbnail') }
-
-    it 'returns http success' do
-      delete :destroy, params: { id: site_upload.id }
-
-      expect(response).to redirect_to(admin_settings_path)
-    end
-  end
-end

spec/system/admin/site_uploads_spec.rb

@@ -0,0 +1,27 @@
+# frozen_string_literal: true
+
+require 'rails_helper'
+
+RSpec.describe 'Admin SiteUploads' do
+  let(:user) { Fabricate(:admin_user) }
+
+  before { sign_in(user) }
+
+  describe 'Removing a site upload' do
+    let!(:site_upload) { Fabricate(:site_upload, var: 'thumbnail') }
+
+    it 'removes the upload and redirects' do
+      visit admin_settings_branding_path
+      expect(page)
+        .to have_title(I18n.t('admin.settings.branding.title'))
+
+      expect { click_on I18n.t('admin.site_uploads.delete') }
+        .to change(SiteUpload, :count).by(-1)
+      expect { site_upload.reload }
+        .to raise_error(ActiveRecord::RecordNotFound)
+      expect(page)
+        .to have_content(I18n.t('admin.site_uploads.destroyed_msg'))
+        .and have_title(I18n.t('admin.settings.branding.title'))
+    end
+  end
+end