~cytrogen/masto-fe

cf18cc2891be6f63fc1f4d96120901be81820fa6 — Matt Jankowski 2 years ago bdcd8a9
Fix RSpec/MultipleSubjects cop (#24738)

M .rubocop_todo.yml => .rubocop_todo.yml +0 -11
@@ 732,17 732,6 @@ RSpec/MultipleExpectations:
RSpec/MultipleMemoizedHelpers:
  Max: 21

# This cop supports safe autocorrection (--autocorrect).
RSpec/MultipleSubjects:
  Exclude:
    - 'spec/controllers/activitypub/collections_controller_spec.rb'
    - 'spec/controllers/activitypub/followers_synchronizations_controller_spec.rb'
    - 'spec/controllers/activitypub/outboxes_controller_spec.rb'
    - 'spec/controllers/api/web/embeds_controller_spec.rb'
    - 'spec/controllers/emojis_controller_spec.rb'
    - 'spec/controllers/follower_accounts_controller_spec.rb'
    - 'spec/controllers/following_accounts_controller_spec.rb'

# Configuration parameters: AllowedGroups.
RSpec/NestedGroups:
  Max: 6

M spec/controllers/activitypub/collections_controller_spec.rb => spec/controllers/activitypub/collections_controller_spec.rb +1 -2
@@ 35,10 35,9 @@ RSpec.describe ActivityPub::CollectionsController, type: :controller do
  describe 'GET #show' do
    context 'when id is "featured"' do
      context 'without signature' do
        subject(:body) { body_as_json }

        subject(:response) { get :show, params: { id: 'featured', account_username: account.username } }

        let(:body) { body_as_json }
        let(:remote_account) { nil }

        it 'returns http success' do

M spec/controllers/activitypub/followers_synchronizations_controller_spec.rb => spec/controllers/activitypub/followers_synchronizations_controller_spec.rb +1 -2
@@ 34,10 34,9 @@ RSpec.describe ActivityPub::FollowersSynchronizationsController, type: :controll
    end

    context 'with signature from example.com' do
      subject(:body) { body_as_json }

      subject(:response) { get :show, params: { account_username: account.username } }

      let(:body) { body_as_json }
      let(:remote_account) { Fabricate(:account, domain: 'example.com', uri: 'https://example.com/instance') }

      it 'returns http success' do

M spec/controllers/activitypub/outboxes_controller_spec.rb => spec/controllers/activitypub/outboxes_controller_spec.rb +1 -2
@@ 35,10 35,9 @@ RSpec.describe ActivityPub::OutboxesController, type: :controller do

  describe 'GET #show' do
    context 'without signature' do
      subject(:body) { body_as_json }

      subject(:response) { get :show, params: { account_username: account.username, page: page } }

      let(:body) { body_as_json }
      let(:remote_account) { nil }

      context 'with page not requested' do

M spec/controllers/api/web/embeds_controller_spec.rb => spec/controllers/api/web/embeds_controller_spec.rb +2 -2
@@ 10,10 10,10 @@ describe Api::Web::EmbedsController do
  before { sign_in user }

  describe 'POST #create' do
    subject(:response) { post :create, params: { url: url } }

    subject(:body) { JSON.parse(response.body, symbolize_names: true) }

    let(:response) { post :create, params: { url: url } }

    context 'when successfully finds status' do
      let(:status) { Fabricate(:status) }
      let(:url) { "http://#{Rails.configuration.x.web_domain}/@#{status.account.username}/#{status.id}" }

M spec/controllers/emojis_controller_spec.rb => spec/controllers/emojis_controller_spec.rb +2 -2
@@ 8,10 8,10 @@ describe EmojisController do
  let(:emoji) { Fabricate(:custom_emoji) }

  describe 'GET #show' do
    subject(:response) { get :show, params: { id: emoji.id, format: :json } }

    subject(:body) { JSON.parse(response.body, symbolize_names: true) }

    let(:response) { get :show, params: { id: emoji.id, format: :json } }

    it 'returns the right response' do
      expect(response).to have_http_status 200
      expect(body[:name]).to eq ':coolcat:'

M spec/controllers/follower_accounts_controller_spec.rb => spec/controllers/follower_accounts_controller_spec.rb +2 -2
@@ 39,10 39,10 @@ describe FollowerAccountsController do
    end

    context 'when format is json' do
      subject(:response) { get :index, params: { account_username: alice.username, page: page, format: :json } }

      subject(:body) { response.parsed_body }

      let(:response) { get :index, params: { account_username: alice.username, page: page, format: :json } }

      context 'with page' do
        let(:page) { 1 }


M spec/controllers/following_accounts_controller_spec.rb => spec/controllers/following_accounts_controller_spec.rb +2 -2
@@ 39,10 39,10 @@ describe FollowingAccountsController do
    end

    context 'when format is json' do
      subject(:response) { get :index, params: { account_username: alice.username, page: page, format: :json } }

      subject(:body) { response.parsed_body }

      let(:response) { get :index, params: { account_username: alice.username, page: page, format: :json } }

      context 'with page' do
        let(:page) { 1 }