From 5fbdb2055becdb4177ad8aa0b3891cb2617d223b Mon Sep 17 00:00:00 2001 From: Matt Jankowski Date: Fri, 26 Jan 2024 11:35:19 -0500 Subject: [PATCH] Combine repeated `subject` in `cli/accounts` spec shared example (#28942) --- spec/lib/mastodon/cli/accounts_spec.rb | 18 ++++++++---------- 1 file changed, 8 insertions(+), 10 deletions(-) diff --git a/spec/lib/mastodon/cli/accounts_spec.rb b/spec/lib/mastodon/cli/accounts_spec.rb index 98be2b2027..137f85c6ca 100644 --- a/spec/lib/mastodon/cli/accounts_spec.rb +++ b/spec/lib/mastodon/cli/accounts_spec.rb @@ -1326,18 +1326,16 @@ describe Mastodon::CLI::Accounts do end shared_examples 'a successful migration' do - it 'calls the MoveService for the last migration' do - expect { subject } - .to output_results('OK') - - last_migration = source_account.migrations.last - - expect(move_service).to have_received(:call).with(last_migration).once - end - - it 'displays a successful message' do + it 'displays a success message and calls the MoveService for the last migration' do expect { subject } .to output_results("OK, migrated #{source_account.acct} to #{target_account.acct}") + + expect(move_service) + .to have_received(:call).with(last_migration).once + end + + def last_migration + source_account.migrations.last end end