[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

bug#25158: [PATCH] A better way for test code to access messages

From: Michael Albinus
Subject: bug#25158: [PATCH] A better way for test code to access messages
Date: Tue, 07 Feb 2017 21:46:57 +0100
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/26.0.50 (gnu/linux)

Michael Albinus <address@hidden> writes:

> Eli Zaretskii <address@hidden> writes:
>> Making these changes didn't introduce any regressions in the relevant
>> tests, so I pushed them.
> There is a new error when running under GNU/Linux from a shell:
> # make SELECTOR=\'file-notify-test03-autorevert-remote -C test 
> filenotify-tests
> [...]
> Testing lisp/filenotify-tests.el
> Running 1 tests (2017-02-05 11:42:33+0100)
> Reverting buffer `file-notify-test141896xu'.
> Error: (wrong-type-argument stringp nil)
> Reverting buffer `file-notify-test141896xu'.
> Error: (wrong-type-argument stringp nil)
>    passed  1/1  file-notify-test03-autorevert-remote
> The test passes, but the error message looks suspicious. I've tried to
> find where it comes from, and it seems to be the new macro
> `ert-with-message-capture'. I didn't debug further; maybe Gemini could
> have a look?
> The bug is not visible when running the test interactively.

Finally, it is not a bug of `ert-with-message-capture'. It comes from
`vc-refresh-state', and it was not visible before using the new macro.

I've adapted the test case `file-notify-test03-autorevert'; it works
fine now. Bug#25158 could be closed.

>> Thanks.

Best regards, Michael.

reply via email to

[Prev in Thread] Current Thread [Next in Thread]