New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
AO3-4475 Fix TODOs in comment test #2343
Conversation
n_comments.to_i.times do |i| | ||
step %{I am logged in as a random user} | ||
step %{I post the comment "Bla bla" on the work "#{title}"} | ||
visit work_url(work) | ||
fill_in("comment[content]", :with => "Bla bla") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Use the new Ruby 1.9 hash syntax.
Then I should see "Comment was successfully updated" | ||
And I should see "Actually, I meant something different" | ||
And I should not see "Mistaken comment" | ||
And I should see "EST" within ".comment .posted" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I am confused / worried about hardcoding this timezone here. Since @cesy is travelling, if anyone else can explain if/why it's safe to do so that would also be good!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's the default timezone in our Archive config file. Would it be better to make it a reference to that config?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please do
AO3-4475 Fix TODOs in comment test
https://otwarchive.atlassian.net/browse/AO3-4475
Our automated test for comment pagination, threading and editing has some TODOs. Fix them.
The last test here is a pretty slow one, given that it needs a lot of comments, so I've also done my best to speed it up.