Date: Wed, 24 Oct 2012 22:11:11 -0400
From: Austin Clements <amdragon@MIT.EDU>
To: Ethan Glasser-Camp <ethan.glasser.camp@gmail.com>
Subject: Re: [PATCH] test: Fix HTML rendering test
Message-ID: <20121025021111.GW14861@mit.edu>
References: <1351109160-11105-1-git-send-email-amdragon@mit.edu>
	<873913joqz.fsf@betacantrips.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <873913joqz.fsf@betacantrips.com>
User-Agent: Mutt/1.5.21 (2010-09-15)
Cc: notmuch@notmuchmail.org
Precedence: list

Quoth Ethan Glasser-Camp on Oct 24 at  9:59 pm:
> Austin Clements <amdragon@MIT.EDU> writes:
> 
> > The test designed to exercise Emacs' rendering of HTML emails
> > containing images inadvertently assumed w3m was available under Emacs
> > 23.  The real point of this test was to exercise Emacs 24's shr
> > renderer, so if shr isn't available, we now fall back to html2text,
> > which comes with Emacs.
> 
> Hi! I'm eager to apply any patch here that makes this better. But this
> one doesn't fix it for me (24.1.1, although it seems to work with
> 23.4.1). OUTPUT is "*\nsmiley " (no space after the asterisk or before
> the word smiley, but after). I see that this sed command is supposed to
> normalize things, but at least on my setup, it doesn't. I also see "nil"
> written to console, but I have no idea what that's about.

Emacs seems to have as many ways to convert HTML to text as there are
people trying to run this test.  What's the value of
mm-text-html-renderer for you in Emacs 24?

> More generally, I guess I don't understand exactly what this test is
> supposed to be exercising. The commit message says "the shr renderer",
> but what about it? In
> id:"1348941314-8377-4-git-send-email-amdragon@mit.edu" you write that
> using shr raised a void-variable error previously, so maybe we're making
> sure that error doesn't show up? In that case, even my semi-broken
> output is good enough.

Yes; this is essentially a regression test.  I would prefer not to
catch exceptions in this test because I think we'll eventually start
eating exceptions in body renderers to prevent them from breaking the
rest of the show buffer.  We could check that the output contains the
word "smiley" somewhere as evidence that it probably didn't crash.

> In a perfect world, test probably shouldn't succeed if shr isn't
> present, but should note that it wasn't run. Maybe the emacs lisp code
> can check for shr, and if it's not present, write "shr not present" to
> an output file, and the shell code can grep for that and then call
> test_skip if it sees it?

That's another possibility.  Unfortunately, skipped tests seem rather
heavyweight in the test infrastructure and get reported loudly, which
would affect anyone running the tests under Emacs 23.

> Still, I'm excited that you're working on this so please let's get it
> fixed!
> 
> Ethan