]> source.dussan.org Git - nextcloud-server.git/commit
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)
commitad0adaea95637e1d8fdfe3d2611170562639b68d
tree076f10b23f789ad8384ced64aab88d82a2d1ef91
parent4c88b12a287ba8247f7f584a475de41c6b50f598
Wait for the comment to be added before changing to another file

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