Gnus development mailing list
 help / color / mirror / Atom feed
From: David Engster <deng@randomsample.de>
To: ding@gnus.org
Subject: Re: Gnus automatic builds
Date: Thu, 07 Apr 2011 21:21:46 +0200	[thread overview]
Message-ID: <87vcyp27g5.fsf@randomsample.de> (raw)
In-Reply-To: <87k4f7mclt.fsf@lifelogs.com> (Ted Zlatanov's message of "Wed, 6 Apr 2011 13:57:50 -0500")

[-- Attachment #1: Type: text/plain, Size: 2252 bytes --]

Ted Zlatanov writes:
> OK.  I'm OK with just the blamelist for failed builds, though I would
> always add Katsumi Yamaoka, Lars, and me to it.

OK, will do. Lars and Katsumi, if you're reading this, please let me
know if this is OK.

>>> Also does it run the ERT tests?  I think it should, maybe as a separate
>>> target.  The registry.el and gnus-registry.el ERT tests write files to
>>> the filesystem (using `make-temp-file') but otherwise there are no side
>>> effects.
>
> DE> Not yet. I think we should use another Makefile rule for that. I'll see
> DE> to it that I add something, probably 'make check', which will then
> DE> depend on many smaller rules which run specific tests (like the registry
> DE> tests).
>
> Up to you, however you want to do it.

I've now added a new rule 'check' which will run the whole test suite. I
think it's a good thing to have a rule for every test in lisp/Makefile,
since during development one often has to run single tests and not the
whole suite. I've now started with "test-registry".

Problem is: ERT does not run under Emacs 22, nor under XEmacs. So only
Emacs 23 and 24 will currently run the tests. Maybe someone will port
ERT to XEmacs, then we can add that any time.

I've also written a little test for NNTP connections; I've attached it
to this mail. It also shows how one can test "real" Gnus functionality
in batch mode on the buildbot server. It will connect news.gmane.org,
get gmane.discuss, retrieve a single article from it (via message-id)
and exit Gnus. The problem with these kinds of tests is that they depend
on an external server. I added a simple check so that those test will
only run if a ping to news.gmane.org succeeds, but still, the server
might have other problems, which will lead to a failing test. So a
failed NNTP test might not necessarily mean the last commit actually
broke something, but a look in the logs should clear that up.

The reason I did not commit gnustest-nntp.el yet is that I don't know
where it should live. I'm hesitant to add it to gnus/lisp, because maybe
we should have a dedicated directory for tests? We could also put other
files in there we'll need for testing (some dummy mails for populating
an IMAP folder, for instance). Suggestions?

-David


[-- Attachment #2: gnustest-nntp.el --]
[-- Type: application/emacs-lisp, Size: 3146 bytes --]

  reply	other threads:[~2011-04-07 19:21 UTC|newest]

Thread overview: 44+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-04-06  4:28 no gnus registry new dependency on ert? Eric Abrahamsen
2011-04-06 10:03 ` Ted Zlatanov
2011-04-06 10:10   ` David Engster
2011-04-06 10:54     ` Ted Zlatanov
2011-04-06 13:10       ` Ted Zlatanov
2011-04-06 13:28         ` David Engster
2011-04-06 14:26           ` Ted Zlatanov
2011-04-06 14:57             ` David Engster
2011-04-06 15:00               ` Ted Zlatanov
2011-04-06 15:03               ` David Engster
2011-04-06 15:51                 ` Ted Zlatanov
2011-04-06 13:15 ` Gnus automatic builds (was: no gnus registry new dependency on ert?) Ted Zlatanov
2011-04-06 13:31   ` Gnus automatic builds David Engster
2011-04-06 16:14     ` David Engster
2011-04-06 16:30       ` Ted Zlatanov
2011-04-06 18:10         ` David Engster
2011-04-06 18:57           ` Ted Zlatanov
2011-04-07 19:21             ` David Engster [this message]
2011-04-07 19:51               ` Ted Zlatanov
2011-04-07 20:02                 ` David Engster
2011-04-07 21:19                   ` Ted Zlatanov
2011-04-12 16:08               ` Lars Magne Ingebrigtsen
2011-04-12 16:29                 ` David Engster
2011-04-12 16:36                   ` Lars Magne Ingebrigtsen
2011-04-12 18:33                     ` Jason L Tibbitts III
2011-04-12 18:41                       ` Lars Magne Ingebrigtsen
2011-04-12 19:35                         ` Jason L Tibbitts III
2011-04-12 19:46                           ` David Engster
2011-04-12 19:51                             ` Jason L Tibbitts III
2011-04-16  8:22                               ` David Engster
2011-04-16 15:19                                 ` Ted Zlatanov
2011-04-20 18:50                                   ` David Engster
2011-04-20 20:58                                     ` Ted Zlatanov
2011-04-22 11:27                                     ` Adam Sjøgren
2011-04-22 21:21                                       ` David Engster
2011-04-25  9:05                                         ` Adam Sjøgren
2011-04-12 20:00                           ` Lars Magne Ingebrigtsen
2011-04-20 20:59                         ` David Engster
2011-04-20 21:03                           ` Ted Zlatanov
2011-04-20 21:31                             ` Jason L Tibbitts III
2011-04-21  7:33                               ` David Engster
2011-04-25 12:28                             ` Ted Zlatanov
2011-04-06 20:26       ` David Engster
2011-04-07 19:30         ` David Engster

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=87vcyp27g5.fsf@randomsample.de \
    --to=deng@randomsample.de \
    --cc=ding@gnus.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.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).