User deletion fixes
This commit is contained in:
parent
6890802a78
commit
8e8d049af0
5 changed files with 5 additions and 10 deletions
|
|
@ -205,6 +205,7 @@ class Event < ActiveRecord::Base
|
||||||
end
|
end
|
||||||
alert
|
alert
|
||||||
end
|
end
|
||||||
|
|
||||||
private
|
private
|
||||||
|
|
||||||
def abstract_limit
|
def abstract_limit
|
||||||
|
|
|
||||||
|
|
@ -5,10 +5,4 @@ class EventUser < ActiveRecord::Base
|
||||||
|
|
||||||
belongs_to :event
|
belongs_to :event
|
||||||
belongs_to :user
|
belongs_to :user
|
||||||
|
|
||||||
def self.create_deleted_eventuser(user,event)
|
|
||||||
event_role = event.event_users.where(user: user).first.event_role
|
|
||||||
event.event_users.where(user: user).destroy_all
|
|
||||||
EventUser.create(user: User.find_by(email: 'deleted@localhost.osem'), event: event, event_role: event_role)
|
|
||||||
end
|
|
||||||
end
|
end
|
||||||
|
|
|
||||||
|
|
@ -130,7 +130,9 @@ class User < ActiveRecord::Base
|
||||||
if event.start_time.nil? || event.start_time > DateTime.now
|
if event.start_time.nil? || event.start_time > DateTime.now
|
||||||
event.destroy
|
event.destroy
|
||||||
else
|
else
|
||||||
EventUser.create_deleted_eventuser(self,event)
|
event_role = event.event_users.where(user: self).first.event_role
|
||||||
|
event.event_users.where(user: self).destroy_all
|
||||||
|
EventUser.create(user: User.find_by(email: 'deleted@localhost.osem'), event: event, event_role: event_role)
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
super
|
super
|
||||||
|
|
|
||||||
|
|
@ -63,4 +63,3 @@
|
||||||
"bLengthChange": false
|
"bLengthChange": false
|
||||||
} );
|
} );
|
||||||
} );
|
} );
|
||||||
>>>>>>> User deletion
|
|
||||||
|
|
|
||||||
|
|
@ -1,7 +1,6 @@
|
||||||
require 'spec_helper'
|
require 'spec_helper'
|
||||||
|
|
||||||
feature User do
|
feature User do
|
||||||
|
|
||||||
shared_examples 'admin ability' do
|
shared_examples 'admin ability' do
|
||||||
scenario 'deletes a user', feature: true, js: true do
|
scenario 'deletes a user', feature: true, js: true do
|
||||||
sign_in(create(:admin))
|
sign_in(create(:admin))
|
||||||
|
|
@ -17,7 +16,7 @@ feature User do
|
||||||
scenario 'deletes a user with scheduled events', feature: true, js: true do
|
scenario 'deletes a user with scheduled events', feature: true, js: true do
|
||||||
sign_in(create(:admin))
|
sign_in(create(:admin))
|
||||||
@user = create(:user)
|
@user = create(:user)
|
||||||
deleted_user = create(:user, email: 'deleted@localhost.osem', name: 'User deleted', biography: 'Data is no longer available for deleted user.')
|
deleted_user = create(:deleted_user, email:'deleted@localhost.osem')
|
||||||
@user.events << create(:event, start_time: DateTime.now)
|
@user.events << create(:event, start_time: DateTime.now)
|
||||||
event = @user.events.first
|
event = @user.events.first
|
||||||
visit admin_users_path
|
visit admin_users_path
|
||||||
|
|
|
||||||
Loading…
Add table
Add a link
Reference in a new issue