]> source.dussan.org Git - nextcloud-server.git/commitdiff
Wait for the comment to be added before changing to another file 12696/head
authorDaniel Calviño Sánchez <danxuliu@gmail.com>
Sat, 24 Nov 2018 17:16:09 +0000 (18:16 +0100)
committerBackportbot <backportbot-noreply@rullzer.com>
Tue, 27 Nov 2018 16:55:32 +0000 (16:55 +0000)
If the server is too slow, changing to a different file immediately
after sending a new comment but without waiting for the comment to be
shown for the original file could cause the new comment to be shown for
the current file instead.

This is, indeed, a bug in the comments. However, it is not possible to
test it reliably in the acceptance tests, as it depends on how fast the
server adds the message and how fast the client changes to a different
file; sometimes the test would fail and sometimes it would not.

Therefore, now it is waited for the comment to be added before changing
to another file, as in this case it can be reliably tested that changing
to a different file does not cause the comments from the previous file
to be shown in the current file (this was a different bug already fixed
and due to which this test was added in the first place).

Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
tests/acceptance/features/app-comments.feature

index 1ee113d0aac57aba0d1875246a0b207dcd2ee101..42893e5ea3533e820064635a4ac1da3ff1c94143 100644 (file)
@@ -24,6 +24,7 @@ Feature: app-comments
     And I open the details view for "Folder"
     And I open the "Comments" tab in the details view
     And I create a new comment with "Comment in Folder" as message
+    And I see a comment with "Comment in Folder" as message
     And I open the details view for "welcome.txt"
     # The "Comments" tab should already be opened
     When I create a new comment with "Comment in welcome.txt" as message