summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorRémy Coutable <remy@rymai.me>2016-11-03 17:23:34 +0000
committerRémy Coutable <remy@rymai.me>2016-11-03 17:23:34 +0000
commit651bdd7c34c9ebf686b8357d8fc3ca4d0a58792e (patch)
tree6951957d83ebdf39804920fbb0520a35f419b0c8
parentd8153e31257c7a0ddc6123cc1a7d893a16e1b87a (diff)
parent9bb25ee18f4baaadafd95ec92cfe39065d349c1d (diff)
downloadgitlab-ce-651bdd7c34c9ebf686b8357d8fc3ca4d0a58792e.tar.gz
Merge branch 'no-include-on-top-level' into 'master'
Don't include Gitlab::Routing.url_helpers on top-level namespace (fix error for url_for) ## What does this MR do? Tries to fix: ``` ActionView::Template::Error: arguments passed to url_for can't be handled. Please require routes or provide your own implementation ``` One of the failed build: https://gitlab.com/gitlab-org/gitlab-ce/builds/5825797 See merge request !7265
-rw-r--r--spec/models/project_services/jira_service_spec.rb7
1 files changed, 5 insertions, 2 deletions
diff --git a/spec/models/project_services/jira_service_spec.rb b/spec/models/project_services/jira_service_spec.rb
index a3e9adae4e2..ee0e38bd373 100644
--- a/spec/models/project_services/jira_service_spec.rb
+++ b/spec/models/project_services/jira_service_spec.rb
@@ -1,7 +1,8 @@
require 'spec_helper'
-include Gitlab::Routing.url_helpers
describe JiraService, models: true do
+ include Gitlab::Routing.url_helpers
+
describe "Associations" do
it { is_expected.to belong_to :project }
it { is_expected.to have_one :service_hook }
@@ -79,7 +80,9 @@ describe JiraService, models: true do
stub_config_setting(relative_url_root: '/gitlab')
stub_config_setting(url: Settings.send(:build_gitlab_url))
- Project.default_url_options[:script_name] = "/gitlab"
+ allow(JiraService).to receive(:default_url_options) do
+ { script_name: '/gitlab' }
+ end
@jira_service.execute(merge_request, ExternalIssue.new("JIRA-123", project))