unofficial mirror of notmuch@notmuchmail.org
 help / color / mirror / code / Atom feed
* [PATCH] test: Make generated message date a real date
@ 2011-12-29  2:34 Austin Clements
  2011-12-29  4:53 ` Tomi Ollila
                   ` (2 more replies)
  0 siblings, 3 replies; 4+ messages in thread
From: Austin Clements @ 2011-12-29  2:34 UTC (permalink / raw)
  To: notmuch

January 5, 2001 was a Tuesday, not a Friday.  Jameson fixed this exact
problem for the multipart test in ec2b0a98cc, but not for
generate_message itself.

As Jameson pointed out in ec2b0a98cc, if we want to test date parsing,
we should do it separately.
---
 test/emacs        |    8 ++++----
 test/encoding     |    2 +-
 test/maildir-sync |    4 ++--
 test/raw          |    4 ++--
 test/test-lib.sh  |    2 +-
 5 files changed, 10 insertions(+), 10 deletions(-)

diff --git a/test/emacs b/test/emacs
index ca82445..a06c223 100755
--- a/test/emacs
+++ b/test/emacs
@@ -81,7 +81,7 @@ cat <<EOF >EXPECTED
 "Invalid " From" <test_suite@notmuchmail.org> (2001-01-05) (inbox)
 Subject: message-with-invalid-from
 To: Notmuch Test Suite <test_suite@notmuchmail.org>
