summaryrefslogtreecommitdiff
path: root/spec
diff options
context:
space:
mode:
authorIllya Klymov <xanf@xanf.me>2019-04-19 12:44:21 +0300
committerIllya Klymov <xanf@xanf.me>2019-07-24 07:33:08 +0300
commita6ac0a2765f07348f618076b8e550e29c9675b77 (patch)
treec676b9deb087388c6cb234e00a2b4a3b7308d286 /spec
parentc47412d48369c5e2becb2a7ce786aac73d0b325c (diff)
downloadgitlab-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.rb37
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? }