diff options
author | Alexis Reigel <alexis.reigel.ext@siemens.com> | 2017-11-23 13:51:15 +0100 |
---|---|---|
committer | Alexis Reigel <alexis.reigel.ext@siemens.com> | 2018-01-04 09:33:41 +0100 |
commit | 57e2488085b623e4b48a6b8518e844fa842b6f4a (patch) | |
tree | ec73646af39d4fd5ace2d18ea13c7d211471414c | |
parent | a2655e3fbda186ea598f6649dee8bb5565d6b980 (diff) | |
download | gitlab-ce-57e2488085b623e4b48a6b8518e844fa842b6f4a.tar.gz |
we only need a test for the happy path
the failure case is already covered by the test in
spec/services/test_hooks/system_service_spec.rb
-rw-r--r-- | spec/features/admin/admin_hooks_spec.rb | 8 |
1 files changed, 0 insertions, 8 deletions
diff --git a/spec/features/admin/admin_hooks_spec.rb b/spec/features/admin/admin_hooks_spec.rb index 197e6cd07c4..725e603d1e3 100644 --- a/spec/features/admin/admin_hooks_spec.rb +++ b/spec/features/admin/admin_hooks_spec.rb @@ -119,14 +119,6 @@ describe 'Admin::Hooks' do WebMock.stub_request(:post, system_hook.url) end - it 'fails if the user does not have any repository with a merge request' do - visit admin_hooks_path - find('.hook-test-button.dropdown').click - click_link 'Merge requests events' - - expect(page).to have_content 'Ensure one of your projects has merge requests.' - end - it 'succeeds if the user has a repository with a merge request' do project = create(:project, :repository) create(:project_member, user: user, project: project) |