unofficial mirror of notmuch@notmuchmail.org
 help / color / mirror / code / Atom feed
* [PATCH v2] test: Fix HTML rendering test
@ 2012-11-12  1:12 Austin Clements
  2012-11-14  2:46 ` Ethan Glasser-Camp
                   ` (2 more replies)
  0 siblings, 3 replies; 4+ messages in thread
From: Austin Clements @ 2012-11-12  1:12 UTC (permalink / raw)
  To: notmuch; +Cc: tomi.ollila

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 check that Emacs 24's shr
renderer didn't crash when given img tags, so use shr if it's
available, html2text otherwise (which is built in), and do only a
simple sanity check of the result.
---

This is version 2 of
id:"1351109160-11105-1-git-send-email-amdragon@mit.edu".  This makes
fewer assumptions about the output, so hopefully it fixes the problems
with the first version.

 test/emacs |   27 ++++++++++-----------------
 1 file changed, 10 insertions(+), 17 deletions(-)

diff --git a/test/emacs b/test/emacs
index 1f84b91..64b00db 100755
--- a/test/emacs
+++ b/test/emacs
@@ -750,13 +750,12 @@ counter=$(test_emacs \
 test_expect_equal "$counter" 2
 
 
-test_begin_subtest "Rendering HTML mail with images"
 add_message '[subject]="HTML mail with images"' \
     '[content-type]="multipart/related; boundary=abcd"' \
     '[body]="--abcd
 Content-Type: text/html
 
-<img src="cid:330@goomoji.gmail">
+<img src="cid:330@goomoji.gmail"> smiley
 
 --abcd
 Content-Type: image/gif
@@ -766,21 +765,15 @@ Content-ID: <330@goomoji.gmail>
 R0lGODlhDAAMAKIFAF5LAP/zxAAAANyuAP/gaP///wAAAAAAACH5BAEAAAUALAAAAAAMAAwAAAMl
 WLPcGjDKFYi9lxKBOaGcF35DhWHamZUW0K4mAbiwWtuf0uxFAgA7
 --abcd--"'
-test_emacs "(notmuch-show \"id:${gen_msg_id}\")
-	    (test-output)"
-# Normalize output for Emacs 23 and Emacs 24
-sed -i 's/\[cid\]/*/' OUTPUT
-cat <<EOF >EXPECTED
-Notmuch Test Suite <test_suite@notmuchmail.org> (2001-01-05) (inbox)
-Subject: HTML mail with images
-To: Notmuch Test Suite <test_suite@notmuchmail.org>
-Date: Fri, 05 Jan 2001 15:43:57 +0000
-
-[ multipart/related ]
-[ text/html ]
-*
-EOF
-test_expect_equal_file OUTPUT EXPECTED
+test_emacs "(let ((mm-text-html-renderer
+		   (if (assq 'shr mm-text-html-renderer-alist)
+		       'shr 'html2text)))
+	      (notmuch-show \"id:${gen_msg_id}\"))
+	    (test-output)" > /dev/null
+# Different Emacs versions and renderers give very different results,
+# so just check that something reasonable showed up.
+test_expect_success "Rendering HTML mail with images" \
+    'grep -q smiley OUTPUT'
 
 
 test_done
-- 
1.7.10.4

^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH v2] test: Fix HTML rendering test
  2012-11-12  1:12 [PATCH v2] test: Fix HTML rendering test Austin Clements
@ 2012-11-14  2:46 ` Ethan Glasser-Camp
  2012-11-15 12:11 ` David Bremner
  2012-11-15 22:49 ` David Bremner
  2 siblings, 0 replies; 4+ messages in thread
From: Ethan Glasser-Camp @ 2012-11-14  2:46 UTC (permalink / raw)
  To: Austin Clements, notmuch; +Cc: tomi.ollila

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 check that Emacs 24's shr
> renderer didn't crash when given img tags, so use shr if it's
> available, html2text otherwise (which is built in), and do only a
> simple sanity check of the result.

This fixes the failing test on my machine.

Ethan

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH v2] test: Fix HTML rendering test
  2012-11-12  1:12 [PATCH v2] test: Fix HTML rendering test Austin Clements
  2012-11-14  2:46 ` Ethan Glasser-Camp
@ 2012-11-15 12:11 ` David Bremner
  2012-11-15 22:49 ` David Bremner
  2 siblings, 0 replies; 4+ messages in thread
From: David Bremner @ 2012-11-15 12:11 UTC (permalink / raw)
  To: Austin Clements, notmuch; +Cc: tomi.ollila

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 check that Emacs 24's shr
> renderer didn't crash when given img tags, so use shr if it's
> available, html2text otherwise (which is built in), and do only a
> simple sanity check of the result.

OK for me, queuing to push.

d

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH v2] test: Fix HTML rendering test
  2012-11-12  1:12 [PATCH v2] test: Fix HTML rendering test Austin Clements
  2012-11-14  2:46 ` Ethan Glasser-Camp
  2012-11-15 12:11 ` David Bremner
@ 2012-11-15 22:49 ` David Bremner
  2 siblings, 0 replies; 4+ messages in thread
From: David Bremner @ 2012-11-15 22:49 UTC (permalink / raw)
  To: Austin Clements, notmuch; +Cc: tomi.ollila

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 check that Emacs 24's shr
> renderer didn't crash when given img tags, so use shr if it's
> available, html2text otherwise (which is built in), and do only a
> simple sanity check of the result.

Pushed, although it looks like there are still issues on emacs24

d

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2012-11-15 22:49 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-11-12  1:12 [PATCH v2] test: Fix HTML rendering test Austin Clements
2012-11-14  2:46 ` Ethan Glasser-Camp
2012-11-15 12:11 ` David Bremner
2012-11-15 22:49 ` David Bremner

Code repositories for project(s) associated with this public inbox

	https://yhetil.org/notmuch.git/

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 read-only IMAP folder(s) and NNTP newsgroup(s).