diff options
Diffstat (limited to 'spec/lib/error_tracking/collector/dsn_spec.rb')
-rw-r--r-- | spec/lib/error_tracking/collector/dsn_spec.rb | 26 |
1 files changed, 17 insertions, 9 deletions
diff --git a/spec/lib/error_tracking/collector/dsn_spec.rb b/spec/lib/error_tracking/collector/dsn_spec.rb index af55e6f20ec..3aa8719fe38 100644 --- a/spec/lib/error_tracking/collector/dsn_spec.rb +++ b/spec/lib/error_tracking/collector/dsn_spec.rb @@ -3,24 +3,32 @@ require 'spec_helper' RSpec.describe ErrorTracking::Collector::Dsn do - describe '.build__url' do - let(:gitlab) do - double( + describe '.build_url' do + let(:setting) do + { protocol: 'https', https: true, + port: 443, host: 'gitlab.example.com', - port: '4567', relative_url_root: nil - ) + } end subject { described_class.build_url('abcdef1234567890', 778) } - it 'returns a valid URL' do - allow(Settings).to receive(:gitlab).and_return(gitlab) - allow(Settings).to receive(:gitlab_on_standard_port?).and_return(false) + it 'returns a valid URL without explicit port' do + stub_config_setting(setting) - is_expected.to eq('https://abcdef1234567890@gitlab.example.com:4567/api/v4/error_tracking/collector/778') + is_expected.to eq('https://abcdef1234567890@gitlab.example.com/api/v4/error_tracking/collector/778') + end + + context 'with non-standard port' do + it 'returns a valid URL with custom port' do + setting[:port] = 4567 + stub_config_setting(setting) + + is_expected.to eq('https://abcdef1234567890@gitlab.example.com:4567/api/v4/error_tracking/collector/778') + end end end end |