diff options
| author | Michael Albinus | 2018-03-30 12:36:07 +0200 |
|---|---|---|
| committer | Michael Albinus | 2018-03-30 12:36:07 +0200 |
| commit | ef0617888c452d1a62d354b3d866f17eea2e1ced (patch) | |
| tree | d8bf928e471bfea3f36fdaea3ad05bc8240e93b6 | |
| parent | 670f2ffae718046c0fb37313965a51c040ed096f (diff) | |
| download | emacs-ef0617888c452d1a62d354b3d866f17eea2e1ced.tar.gz emacs-ef0617888c452d1a62d354b3d866f17eea2e1ced.zip | |
Improve Tramp test performance
* lisp/net/tramp.el (tramp-backtrace): Improve performance.
* test/lisp/net/tramp-tests.el (tramp-test03-file-name-host-rules):
Skip for older Emacsen.
(tramp-test39-utf8): Remove instrumentation.
| -rw-r--r-- | lisp/net/tramp.el | 9 | ||||
| -rw-r--r-- | test/lisp/net/tramp-tests.el | 6 |
2 files changed, 9 insertions, 6 deletions
diff --git a/lisp/net/tramp.el b/lisp/net/tramp.el index 43b5e77428a..52ff021c500 100644 --- a/lisp/net/tramp.el +++ b/lisp/net/tramp.el | |||
| @@ -1691,10 +1691,11 @@ applicable)." | |||
| 1691 | "Dump a backtrace into the debug buffer. | 1691 | "Dump a backtrace into the debug buffer. |
| 1692 | If VEC-OR-PROC is nil, the buffer *debug tramp* is used. This | 1692 | If VEC-OR-PROC is nil, the buffer *debug tramp* is used. This |
| 1693 | function is meant for debugging purposes." | 1693 | function is meant for debugging purposes." |
| 1694 | (if vec-or-proc | 1694 | (when (>= tramp-verbose 10) |
| 1695 | (tramp-message vec-or-proc 10 "\n%s" (with-output-to-string (backtrace))) | 1695 | (if vec-or-proc |
| 1696 | (if (>= tramp-verbose 10) | 1696 | (tramp-message |
| 1697 | (with-output-to-temp-buffer "*debug tramp*" (backtrace))))) | 1697 | vec-or-proc 10 "\n%s" (with-output-to-string (backtrace))) |
| 1698 | (with-output-to-temp-buffer "*debug tramp*" (backtrace))))) | ||
| 1698 | 1699 | ||
| 1699 | (defsubst tramp-error (vec-or-proc signal fmt-string &rest arguments) | 1700 | (defsubst tramp-error (vec-or-proc signal fmt-string &rest arguments) |
| 1700 | "Emit an error. | 1701 | "Emit an error. |
diff --git a/test/lisp/net/tramp-tests.el b/test/lisp/net/tramp-tests.el index 5e79a4bce6f..3ca401b2fa1 100644 --- a/test/lisp/net/tramp-tests.el +++ b/test/lisp/net/tramp-tests.el | |||
| @@ -1727,6 +1727,8 @@ handled properly. BODY shall not contain a timeout." | |||
| 1727 | "Check host name rules for host-less methods." | 1727 | "Check host name rules for host-less methods." |
| 1728 | (skip-unless (tramp--test-enabled)) | 1728 | (skip-unless (tramp--test-enabled)) |
| 1729 | (skip-unless (tramp--test-sh-p)) | 1729 | (skip-unless (tramp--test-sh-p)) |
| 1730 | ;; `user-error' has appeared in Emacs 24.3. | ||
| 1731 | (skip-unless (fboundp 'user-error)) | ||
| 1730 | 1732 | ||
| 1731 | ;; Host names must match rules in case the command template of a | 1733 | ;; Host names must match rules in case the command template of a |
| 1732 | ;; method doesn't use them. | 1734 | ;; method doesn't use them. |
| @@ -4687,8 +4689,7 @@ Use the `ls' command." | |||
| 4687 | (skip-unless (not (tramp--test-windows-nt-and-batch))) | 4689 | (skip-unless (not (tramp--test-windows-nt-and-batch))) |
| 4688 | (skip-unless (not (tramp--test-windows-nt-and-pscp-psftp-p))) | 4690 | (skip-unless (not (tramp--test-windows-nt-and-pscp-psftp-p))) |
| 4689 | 4691 | ||
| 4690 | (tramp--test-instrument-test-case 10 | 4692 | (tramp--test-utf8)) |
| 4691 | (tramp--test-utf8))) | ||
| 4692 | 4693 | ||
| 4693 | (ert-deftest tramp-test39-utf8-with-stat () | 4694 | (ert-deftest tramp-test39-utf8-with-stat () |
| 4694 | "Check UTF8 encoding in file names and file contents. | 4695 | "Check UTF8 encoding in file names and file contents. |
| @@ -5117,6 +5118,7 @@ Since it unloads Tramp, it shall be the last test to run." | |||
| 5117 | ;; * file-name-case-insensitive-p | 5118 | ;; * file-name-case-insensitive-p |
| 5118 | 5119 | ||
| 5119 | ;; * Work on skipped tests. Make a comment, when it is impossible. | 5120 | ;; * Work on skipped tests. Make a comment, when it is impossible. |
| 5121 | ;; * Revisit expensive tests, once problems in tramp-error are solved. | ||
| 5120 | ;; * Fix `tramp-test05-expand-file-name-relative' in `expand-file-name'. | 5122 | ;; * Fix `tramp-test05-expand-file-name-relative' in `expand-file-name'. |
| 5121 | ;; * Fix `tramp-test06-directory-file-name' for `ftp'. | 5123 | ;; * Fix `tramp-test06-directory-file-name' for `ftp'. |
| 5122 | ;; * Investigate, why `tramp-test11-copy-file' and `tramp-test12-rename-file' | 5124 | ;; * Investigate, why `tramp-test11-copy-file' and `tramp-test12-rename-file' |