]> source.dussan.org Git - nextcloud-server.git/commit
Wait for the comment to be added before changing to another file 12697/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:56:38 +0000 (16:56 +0000)
commit876ee0fd76b6c464740379cc8fb8d5001e8628c7
treeb405c055c2aaecae87b7e8fe08f771235953db42
parentfc0a1d10159e60d1e76a3f141b5c13da7a633c1f
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