From: Michael Albinus <michael.albinus@gmx.de>
To: Stefan Kangas <stefan@marxist.se>
Cc: Paul Eggert <eggert@cs.ucla.edu>, 36940@debbugs.gnu.org
Subject: bug#36940: tests slowness and failure after recent Tramp changes
Date: Sat, 24 Aug 2019 10:08:22 +0200 [thread overview]
Message-ID: <8736hryoi1.fsf@gmx.de> (raw)
In-Reply-To: <CADwFkmm4r0WZ-LnXNo7x6TvJQohun9PS8bCHzri5zOdjztBjPw@mail.gmail.com> (Stefan Kangas's message of "Sat, 24 Aug 2019 03:51:26 +0200")
[-- Attachment #1: Type: text/plain, Size: 4129 bytes --]
Stefan Kangas <stefan@marxist.se> writes:
> Hi Michael,
Hi Stefan,
> I'm still seeing failures on my MacOS machine on current master. I
> have attached a log with two failing tests from when I run "make
> check" in the top level emacs folder as the file tramp-tests.log.
>
> Weirdly enough, when I say "cd test ; make tramp-tests", I get six
> errors instead -- but no log file is saved. Not sure if I'm doing
> something wrong here, but I've included the console output from this
> test as tramp-tests2.console.log. I tried running the tests in both
> ways a couple of times, and this odd result is reproducible every
> time.
If you say "make check", the default ert selector is applied, which
excludes expensive tests. if you say "make FOO", all test from package
foo are applied, also the expensive tests. You see it by the different
numbers of tests, 47 vs 67 tests.
If you want to get a log file when running tests for package FOO, say
"make FOO.log". This seems to be broken now; will check.
All of this is described in test/README.
> (Not sure if you want to reopen the bug report for this.)
For the time being, I'll continue to reply here. If needed (more complex
failure, not reported initially here) we might need to create another
bug report.
> (t 3 501 20 (0 0 0 1000) (0 0 0 1000) (0 0 0 1000) 0 "drwxr-xr-x" nil 41369843 (-1 . 18))
> (t 5 501 20 (0 0 0 1000) (0 0 0 1000) (0 0 0 1000) 0 "drwxr-xr-x" nil (631 . 16627) (-1 . 18))
> Test tramp-test19-directory-files-and-attributes condition:
> (ert-test-failed
> ((should
> (tramp--test-file-attributes-equal-p
> (file-attributes ...)
> (cdr elt)))
> :form
> (tramp--test-file-attributes-equal-p
> (t 3 501 20
> (23904 37780)
> (23904 37780)
> (23904 37780)
> 102 "drwxr-xr-x" nil ...)
> (t 5 501 20
> (23904 37780)
> (23904 37780)
> (23904 37780)
> 170 "drwxr-xr-x" nil ...))
> :value nil :explanation
> (list-elt 1
> (different-atoms
> (3 "#x3" "?")
> (5 "#x5" "?")))))
> FAILED 30/67 tramp-test19-directory-files-and-attributes (0.996916 sec)
Well, this time it is because of a different inode number, 41369843 vs
(631 . 16627). Will check what's up.
> Test tramp-test22-file-times condition:
> (ert-test-failed
> ((should
> (equal
> (tramp-compat-file-attribute-modification-time ...)
> (seconds-to-time 1)))
> :form
> (equal
> (0 1)
> (0 1 0 0))
> :value nil :explanation
> (proper-lists-of-different-length 2 4
> (0 1)
> (0 1 0 0)
> first-mismatch-at 2)))
> FAILED 33/67 tramp-test22-file-times (0.567633 sec)
Grrr, the time stamps (0 1) and (0 1 0 0) are the same, but handled
differently internally. Fixed in master.
> Test tramp-test30-make-process condition:
> (ert-test-failed
> ((should
> (string-match "killed
> \\'"
> (buffer-string)))
> :form
> (string-match "killed
> \\'" "killed: 9
> ")
> :value nil))
> FAILED 41/67 tramp-test30-make-process (1.024851 sec)
Ahh, the message on macOS is slightly different. Fixed in master.
> Test tramp-test41-utf8 condition:
> (search-failed "^VAR_971C287AFA5BBEDD54ACB58B1CE718B4=Γυρίστε το Γαλαξία με Ώτο Στοπ$")
> FAILED 58/67 tramp-test41-utf8 (35.641851 sec)
> Test tramp-test41-utf8-with-ls condition:
> (search-failed "^VAR_971C287AFA5BBEDD54ACB58B1CE718B4=Γυρίστε το Γαλαξία με Ώτο Στοπ$")
> FAILED 59/67 tramp-test41-utf8-with-ls (29.942527 sec)
> Test tramp-test41-utf8-with-perl condition:
> (search-failed "^VAR_971C287AFA5BBEDD54ACB58B1CE718B4=Γυρίστε το Γαλαξία με Ώτο Στοπ$")
> FAILED 60/67 tramp-test41-utf8-with-perl (36.502001 sec)
These three errors are basically the same. Could you please apply the
following patch to tramp-tests.el, and show me the output? You might run
make -C test tramp-tests >tramp-tests.log 2>&1
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: Type: text/x-diff, Size: 546 bytes --]
diff --git a/test/lisp/net/tramp-tests.el b/test/lisp/net/tramp-tests.el
index 180f746c64..42d44f780b 100644
--- a/test/lisp/net/tramp-tests.el
+++ b/test/lisp/net/tramp-tests.el
@@ -5459,7 +5459,8 @@ tramp--test-utf8
(skip-unless (not (tramp--test-windows-nt-and-batch)))
(skip-unless (not (tramp--test-windows-nt-and-pscp-psftp-p)))
- (tramp--test-utf8))
+ (tramp--test-instrument-test-case 10
+ (tramp--test-utf8)))
(ert-deftest tramp-test41-utf8-with-stat ()
"Check UTF8 encoding in file names and file contents.
[-- Attachment #3: Type: text/plain, Size: 51 bytes --]
> Thanks,
> Stefan Kangas
Best regards, Michael.
next prev parent reply other threads:[~2019-08-24 8:08 UTC|newest]
Thread overview: 49+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-08-06 0:28 bug#36940: tests slowness and failure after recent Tramp changes Paul Eggert
2019-08-06 12:59 ` Michael Albinus
2019-08-06 16:13 ` Paul Eggert
2019-08-06 19:16 ` Michael Albinus
2019-08-06 19:54 ` Michael Albinus
2019-08-06 23:27 ` Paul Eggert
2019-08-07 15:20 ` Michael Albinus
2019-08-07 19:36 ` Paul Eggert
2019-08-08 14:14 ` Michael Albinus
2019-08-07 21:42 ` Glenn Morris
2019-08-08 13:52 ` Michael Albinus
2019-08-10 1:39 ` Paul Eggert
2019-08-10 9:43 ` Michael Albinus
2019-08-10 20:24 ` Paul Eggert
2019-08-11 10:12 ` Michael Albinus
2019-08-14 10:31 ` Michael Albinus
2019-08-15 4:26 ` Paul Eggert
2019-08-24 1:51 ` Stefan Kangas
2019-08-24 8:08 ` Michael Albinus [this message]
2019-08-24 12:51 ` Stefan Kangas
[not found] ` <CADwFkmnZ1D-t3BchTSuUrkbkOpKG=yCH9c1ZJbkyGr9mUZrAUg@mail.gmail.com>
2019-08-25 9:27 ` Michael Albinus
2019-08-25 9:51 ` Eli Zaretskii
2019-08-25 10:07 ` Eli Zaretskii
2019-08-25 11:26 ` Michael Albinus
2019-08-25 11:39 ` Eli Zaretskii
2019-08-25 11:46 ` Michael Albinus
2019-08-25 11:58 ` Eli Zaretskii
2019-08-26 9:22 ` Michael Albinus
2019-08-26 9:59 ` Eli Zaretskii
2019-08-26 11:47 ` Michael Albinus
2019-08-26 12:54 ` Stefan Kangas
2019-08-26 14:19 ` Michael Albinus
2019-08-26 14:36 ` Stefan Kangas
2019-08-26 15:09 ` Michael Albinus
2019-08-26 15:46 ` Stefan Kangas
2019-08-26 16:43 ` Michael Albinus
2019-08-26 17:46 ` Stefan Kangas
2019-08-26 19:47 ` Michael Albinus
2019-08-27 16:34 ` Stefan Kangas
2019-08-27 16:56 ` Michael Albinus
2019-08-28 0:23 ` Stefan Kangas
2019-08-25 11:28 ` Michael Albinus
2019-08-25 11:48 ` Michael Albinus
2019-08-25 15:39 ` Paul Eggert
2019-08-25 16:34 ` Michael Albinus
2019-08-25 20:36 ` Paul Eggert
2019-08-26 8:44 ` Michael Albinus
2019-08-27 2:17 ` Paul Eggert
2019-08-27 11:03 ` Michael Albinus
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=8736hryoi1.fsf@gmx.de \
--to=michael.albinus@gmx.de \
--cc=36940@debbugs.gnu.org \
--cc=eggert@cs.ucla.edu \
--cc=stefan@marxist.se \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this external index
https://git.savannah.gnu.org/cgit/emacs.git
https://git.savannah.gnu.org/cgit/emacs/org-mode.git
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.