all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Xiyue Deng <manphiz@gmail.com>
To: Eli Zaretskii <eliz@gnu.org>
Cc: mattiase@acm.org, 66782@debbugs.gnu.org
Subject: bug#66782: 29.1; ERT tests report test redefined depending on loading sequence
Date: Sat, 28 Oct 2023 04:14:15 -0700	[thread overview]
Message-ID: <87jzr7gfso.fsf@debian-hx90.lan> (raw)
In-Reply-To: <83pm0zcc25.fsf@gnu.org> (Eli Zaretskii's message of "Sat, 28 Oct 2023 12:48:34 +0300")

Eli Zaretskii <eliz@gnu.org> writes:

>> From: Xiyue Deng <manphiz@gmail.com>
>> Cc: Mattias Engdegård <mattiase@acm.org>,
>>   66782@debbugs.gnu.org
>> Date: Sat, 28 Oct 2023 02:39:46 -0700
>> 
>> Eli Zaretskii <eliz@gnu.org> writes:
>> 
>> > If test2 requires test1, why are both of them explicitly run from the
>> > command line?  Isn't that redundant, since running test2 will also run
>> > the tests defined by test1?
>> 
>> IIUC most projects just run all tests without explicit dependency
>> detection.  For example, in lsp-mode it uses eask ert-runner to run all
>> tests[1].
>
> OK, but why do that in a single Emacs session?  That is unclean for
> much more important reasons: some left-overs from previous tests could
> affect the following tests.  IMO, each test should be run in a fresh
> Emacs session, i.e. no command line should ever load more than a
> single test file.

I think one reason is that doing all test in one session gives you a
nice summary of how many tests passed and which ones failed in the end.
If you run just one test per session you'll have to inspect the results
manually which could be much more time consuming for large test suites,
and plus it may be slower.

-- 
Xiyue Deng





  reply	other threads:[~2023-10-28 11:14 UTC|newest]

Thread overview: 21+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-10-27 20:59 bug#66782: 29.1; ERT tests reports test redefined depending on loading sequence Xiyue Deng
2023-10-28  6:32 ` Eli Zaretskii
2023-10-28  9:39   ` bug#66782: 29.1; ERT tests report " Xiyue Deng
2023-10-28  9:48     ` Eli Zaretskii
2023-10-28 11:14       ` Xiyue Deng [this message]
2023-10-28 11:34         ` Eli Zaretskii
2023-10-28 11:50           ` Xiyue Deng
2023-10-28 12:03             ` Eli Zaretskii
2023-10-28 12:46               ` Mattias Engdegård
2023-10-28 21:19                 ` Xiyue Deng
2023-10-29 12:09                   ` Mattias Engdegård
2023-10-29 21:53                     ` Xiyue Deng
2023-10-30 12:04                       ` Mattias Engdegård
2023-10-30 20:47                         ` Xiyue Deng
2023-11-02 17:17                           ` Xiyue Deng
2023-11-02 18:56                             ` Mattias Engdegård
2023-11-02 22:00                               ` Xiyue Deng
2023-11-03 17:14                                 ` Mattias Engdegård
2023-11-03 17:56                                   ` Xiyue Deng
2023-11-04  8:51                                     ` Eli Zaretskii
2023-10-28 19:41               ` Xiyue Deng

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=87jzr7gfso.fsf@debian-hx90.lan \
    --to=manphiz@gmail.com \
    --cc=66782@debbugs.gnu.org \
    --cc=eliz@gnu.org \
    --cc=mattiase@acm.org \
    /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.