Commit 2d2f13f1 authored by Douglas Barbosa Alexandre's avatar Douglas Barbosa Alexandre Committed by Robert Speicher

Update last_repository_updated_at when a push event is created

parent a7fd4947
......@@ -352,6 +352,12 @@ class Event < ActiveRecord::Base
Project.unscoped.where(id: project_id).
where('last_activity_at <= ?', RESET_PROJECT_ACTIVITY_INTERVAL.ago).
update_all(last_activity_at: created_at)
if push?
Project.unscoped.where(id: project_id).
where('last_repository_updated_at <= ?', RESET_PROJECT_ACTIVITY_INTERVAL.ago).
update_all(last_repository_updated_at: created_at)
end
end
def authored_by?(user)
......
......@@ -243,19 +243,38 @@ describe Event, models: true do
expect(project).not_to receive(:update_column).
with(:last_activity_at, a_kind_of(Time))
expect(project).not_to receive(:update_column).
with(:last_repository_updated_at, a_kind_of(Time))
create_event(project, project.owner)
end
end
context 'when a project was updated more than 1 hour ago' do
it 'updates the project' do
project.update(last_activity_at: 1.year.ago)
context 'with a push event' do
it 'updates the project last_activity_at and last_repository_updated_at' do
project.update(last_activity_at: 1.year.ago, last_repository_updated_at: 1.year.ago)
create_event(project, project.owner)
create_event(project, project.owner)
project.reload
expect(project.last_activity_at).to be_within(1.minute).of(Time.now)
expect(project.last_repository_updated_at).to be_within(1.minute).of(Time.now)
end
end
context 'without a push event' do
it 'does not updates the project last_repository_updated_at' do
project.update(last_activity_at: 1.year.ago, last_repository_updated_at: 1.year.ago)
project.reload
create(:closed_issue_event, project: project, author: project.owner)
project.last_activity_at <= 1.minute.ago
project.reload
expect(project.last_activity_at).to be_within(1.minute).of(Time.now)
expect(project.last_repository_updated_at).to be_within(1.minute).of(1.year.ago)
end
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