diff options
author | Rémy Coutable <remy@rymai.me> | 2016-11-03 17:23:34 +0000 |
---|---|---|
committer | Rémy Coutable <remy@rymai.me> | 2016-11-03 17:23:34 +0000 |
commit | 651bdd7c34c9ebf686b8357d8fc3ca4d0a58792e (patch) | |
tree | 6951957d83ebdf39804920fbb0520a35f419b0c8 | |
parent | d8153e31257c7a0ddc6123cc1a7d893a16e1b87a (diff) | |
parent | 9bb25ee18f4baaadafd95ec92cfe39065d349c1d (diff) | |
download | gitlab-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.rb | 7 |
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)) |