Re: Question about test failure on Hydra

[Stephen Berman <stephen.berman@gmx.net>]

On Tue, 01 Aug 2017 16:49:16 +0300 Eli Zaretskii <eliz@gnu.org> wrote:

>> From: Stephen Berman <stephen.berman@gmx.net>
>> Cc: emacs-devel@gnu.org
>> Date: Tue, 01 Aug 2017 11:57:14 +0200
>> 
>> On Tue, 01 Aug 2017 06:12:49 +0300 Eli Zaretskii <eliz@gnu.org> wrote:
>> 
>> >> From: Stephen Berman <stephen.berman@gmx.net>
>> >> Cc: emacs-devel@gnu.org
>> >> Date: Mon, 31 Jul 2017 11:20:10 +0200
>> >> 
>> >> (dired test-dir) reverts an existing Dired buffer, because of setting
>> >> dired-auto-revert-buffer to t.  (When I step through the code, it is
>> >> after reverting that point moves to the subdirectory line
>> >
>> > By "the point moves" you mean what dired-restore-positions does?
>> 
>> Yes.
>
> Then I'd begin with logging the values saved in POSITIONS that
> dired-restore-positions uses, perhaps the recorded positions are
> sometimes wrong.  Then we could take it from there, either by looking
> how the values are recorded or how they are restored.

In commit c5305ff6 Tino Calancha added more assertions to track point,
so this may already be enough to locate the problem if the test fails in
Hydra again.  If it isn't, we can add still more.

Steve Berman