Commit 6cfaefc5 authored by Markus Koller's avatar Markus Koller

Merge branch '223916-remove-check-issues-available' into 'master'

Allow Jira issues without GitLab issues enabled

See merge request gitlab-org/gitlab!35886
parents d868d313 ac31877d
......@@ -7,7 +7,6 @@ module Projects
include RecordUserLastActivity
before_action :check_feature_enabled!
before_action :check_issues_available!
before_action do
push_frontend_feature_flag(:jira_integration, project)
......
......@@ -16,20 +16,16 @@ RSpec.describe Projects::Integrations::Jira::IssuesController do
create(:jira_service, project: project)
end
context 'when jira_integration feature disabled' do
it 'returns 404 status' do
stub_feature_flags(jira_integration: false)
it 'renders the "index" template' do
get :index, params: { namespace_id: project.namespace, project_id: project }
get :index, params: { namespace_id: project.namespace, project_id: project }
expect(response).to have_gitlab_http_status(:not_found)
end
expect(response).to have_gitlab_http_status(:ok)
expect(response).to render_template(:index)
end
context 'when GitLab issues disabled' do
context 'when jira_integration feature disabled' do
it 'returns 404 status' do
project.issues_enabled = false
project.save!
stub_feature_flags(jira_integration: false)
get :index, params: { namespace_id: project.namespace, project_id: project }
......@@ -37,15 +33,6 @@ RSpec.describe Projects::Integrations::Jira::IssuesController do
end
end
context 'when GitLab issues enabled' do
it 'renders the "index" template' do
get :index, params: { namespace_id: project.namespace, project_id: project }
expect(response).to have_gitlab_http_status(:ok)
expect(response).to render_template(:index)
end
end
context 'when project has moved' do
let(:new_project) { create(:project) }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment