diff options
Diffstat (limited to 'spec/migrations/update_routes_for_lost_and_found_group_and_orphaned_projects_spec.rb')
| -rw-r--r-- | spec/migrations/update_routes_for_lost_and_found_group_and_orphaned_projects_spec.rb | 182 |
1 files changed, 182 insertions, 0 deletions
diff --git a/spec/migrations/update_routes_for_lost_and_found_group_and_orphaned_projects_spec.rb b/spec/migrations/update_routes_for_lost_and_found_group_and_orphaned_projects_spec.rb new file mode 100644 index 00000000000..4cf096b9df6 --- /dev/null +++ b/spec/migrations/update_routes_for_lost_and_found_group_and_orphaned_projects_spec.rb @@ -0,0 +1,182 @@ +# frozen_string_literal: true + +require 'spec_helper' + +require Rails.root.join('db', 'post_migrate', '20200602143020_update_routes_for_lost_and_found_group_and_orphaned_projects.rb') + +describe UpdateRoutesForLostAndFoundGroupAndOrphanedProjects, :migration do + let(:users) { table(:users) } + let(:namespaces) { table(:namespaces) } + let(:members) { table(:members) } + let(:projects) { table(:projects) } + let(:routes) { table(:routes) } + + before do + # Create a Ghost User and its namnespace, but skip the route + ghost_user = users.create!( + name: 'Ghost User', + username: 'ghost', + email: 'ghost@example.com', + user_type: described_class::User::USER_TYPE_GHOST, + projects_limit: 100, + state: :active, + bio: 'This is a "Ghost User"' + ) + + namespaces.create!( + name: 'Ghost User', + path: 'ghost', + owner_id: ghost_user.id, + visibility_level: 20 + ) + + # Create the 'lost-and-found', owned by the Ghost user, but with no route + lost_and_found_group = namespaces.create!( + name: described_class::User::LOST_AND_FOUND_GROUP, + path: described_class::User::LOST_AND_FOUND_GROUP, + type: 'Group', + description: 'Group to store orphaned projects', + visibility_level: 0 + ) + + members.create!( + type: 'GroupMember', + source_id: lost_and_found_group.id, + user_id: ghost_user.id, + source_type: 'Namespace', + access_level: described_class::User::ACCESS_LEVEL_OWNER, + notification_level: 3 + ) + + # Add an orphaned project under 'lost-and-found' but with the wrong path in its route + orphaned_project = projects.create!( + name: 'orphaned_project', + path: 'orphaned_project', + visibility_level: 20, + archived: false, + namespace_id: lost_and_found_group.id + ) + + routes.create!( + source_id: orphaned_project.id, + source_type: 'Project', + path: 'orphaned_project', + name: 'orphaned_project', + created_at: Time.zone.now, + updated_at: Time.zone.now + ) + + # Create another user named ghost which is not the Ghost User + # Also create a 'lost-and-found' group for them and add projects to it + # Purpose: test that the routes added for the 'lost-and-found' group and + # its projects are unique + fake_ghost_user = users.create!( + name: 'Ghost User', + username: 'ghost1', + email: 'ghost1@example.com', + user_type: nil, + projects_limit: 100, + state: :active, + bio: 'This is NOT a "Ghost User"' + ) + + fake_ghost_user_namespace = namespaces.create!( + name: 'Ghost User', + path: 'ghost1', + owner_id: fake_ghost_user.id, + visibility_level: 20 + ) + + routes.create!( + source_id: fake_ghost_user_namespace.id, + source_type: 'Namespace', + path: 'Ghost User', + name: 'ghost1', + created_at: Time.zone.now, + updated_at: Time.zone.now + ) + + fake_lost_and_found_group = namespaces.create!( + name: 'Lost and Found', + path: described_class::User::LOST_AND_FOUND_GROUP, # same path as the lost-and-found group + type: 'Group', + description: 'Fake lost and found group with the same path as the real one', + visibility_level: 20 + ) + + routes.create!( + source_id: fake_lost_and_found_group.id, + source_type: 'Namespace', + path: described_class::User::LOST_AND_FOUND_GROUP, # same path as the lost-and-found group + name: 'Lost and Found', + created_at: Time.zone.now, + updated_at: Time.zone.now + ) + + members.create!( + type: 'GroupMember', + source_id: fake_lost_and_found_group.id, + user_id: fake_ghost_user.id, + source_type: 'Namespace', + access_level: described_class::User::ACCESS_LEVEL_OWNER, + notification_level: 3 + ) + + normal_project = projects.create!( + name: 'normal_project', + path: 'normal_project', + visibility_level: 20, + archived: false, + namespace_id: fake_lost_and_found_group.id + ) + + routes.create!( + source_id: normal_project.id, + source_type: 'Project', + path: "#{described_class::User::LOST_AND_FOUND_GROUP}/normal_project", + name: 'Lost and Found / normal_project', + created_at: Time.zone.now, + updated_at: Time.zone.now + ) + end + + it 'creates the route for the ghost user namespace' do + expect(routes.where(path: 'ghost').count).to eq(0) + + disable_migrations_output { migrate! } + + expect(routes.where(path: 'ghost').count).to eq(1) + end + + it 'fixes the path for the lost-and-found group by generating a unique one' do + expect(namespaces.where(path: described_class::User::LOST_AND_FOUND_GROUP).count).to eq(2) + + disable_migrations_output { migrate! } + + expect(namespaces.where(path: described_class::User::LOST_AND_FOUND_GROUP).count).to eq(1) + + lost_and_found_group = namespaces.find_by(name: described_class::User::LOST_AND_FOUND_GROUP) + expect(lost_and_found_group.path).to eq('lost-and-found1') + end + + it 'creates the route for the lost-and-found group' do + expect(routes.where(path: described_class::User::LOST_AND_FOUND_GROUP).count).to eq(1) + expect(routes.where(path: 'lost-and-found1').count).to eq(0) + + disable_migrations_output { migrate! } + + expect(routes.where(path: described_class::User::LOST_AND_FOUND_GROUP).count).to eq(1) + expect(routes.where(path: 'lost-and-found1').count).to eq(1) + end + + it 'updates the route for the orphaned project' do + orphaned_project_route = routes.find_by(path: 'orphaned_project') + expect(orphaned_project_route.name).to eq('orphaned_project') + + disable_migrations_output { migrate! } + + updated_route = routes.find_by(id: orphaned_project_route.id) + expect(updated_route.path).to eq('lost-and-found1/orphaned_project') + expect(updated_route.name).to eq("#{described_class::User::LOST_AND_FOUND_GROUP} / orphaned_project") + end +end |