Re: master c8ee570: Suppress undesired error messages in filenotify-tests.el

[Michael Albinus]

Glenn Morris <address@hidden> writes:

>> --- a/test/lisp/filenotify-tests.el
>> +++ b/test/lisp/filenotify-tests.el
>> @@ -665,6 +665,9 @@ delivered."
>>  (setq auto-revert-notify-exclude-dir-regexp "nothing-to-be-excluded"
>>        auto-revert-remote-files t
>>        auto-revert-stop-on-user-input nil)
>> +;; In the remote case, `vc-refresh-state' returns undesired error
>> +;; messages.  Let's suppress them.
>> +(defalias 'vc-refresh-state 'ignore)
>
>
> Hi - this means that loading filenotify-tests.el breaks Emacs VC normal
> operation for the rest of the session, which seems undesirable. Is there
> an alternative solution?

I'm aware of the problem. I've tried to use `cl-letf' in the
test. Unfortunately, this doesn't work, because the revert is called
when an event arrives; the `cl-letf' settings do not work there.

Hmmm. Usually, the test is not called by average Emacs users, so I
believe it would be acceptable. If not, we must find something else.

Best regards, Michael.