diff options
author | Illya Klymov <xanf@xanf.me> | 2019-04-19 12:44:21 +0300 |
---|---|---|
committer | Illya Klymov <xanf@xanf.me> | 2019-07-24 07:33:08 +0300 |
commit | a6ac0a2765f07348f618076b8e550e29c9675b77 (patch) | |
tree | c676b9deb087388c6cb234e00a2b4a3b7308d286 /spec | |
parent | c47412d48369c5e2becb2a7ce786aac73d0b325c (diff) | |
download | gitlab-ce-a6ac0a2765f07348f618076b8e550e29c9675b77.tar.gz |
Add links in admin area overviewce-xanf-add-admin-area-links
Introduces new `feature_entry` helper for dashboard.
This helper reduces code duplication when listing available features
and relevant links to configuration sections
Diffstat (limited to 'spec')
-rw-r--r-- | spec/helpers/dashboard_helper_spec.rb | 37 |
1 files changed, 37 insertions, 0 deletions
diff --git a/spec/helpers/dashboard_helper_spec.rb b/spec/helpers/dashboard_helper_spec.rb index 023238ee0ae..49e23366355 100644 --- a/spec/helpers/dashboard_helper_spec.rb +++ b/spec/helpers/dashboard_helper_spec.rb @@ -22,6 +22,43 @@ describe DashboardHelper do end end + describe '#feature_entry' do + context 'when implicitly enabled' do + it 'considers feature enabled by default' do + entry = feature_entry('Demo', href: 'demo.link') + + expect(entry).to include('<p aria-label="Demo: status on">') + expect(entry).to include('<a href="demo.link">Demo</a>') + end + end + + context 'when explicitly enabled' do + it 'returns a link' do + entry = feature_entry('Demo', href: 'demo.link', enabled: true) + + expect(entry).to include('<p aria-label="Demo: status on">') + expect(entry).to include('<a href="demo.link">Demo</a>') + end + + it 'returns text if href is not provided' do + entry = feature_entry('Demo', enabled: true) + + expect(entry).to include('<p aria-label="Demo: status on">') + expect(entry).not_to match(/<a[^>]+>/) + end + end + + context 'when disabled' do + it 'returns text without link' do + entry = feature_entry('Demo', href: 'demo.link', enabled: false) + + expect(entry).to include('<p aria-label="Demo: status off">') + expect(entry).not_to match(/<a[^>]+>/) + expect(entry).to include('Demo') + end + end + end + describe '.has_start_trial?' do subject { helper.has_start_trial? } |