From f1b6f2fc784639a54ca1ca47860ba2d840e0e6b0 Mon Sep 17 00:00:00 2001 From: Matt Jankowski Date: Mon, 18 Aug 2025 09:58:12 -0400 Subject: [PATCH] Use request specs --- .../admin/accounts/avatars_spec.rb} | 6 ++---- .../admin/accounts/headers_spec.rb} | 6 ++---- 2 files changed, 4 insertions(+), 8 deletions(-) rename spec/{controllers/admin/accounts/avatars_controller_spec.rb => requests/admin/accounts/avatars_spec.rb} (86%) rename spec/{controllers/admin/accounts/headers_controller_spec.rb => requests/admin/accounts/headers_spec.rb} (86%) diff --git a/spec/controllers/admin/accounts/avatars_controller_spec.rb b/spec/requests/admin/accounts/avatars_spec.rb similarity index 86% rename from spec/controllers/admin/accounts/avatars_controller_spec.rb rename to spec/requests/admin/accounts/avatars_spec.rb index 2eb1dedaf7e..b3c9eb28b1e 100644 --- a/spec/controllers/admin/accounts/avatars_controller_spec.rb +++ b/spec/requests/admin/accounts/avatars_spec.rb @@ -2,13 +2,11 @@ require 'rails_helper' -RSpec.describe Admin::Accounts::AvatarsController do - render_views - +RSpec.describe 'Admin Accounts Avatars' do before { sign_in current_user } describe 'DELETE #destroy' do - subject { delete :destroy, params: { account_id: account.id } } + subject { delete "/admin/accounts/#{account.id}/avatar" } let(:current_user) { Fabricate(:user, role: role) } let(:account) { Fabricate(:account, avatar: fixture_file_upload('avatar.gif', 'image/gif')) } diff --git a/spec/controllers/admin/accounts/headers_controller_spec.rb b/spec/requests/admin/accounts/headers_spec.rb similarity index 86% rename from spec/controllers/admin/accounts/headers_controller_spec.rb rename to spec/requests/admin/accounts/headers_spec.rb index 161b138d7e4..4fec01e98a3 100644 --- a/spec/controllers/admin/accounts/headers_controller_spec.rb +++ b/spec/requests/admin/accounts/headers_spec.rb @@ -2,13 +2,11 @@ require 'rails_helper' -RSpec.describe Admin::Accounts::HeadersController do - render_views - +RSpec.describe 'Admin Accounts Headers' do before { sign_in current_user } describe 'DELETE #destroy' do - subject { delete :destroy, params: { account_id: account.id } } + subject { delete "/admin/accounts/#{account.id}/header" } let(:current_user) { Fabricate(:user, role: role) } let(:account) { Fabricate(:account, header: fixture_file_upload('attachment.jpg', 'image/jpeg')) }