Read about our upcoming Code of Conduct on this issue

Commit 9b0899cb authored by Bob Van Landuyt's avatar Bob Van Landuyt
Browse files

Remove EE-specific group paths

parent b50f29e7a1da
......@@ -112,13 +112,6 @@ module PathRegex
# this would map to the activity-page of its parent.
GROUP_ROUTES = %w[
-
analytics
audit_events
hooks
ldap
ldap_group_links
notification_setting
pipeline_quota
].freeze
ILLEGAL_PROJECT_PATH_WORDS = PROJECT_WILDCARD_ROUTES
......
......@@ -152,16 +152,7 @@ def failure_message(constant_name, migration_helper, missing_words: [], addition
let(:paths_after_group_id) do
group_routes.map do |route|
route.gsub(STARTING_WITH_GROUP, '').split('/').first
end.uniq + ee_paths_after_group_id
end
let(:ee_paths_after_group_id) do
%w(analytics
ldap
ldap_group_links
notification_setting
audit_events
pipeline_quota hooks)
end.uniq
end
describe 'TOP_LEVEL_ROUTES' do
......@@ -292,7 +283,7 @@ def failure_message(constant_name, migration_helper, missing_words: [], addition
end
it 'rejects group routes' do
expect(subject).not_to match('root/analytics/')
expect(subject).not_to match('root/-/')
end
end
......@@ -312,7 +303,7 @@ def failure_message(constant_name, migration_helper, missing_words: [], addition
end
it 'rejects group routes' do
expect(subject).not_to match('root/analytics/more/')
expect(subject).not_to match('root/-/')
end
end
end
......
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