diff options
author | Ash McKenzie <amckenzie@gitlab.com> | 2018-08-20 14:34:24 +1000 |
---|---|---|
committer | Ash McKenzie <amckenzie@gitlab.com> | 2018-09-07 15:09:50 +1000 |
commit | ddb95f0dd0463b884860238fd250092bffed0e78 (patch) | |
tree | df56735a910d4bf1b98861ea736c3ba475741172 /spec/gitlab_config_spec.rb | |
parent | b0a526354eb012d2a2d874d22515d20c035ea5d6 (diff) | |
download | gitlab-shell-ddb95f0dd0463b884860238fd250092bffed0e78.tar.gz |
GitlabConfig, GitlabShell & GitlabKeys spec fixes
GitlabConfig spec was relying upon real config.yml
Diffstat (limited to 'spec/gitlab_config_spec.rb')
-rw-r--r-- | spec/gitlab_config_spec.rb | 15 |
1 files changed, 10 insertions, 5 deletions
diff --git a/spec/gitlab_config_spec.rb b/spec/gitlab_config_spec.rb index c262116..d12b657 100644 --- a/spec/gitlab_config_spec.rb +++ b/spec/gitlab_config_spec.rb @@ -3,29 +3,34 @@ require_relative '../lib/gitlab_config' describe GitlabConfig do let(:config) { GitlabConfig.new } + let(:config_data) { {} } - describe :gitlab_url do + before { expect(YAML).to receive(:load_file).and_return(config_data) } + + describe '#gitlab_url' do let(:url) { 'http://test.com' } + subject { config.gitlab_url } - before { config.send(:config)['gitlab_url'] = url } + + before { config_data['gitlab_url'] = url } it { is_expected.not_to be_empty } it { is_expected.to eq(url) } context 'remove trailing slashes' do - before { config.send(:config)['gitlab_url'] = url + '//' } + before { config_data['gitlab_url'] = url + '//' } it { is_expected.to eq(url) } end end - describe :audit_usernames do + describe '#audit_usernames' do subject { config.audit_usernames } it("returns false by default") { is_expected.to eq(false) } end - describe :log_format do + describe '#log_format' do subject { config.log_format } it 'returns "text" by default' do |