diff options
Diffstat (limited to 'spec/lib/system_check')
6 files changed, 36 insertions, 22 deletions
diff --git a/spec/lib/system_check/app/hashed_storage_all_projects_check_spec.rb b/spec/lib/system_check/app/hashed_storage_all_projects_check_spec.rb index 14bf9d61ab5..4536618ba61 100644 --- a/spec/lib/system_check/app/hashed_storage_all_projects_check_spec.rb +++ b/spec/lib/system_check/app/hashed_storage_all_projects_check_spec.rb @@ -1,13 +1,8 @@ # frozen_string_literal: true require 'spec_helper' -require 'rake_helper' - -RSpec.describe SystemCheck::App::HashedStorageAllProjectsCheck do - before do - silence_output - end +RSpec.describe SystemCheck::App::HashedStorageAllProjectsCheck, :silence_stdout do describe '#check?' do it 'fails when at least one project is in legacy storage' do create(:project, :legacy_storage) diff --git a/spec/lib/system_check/app/hashed_storage_enabled_check_spec.rb b/spec/lib/system_check/app/hashed_storage_enabled_check_spec.rb index 32a2f409858..36c6f0a2be1 100644 --- a/spec/lib/system_check/app/hashed_storage_enabled_check_spec.rb +++ b/spec/lib/system_check/app/hashed_storage_enabled_check_spec.rb @@ -1,13 +1,8 @@ # frozen_string_literal: true require 'spec_helper' -require 'rake_helper' - -RSpec.describe SystemCheck::App::HashedStorageEnabledCheck do - before do - silence_output - end +RSpec.describe SystemCheck::App::HashedStorageEnabledCheck, :silence_stdout do describe '#check?' do it 'fails when hashed storage is disabled' do stub_application_setting(hashed_storage_enabled: false) diff --git a/spec/lib/system_check/incoming_email/imap_authentication_check_spec.rb b/spec/lib/system_check/incoming_email/imap_authentication_check_spec.rb new file mode 100644 index 00000000000..d7a77a84472 --- /dev/null +++ b/spec/lib/system_check/incoming_email/imap_authentication_check_spec.rb @@ -0,0 +1,31 @@ +# frozen_string_literal: true + +require 'fast_spec_helper' + +MAIL_ROOM_CONFIG_ENABLED_SAMPLE = + ":mailboxes:\n"\ + " \n"\ + " -\n"\ + " :host: \"gitlab.example.com\"\n"\ + " :port: 143\n"\ + "" + +RSpec.describe SystemCheck::IncomingEmail::ImapAuthenticationCheck do + subject(:system_check) { described_class.new } + + describe '#load_config' do + subject { system_check.send(:load_config) } + + context 'returns no mailbox configurations with mailroom default configuration' do + it { is_expected.to be_nil } + end + + context 'returns an array of mailbox configurations with mailroom configured' do + before do + allow(File).to receive(:read).and_return(MAIL_ROOM_CONFIG_ENABLED_SAMPLE) + end + + it { is_expected.to eq([{ host: "gitlab.example.com", port: 143 }]) } + end + end +end diff --git a/spec/lib/system_check/orphans/namespace_check_spec.rb b/spec/lib/system_check/orphans/namespace_check_spec.rb index 795dfde9029..e764c2313cd 100644 --- a/spec/lib/system_check/orphans/namespace_check_spec.rb +++ b/spec/lib/system_check/orphans/namespace_check_spec.rb @@ -1,15 +1,13 @@ # frozen_string_literal: true require 'spec_helper' -require 'rake_helper' -RSpec.describe SystemCheck::Orphans::NamespaceCheck do +RSpec.describe SystemCheck::Orphans::NamespaceCheck, :silence_stdout do let(:storages) { Gitlab.config.repositories.storages.reject { |key, _| key.eql? 'broken' } } before do allow(Gitlab.config.repositories).to receive(:storages).and_return(storages) allow(subject).to receive(:fetch_disk_namespaces).and_return(disk_namespaces) - silence_output end describe '#multi_check' do diff --git a/spec/lib/system_check/orphans/repository_check_spec.rb b/spec/lib/system_check/orphans/repository_check_spec.rb index 2ab30f4802d..91b48969cc1 100644 --- a/spec/lib/system_check/orphans/repository_check_spec.rb +++ b/spec/lib/system_check/orphans/repository_check_spec.rb @@ -1,16 +1,14 @@ # frozen_string_literal: true require 'spec_helper' -require 'rake_helper' -RSpec.describe SystemCheck::Orphans::RepositoryCheck do +RSpec.describe SystemCheck::Orphans::RepositoryCheck, :silence_stdout do let(:storages) { Gitlab.config.repositories.storages.reject { |key, _| key.eql? 'broken' } } before do allow(Gitlab.config.repositories).to receive(:storages).and_return(storages) allow(subject).to receive(:fetch_disk_namespaces).and_return(disk_namespaces) allow(subject).to receive(:fetch_disk_repositories).and_return(disk_repositories) - # silence_output end describe '#multi_check' do diff --git a/spec/lib/system_check/simple_executor_spec.rb b/spec/lib/system_check/simple_executor_spec.rb index c9a09d86e32..96d93146d5e 100644 --- a/spec/lib/system_check/simple_executor_spec.rb +++ b/spec/lib/system_check/simple_executor_spec.rb @@ -1,9 +1,8 @@ # frozen_string_literal: true require 'spec_helper' -require 'rake_helper' -RSpec.describe SystemCheck::SimpleExecutor do +RSpec.describe SystemCheck::SimpleExecutor, :silence_stdout do before do stub_const('SimpleCheck', Class.new(SystemCheck::BaseCheck)) stub_const('OtherCheck', Class.new(SystemCheck::BaseCheck)) @@ -154,8 +153,6 @@ RSpec.describe SystemCheck::SimpleExecutor do describe '#execute' do before do - silence_output - subject << SimpleCheck subject << OtherCheck end |