kanbanworkflowstimelinescrumrubyroadmapproject-planningproject-managementopenprojectangularissue-trackerifcgantt-chartganttbug-trackerboardsbcf
You can not select more than 25 topics
Topics must start with a letter or number, can include dashes ('-') and can be up to 35 characters long.
198 lines
7.0 KiB
198 lines
7.0 KiB
require 'spec_helper'
|
|
|
|
require 'features/work_packages/work_packages_page'
|
|
require 'support/work_packages/work_package_field'
|
|
|
|
describe 'Activity tab', js: true, selenium: true do
|
|
def alter_work_package_at(work_package, attributes:, at:, user: User.current)
|
|
work_package.update_attributes(attributes.merge(updated_at: at))
|
|
|
|
note_journal = work_package.journals.last
|
|
note_journal.update_attributes(created_at: at, user: attributes[:user])
|
|
end
|
|
|
|
let(:project) { FactoryBot.create :project_with_types, is_public: true }
|
|
let!(:work_package) do
|
|
work_package = FactoryBot.create(:work_package,
|
|
project: project,
|
|
created_at: 5.days.ago.to_date.to_s(:db),
|
|
subject: initial_subject,
|
|
journal_notes: initial_comment)
|
|
|
|
note_journal = work_package.journals.last
|
|
note_journal.update_attributes(created_at: 5.days.ago.to_date.to_s)
|
|
|
|
work_package
|
|
end
|
|
|
|
let(:initial_subject) { 'My Subject' }
|
|
let(:initial_comment) { 'First comment on this wp.' }
|
|
let(:comments_in_reverse) { false }
|
|
let(:activity_tab) { ::Components::WorkPackages::Activities.new(work_package) }
|
|
|
|
let(:initial_note) do
|
|
work_package.journals[0]
|
|
end
|
|
|
|
let!(:note_1) do
|
|
attributes = { subject: 'New subject', description: 'Some not so long description.' }
|
|
|
|
alter_work_package_at(work_package,
|
|
attributes: attributes,
|
|
at: 3.days.ago.to_date.to_s(:db),
|
|
user: user)
|
|
|
|
work_package.journals.last
|
|
end
|
|
|
|
let!(:note_2) do
|
|
attributes = { journal_notes: 'Another comment by a different user' }
|
|
|
|
alter_work_package_at(work_package,
|
|
attributes: attributes,
|
|
at: 1.days.ago.to_date.to_s(:db),
|
|
user: FactoryBot.create(:admin))
|
|
|
|
work_package.journals.last
|
|
end
|
|
|
|
before do
|
|
login_as(user)
|
|
allow(user.pref).to receive(:warn_on_leaving_unsaved?).and_return(false)
|
|
allow(user.pref).to receive(:comments_sorting).and_return(comments_in_reverse ? 'desc' : 'asc')
|
|
allow(user.pref).to receive(:comments_in_reverse_order?).and_return(comments_in_reverse)
|
|
end
|
|
|
|
shared_examples 'shows activities in order' do
|
|
let(:journals) do
|
|
journals = [initial_note, note_1, note_2]
|
|
|
|
journals
|
|
end
|
|
|
|
it 'shows activities in ascending order' do
|
|
journals.each_with_index do |journal, idx|
|
|
actual_index =
|
|
if comments_in_reverse
|
|
journals.length - idx
|
|
else
|
|
idx + 1
|
|
end
|
|
|
|
date_selector = ".work-package-details-activities-activity:nth-of-type(#{actual_index}) " +
|
|
'.activity-date'
|
|
# Do not use :long format to match the printed date without double spaces
|
|
# on the first 9 days of the month
|
|
expect(page).to have_selector(date_selector,
|
|
text: journal.created_at.to_date.strftime("%B %-d, %Y"))
|
|
|
|
activity = page.find("#activity-#{idx + 1}")
|
|
|
|
if journal.id != note_1.id
|
|
expect(activity).to have_selector('.user', text: journal.user.name)
|
|
expect(activity).to have_selector('.user-comment > .message', text: journal.notes, visible: :all)
|
|
end
|
|
|
|
if activity == note_1
|
|
expect(activity).to have_selector('.work-package-details-activities-messages .message',
|
|
count: 2)
|
|
expect(activity).to have_selector('.message',
|
|
text: "Subject changed from #{initial_subject} " \
|
|
"to #{journal.data.subject}")
|
|
end
|
|
end
|
|
end
|
|
end
|
|
|
|
shared_examples 'activity tab' do
|
|
before do
|
|
work_package_page.visit_tab! 'activity'
|
|
work_package_page.ensure_page_loaded
|
|
expect(page).to have_selector('.user-comment > .message',
|
|
text: initial_comment)
|
|
end
|
|
|
|
context 'with permission' do
|
|
let(:role) do
|
|
FactoryBot.create(:role, permissions: %i[view_work_packages
|
|
add_work_package_notes])
|
|
end
|
|
let(:user) do
|
|
FactoryBot.create(:user,
|
|
member_in_project: project,
|
|
member_through_role: role)
|
|
end
|
|
|
|
context 'with ascending comments' do
|
|
let(:comments_in_reverse) { false }
|
|
it_behaves_like 'shows activities in order'
|
|
end
|
|
|
|
context 'with reversed comments' do
|
|
let(:comments_in_reverse) { true }
|
|
it_behaves_like 'shows activities in order'
|
|
end
|
|
|
|
it 'can toggle between activities and comments-only' do
|
|
expect(page).to have_selector('.work-package-details-activities-activity-contents', count: 3)
|
|
expect(page).to have_selector('.user-comment > .message', text: note_2.notes)
|
|
|
|
# Show only comments
|
|
find('.activity-comments--toggler').click
|
|
|
|
# It should remove the middle
|
|
expect(page).to have_selector('.work-package-details-activities-activity-contents', count: 2)
|
|
expect(page).to have_selector('.user-comment > .message', text: initial_comment)
|
|
expect(page).to have_selector('.user-comment > .message', text: note_2.notes)
|
|
|
|
# Show all again
|
|
find('.activity-comments--toggler').click
|
|
expect(page).to have_selector('.work-package-details-activities-activity-contents', count: 3)
|
|
end
|
|
|
|
it 'can quote a previous comment' do
|
|
activity_tab.hover_action('1', :quote)
|
|
|
|
field = WorkPackageEditorField.new work_package_page,
|
|
'comment',
|
|
selector: '.work-packages--activity--add-comment'
|
|
|
|
field.expect_active!
|
|
|
|
# Add our comment
|
|
quote = field.input_element.text
|
|
expect(quote).to include(initial_comment)
|
|
field.input_element.base.send_keys "\nthis is some remark under a quote"
|
|
field.submit_by_click
|
|
|
|
expect(page).to have_selector('.user-comment > .message', count: 3)
|
|
expect(page).to have_selector('.user-comment > .message blockquote')
|
|
end
|
|
end
|
|
|
|
context 'with no permission' do
|
|
let(:role) do
|
|
FactoryBot.create(:role, permissions: [:view_work_packages])
|
|
end
|
|
let(:user) do
|
|
FactoryBot.create(:user,
|
|
member_in_project: project,
|
|
member_through_role: role)
|
|
end
|
|
|
|
it 'shows the activities, but does not allow commenting' do
|
|
expect(page).not_to have_selector('.work-packages--activity--add-comment', visible: true)
|
|
end
|
|
end
|
|
end
|
|
|
|
context 'split screen' do
|
|
let(:work_package_page) { Pages::SplitWorkPackage.new(work_package, project) }
|
|
it_behaves_like 'activity tab'
|
|
end
|
|
|
|
context 'full screen' do
|
|
let(:work_package_page) { Pages::FullWorkPackage.new(work_package) }
|
|
it_behaves_like 'activity tab'
|
|
end
|
|
end
|
|
|