Update Issues/MRs updated_at when user leaves comments.
This commit is contained in:
parent
9f03ae8c91
commit
78754cb1c9
|
@ -10,7 +10,9 @@ module Notes
|
|||
|
||||
# Skip system notes, like status changes and cross-references and awards
|
||||
unless note.system || note.is_award
|
||||
event_service.leave_note(note, note.author)
|
||||
event = event_service.leave_note(note, note.author)
|
||||
|
||||
note.noteable.touch if event.commented?
|
||||
note.create_cross_references!
|
||||
execute_hooks(note)
|
||||
end
|
||||
|
|
|
@ -51,6 +51,14 @@ Feature: Project Issues
|
|||
Then I should see comment "XML attached"
|
||||
And I should see an error alert section within the comment form
|
||||
|
||||
@javascript
|
||||
Scenario: Visiting Issues after leaving a comment
|
||||
Given I visit issue page "Release 0.4"
|
||||
And I leave a comment like "XML attached"
|
||||
And I visit project "Shop" issues page
|
||||
And I sort the list by "Last updated"
|
||||
Then I should see "Release 0.4" at the top
|
||||
|
||||
@javascript
|
||||
Scenario: I search issue
|
||||
Given I fill in issue search with "Re"
|
||||
|
|
|
@ -75,6 +75,25 @@ Feature: Project Merge Requests
|
|||
And I leave a comment like "XML attached"
|
||||
Then I should see comment "XML attached"
|
||||
|
||||
@javascript
|
||||
Scenario: Visiting Merge Requests after leaving a comment
|
||||
Given project "Shop" have "Bug NS-05" open merge request with diffs inside
|
||||
And I visit merge request page "Bug NS-04"
|
||||
And I leave a comment like "XML attached"
|
||||
And I visit project "Shop" merge requests page
|
||||
And I sort the list by "Last updated"
|
||||
Then I should see "Bug NS-04" at the top
|
||||
|
||||
@javascript
|
||||
Scenario: Visiting Merge Requests after commenting on diffs
|
||||
Given project "Shop" have "Bug NS-05" open merge request with diffs inside
|
||||
And I visit merge request page "Bug NS-05"
|
||||
And I click on the Changes tab
|
||||
And I leave a comment like "Line is wrong" on diff
|
||||
And I visit project "Shop" merge requests page
|
||||
And I sort the list by "Last updated"
|
||||
Then I should see "Bug NS-05" at the top
|
||||
|
||||
@javascript
|
||||
Scenario: I comment on a merge request diff
|
||||
Given project "Shop" have "Bug NS-05" open merge request with diffs inside
|
||||
|
|
|
@ -293,6 +293,11 @@ class Spinach::Features::ProjectIssues < Spinach::FeatureSteps
|
|||
expect(page).to have_content('Yay!')
|
||||
end
|
||||
end
|
||||
|
||||
step 'I should see "Release 0.4" at the top' do
|
||||
expect(page.find('ul.content-list.issues-list li.issue:first-child')).to have_content("Release 0.4")
|
||||
end
|
||||
|
||||
def filter_issue(text)
|
||||
fill_in 'issue_search', with: text
|
||||
end
|
||||
|
|
|
@ -415,6 +415,14 @@ class Spinach::Features::ProjectMergeRequests < Spinach::FeatureSteps
|
|||
end
|
||||
end
|
||||
|
||||
step 'I should see "Bug NS-05" at the top' do
|
||||
expect(page.find('ul.content-list.mr-list li.merge-request:first-child')).to have_content("Bug NS-05")
|
||||
end
|
||||
|
||||
step 'I should see "Bug NS-04" at the top' do
|
||||
expect(page.find('ul.content-list.mr-list li.merge-request:first-child')).to have_content("Bug NS-04")
|
||||
end
|
||||
|
||||
def merge_request
|
||||
@merge_request ||= MergeRequest.find_by!(title: "Bug NS-05")
|
||||
end
|
||||
|
|
|
@ -144,4 +144,11 @@ module SharedNote
|
|||
expect(page).to have_content("+1 Awesome!")
|
||||
end
|
||||
end
|
||||
|
||||
step 'I sort the list by "Last updated"' do
|
||||
find('button.dropdown-toggle.btn').click
|
||||
page.within('ul.dropdown-menu.dropdown-menu-align-right li') do
|
||||
click_link "Last updated"
|
||||
end
|
||||
end
|
||||
end
|
||||
|
|
Loading…
Reference in New Issue