-Date: Tue, 05 Jan 2001 15:43:57 -0000
+Date: Fri, 05 Jan 2001 15:43:57 +0000
 
 This is just a test message (#1)
 EOF
@@ -320,7 +320,7 @@ add_message '[from]="Top Poster <top@poster.com>"' \
 ----- Original Message -----
 From: Notmuch Test Suite <test_suite@notmuchmail.org>
 To: Notmuch Test Suite <test_suite@notmuchmai.org>
-Sent: Tue, 05 Jan 2001 15:43:57 -0000
+Sent: Fri, 05 Jan 2001 15:43:57 +0000
 Subject: The problem with top-posting
 
 Q: Why is top-posting such a bad thing?
@@ -331,7 +331,7 @@ test_emacs "(notmuch-show \"top-posting\")
 echo "Notmuch Test Suite <test_suite@notmuchmail.org> (2001-01-05) (inbox)
 Subject: The problem with top-posting
 To: Notmuch Test Suite <test_suite@notmuchmail.org>
-Date: Tue, 05 Jan 2001 15:43:57 -0000
+Date: Fri, 05 Jan 2001 15:43:57 +0000
 
 A: Because it messes up the order in which people normally read text.
 Q: Why is top-posting such a bad thing?
@@ -340,7 +340,7 @@ Q: What is the most annoying thing in e-mail?
 Top Poster <top@poster.com> (2001-01-05) (inbox unread)
 Subject: Re: The problem with top-posting
 To: Notmuch Test Suite <test_suite@notmuchmail.org>
-Date: Tue, 05 Jan 2001 15:43:57 -0000
+Date: Fri, 05 Jan 2001 15:43:57 +0000
 
 Thanks for the advice! I will be sure to put it to good use.
 
diff --git a/test/encoding b/test/encoding
index 673b039..e875c8b 100755
--- a/test/encoding
+++ b/test/encoding
@@ -12,7 +12,7 @@ Notmuch Test Suite <test_suite@notmuchmail.org> (2001-01-05) (inbox unread)
 Subject: Test message #1
 From: Notmuch Test Suite <test_suite@notmuchmail.org>
 To: Notmuch Test Suite <test_suite@notmuchmail.org>
-Date: Tue, 05 Jan 2001 15:43:57 -0000
+Date: Fri, 05 Jan 2001 15:43:57 +0000
 \fheader}
 \fbody{
 \fpart{ ID: 1, Content-type: text/plain
diff --git a/test/maildir-sync b/test/maildir-sync
index a60854f..d5872a5 100755
--- a/test/maildir-sync
+++ b/test/maildir-sync
@@ -55,8 +55,8 @@ test_expect_equal "$output" '[[[{"id": "adding-replied-tag@notmuch-test-suite",
 "To": "Notmuch Test Suite <test_suite@notmuchmail.org>",
 "Cc": "",
 "Bcc": "",
-"Date": "Tue,
-05 Jan 2001 15:43:57 -0000"},
+"Date": "Fri,
+05 Jan 2001 15:43:57 +0000"},
 "body": [{"id": 1,
 "content-type": "text/plain",
 "content": "This is just a test message (#3)\n"}]},
diff --git a/test/raw b/test/raw
index 99d3a3b..0171e64 100755
--- a/test/raw
+++ b/test/raw
@@ -19,7 +19,7 @@ test_expect_equal "$output" "From: Notmuch Test Suite <test_suite@notmuchmail.or
 To: Notmuch Test Suite <test_suite@notmuchmail.org>
 Message-Id: <msg-001@notmuch-test-suite>
 Subject: Test message #1
-Date: Tue, 05 Jan 2001 15:43:57 -0000
+Date: Fri, 05 Jan 2001 15:43:57 +0000
 
 This is just a test message (#1)"
 
@@ -29,7 +29,7 @@ test_expect_equal "$output" "From: Notmuch Test Suite <test_suite@notmuchmail.or
 To: Notmuch Test Suite <test_suite@notmuchmail.org>
 Message-Id: <msg-002@notmuch-test-suite>
 Subject: Test message #2
-Date: Tue, 05 Jan 2001 15:43:57 -0000
+Date: Fri, 05 Jan 2001 15:43:57 +0000
 
 This is just a test message (#2)"
 
diff --git a/test/test-lib.sh b/test/test-lib.sh
index 8edf256..862095c 100644
--- a/test/test-lib.sh
+++ b/test/test-lib.sh
@@ -323,7 +323,7 @@ generate_message ()
     fi
 
     if [ -z "${template[date]}" ]; then
-	template[date]="Tue, 05 Jan 2001 15:43:57 -0000"
+	template[date]="Fri, 05 Jan 2001 15:43:57 +0000"
     fi
 
     additional_headers=""
-- 
1.7.7.3

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

* Re: [PATCH] test: Make generated message date a real date
  2011-12-29  2:34 [PATCH] test: Make generated message date a real date Austin Clements
@ 2011-12-29  4:53 ` Tomi Ollila
  2011-12-29  8:59 ` Jameson Graef Rollins
  2011-12-29 22:06 ` David Bremner
  2 siblings, 0 replies; 4+ messages in thread
From: Tomi Ollila @ 2011-12-29  4:53 UTC (permalink / raw)
  To: Austin Clements, notmuch

On Wed, 28 Dec 2011 21:34:43 -0500, Austin Clements <amdragon@MIT.EDU> wrote:
> January 5, 2001 was a Tuesday, not a Friday.  Jameson fixed this exact
> problem for the multipart test in ec2b0a98cc, but not for
> generate_message itself.
> 
> As Jameson pointed out in ec2b0a98cc, if we want to test date parsing,
> we should do it separately.
> ---
>  test/emacs        |    8 ++++----
>  test/encoding     |    2 +-
>  test/maildir-sync |    4 ++--
>  test/raw          |    4 ++--
>  test/test-lib.sh  |    2 +-
>  5 files changed, 10 insertions(+), 10 deletions(-)
> 

+1

Tomi

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

* Re: [PATCH] test: Make generated message date a real date
  2011-12-29  2:34 [PATCH] test: Make generated message date a real date Austin Clements
  2011-12-29  4:53 ` Tomi Ollila
@ 2011-12-29  8:59 ` Jameson Graef Rollins
  2011-12-29 22:06 ` David Bremner
  2 siblings, 0 replies; 4+ messages in thread
From: Jameson Graef Rollins @ 2011-12-29  8:59 UTC (permalink / raw)
  To: Austin Clements, notmuch

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

On Wed, 28 Dec 2011 21:34:43 -0500, Austin Clements <amdragon@MIT.EDU> wrote:
> January 5, 2001 was a Tuesday, not a Friday.  Jameson fixed this exact
> problem for the multipart test in ec2b0a98cc, but not for
> generate_message itself.

In retrospect, I should have done a grep for that date string at the
time.  Thanks for the catch, Austin.

jamie.

[-- Attachment #2: Type: application/pgp-signature, Size: 835 bytes --]

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

* Re: [PATCH] test: Make generated message date a real date
  2011-12-29  2:34 [PATCH] test: Make generated message date a real date Austin Clements
  2011-12-29  4:53 ` Tomi Ollila
  2011-12-29  8:59 ` Jameson Graef Rollins
@ 2011-12-29 22:06 ` David Bremner
  2 siblings, 0 replies; 4+ messages in thread
From: David Bremner @ 2011-12-29 22:06 UTC (permalink / raw)
  To: Austin Clements, notmuch

On Wed, 28 Dec 2011 21:34:43 -0500, Austin Clements <amdragon@MIT.EDU> wrote:
> January 5, 2001 was a Tuesday, not a Friday.  Jameson fixed this exact
> problem for the multipart test in ec2b0a98cc, but not for
> generate_message itself.

Pushed, 

d

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

end of thread, other threads:[~2011-12-29 22:06 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-12-29  2:34 [PATCH] test: Make generated message date a real date Austin Clements
2011-12-29  4:53 ` Tomi Ollila
2011-12-29  8:59 ` Jameson Graef Rollins
2011-12-29 22:06 ` 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).