mirror of
https://github.com/wahyd4/gitlabhq.git
synced 2026-08-10 05:06:46 +10:00
Include missing events and fix save functionality in admin service template settings form ### What does this MR do? This MR includes missing settings left out in the Admin -> Service Templates page and fixes the inability to save certain settings. ### Are there points in the code the reviewer needs to double check? No. ### Why was this MR needed? Because the service template form was broken and untested. ### What are the relevant issue numbers / [Feature requests](http://feedback.gitlab.com/)? #1275 Before:  After:  See merge request !427
48 lines
1.3 KiB
Ruby
48 lines
1.3 KiB
Ruby
class Spinach::Features::AdminSettings < Spinach::FeatureSteps
|
|
include SharedAuthentication
|
|
include SharedPaths
|
|
include SharedAdmin
|
|
include Gitlab::CurrentSettings
|
|
|
|
step 'I modify settings and save form' do
|
|
uncheck 'Gravatar enabled'
|
|
fill_in 'Home page url', with: 'https://about.gitlab.com/'
|
|
click_button 'Save'
|
|
end
|
|
|
|
step 'I should see application settings saved' do
|
|
current_application_settings.gravatar_enabled.should be_false
|
|
current_application_settings.home_page_url.should == 'https://about.gitlab.com/'
|
|
page.should have_content 'Application settings saved successfully'
|
|
end
|
|
|
|
step 'I click on "Service Templates"' do
|
|
click_link 'Service Templates'
|
|
end
|
|
|
|
step 'I click on "Slack" service' do
|
|
click_link 'Slack'
|
|
end
|
|
|
|
step 'I check all events and submit form' do
|
|
page.check('Active')
|
|
page.check('Push events')
|
|
page.check('Tag push events')
|
|
page.check('Comments')
|
|
page.check('Issues events')
|
|
page.check('Merge Request events')
|
|
fill_in 'Webhook', with: "http://localhost"
|
|
click_on 'Save'
|
|
end
|
|
|
|
step 'I should see service template settings saved' do
|
|
page.should have_content 'Application settings saved successfully'
|
|
end
|
|
|
|
step 'I should see all checkboxes checked' do
|
|
all('input[type=checkbox]').each do |checkbox|
|
|
checkbox.should be_checked
|
|
end
|
|
end
|
|
end
|