55/b716c03653fbaee1f1604f8efd763afdabd78a

   1 Return-Path: <jrollins@finestructure.net>\r
   2 X-Original-To: notmuch@notmuchmail.org\r
   3 Delivered-To: notmuch@notmuchmail.org\r
   4 Received: from localhost (localhost [127.0.0.1])\r
   5         by olra.theworths.org (Postfix) with ESMTP id 0A6B4431FAF\r
   6         for <notmuch@notmuchmail.org>; Sat, 31 Mar 2012 14:47:11 -0700 (PDT)\r
   7 X-Virus-Scanned: Debian amavisd-new at olra.theworths.org\r
   8 X-Spam-Flag: NO\r
   9 X-Spam-Score: -2.29\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-2.29 tagged_above=-999 required=5\r
  12         tests=[RCVD_IN_DNSWL_MED=-2.3, T_MIME_NO_TEXT=0.01] autolearn=disabled\r
  13 Received: from olra.theworths.org ([127.0.0.1])\r
  14         by localhost (olra.theworths.org [127.0.0.1]) (amavisd-new, port 10024)\r
  15         with ESMTP id LgG2oZBcA2t5 for <notmuch@notmuchmail.org>;\r
  16         Sat, 31 Mar 2012 14:47:10 -0700 (PDT)\r
  17 Received: from outgoing-mail.its.caltech.edu (outgoing-mail.its.caltech.edu\r
  18         [131.215.239.19])\r
  19         by olra.theworths.org (Postfix) with ESMTP id 96B28431FAE\r
  20         for <notmuch@notmuchmail.org>; Sat, 31 Mar 2012 14:47:10 -0700 (PDT)\r
  21 Received: from fire-doxen.imss.caltech.edu (localhost [127.0.0.1])\r
  22         by fire-doxen-postvirus (Postfix) with ESMTP id 141752E50B6E;\r
  23         Sat, 31 Mar 2012 14:47:08 -0700 (PDT)\r
  24 X-Spam-Scanned: at Caltech-IMSS on fire-doxen by amavisd-new\r
  25 Received: from finestructure.net (unknown [76.89.193.65])\r
  26         (Authenticated sender: jrollins)\r
  27         by fire-doxen-submit (Postfix) with ESMTP id 53CFF2E50D08;\r
  28         Sat, 31 Mar 2012 14:47:04 -0700 (PDT)\r
  29 Received: by finestructure.net (Postfix, from userid 1000)\r
  30         id E2FE22F5; Sat, 31 Mar 2012 14:47:03 -0700 (PDT)\r
  31 From: Jameson Graef Rollins <jrollins@finestructure.net>\r
  32 To: Mark Walters <markwalters1009@gmail.com>, Peter Wang <novalazy@gmail.com>,\r
  33         notmuch@notmuchmail.org\r
  34 Subject: Re: [PATCH v2 2/5] test: Add tests for 'config' command\r
  35 In-Reply-To: <877gy1p622.fsf@qmul.ac.uk>\r
  36 References: <1332282698-7951-1-git-send-email-novalazy@gmail.com>\r
  37         <1333149350-22616-1-git-send-email-novalazy@gmail.com>\r
  38         <1333149350-22616-3-git-send-email-novalazy@gmail.com>\r
  39         <877gy1p622.fsf@qmul.ac.uk>\r
  40 User-Agent: Notmuch/0.12+93~g9442054 (http://notmuchmail.org) Emacs/23.3.1\r
  41         (x86_64-pc-linux-gnu)\r
  42 Date: Sat, 31 Mar 2012 14:47:01 -0700\r
  43 Message-ID: <87wr60moiy.fsf@servo.finestructure.net>\r
  44 MIME-Version: 1.0\r
  45 Content-Type: multipart/signed; boundary="=-=-=";\r
  46         micalg=pgp-sha256; protocol="application/pgp-signature"\r
  47 X-BeenThere: notmuch@notmuchmail.org\r
  48 X-Mailman-Version: 2.1.13\r
  49 Precedence: list\r
  50 List-Id: "Use and development of the notmuch mail system."\r
  51         <notmuch.notmuchmail.org>\r
  52 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  53         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  54 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  55 List-Post: <mailto:notmuch@notmuchmail.org>\r
  56 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  57 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  58         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  59 X-List-Received-Date: Sat, 31 Mar 2012 21:47:11 -0000\r
  60 \r
  61 --=-=-=\r
  62 \r
  63 On Sat, Mar 31 2012, Mark Walters <markwalters1009@gmail.com> wrote:\r
  64 > It seems off to call is success without checking that the value has\r
  65 > actually been set. Of course it is checked in the notmuch config list\r
  66 > test introduced in the next commit but I think if it would be better to\r
  67 > check with notmuch config get here too (i.e. check that reading back the\r
  68 > value gives what you want). Otherwise a failure in `setting' will show up\r
  69 > as a test failure in `listing'.\r
  70 \r
  71 Hey, Peter.  I think Mark makes a good point here.  I think it would\r
  72 make more sense for the test to set the value, and then check that the\r
  73 value is properly set as expected.  It would make the tests multi-step,\r
  74 but that's fine.  There's plenty of precedent for that.\r
  75 \r
  76 jamie.\r
  77 \r
  78 --=-=-=\r
  79 Content-Type: application/pgp-signature\r
  80 \r
  81 -----BEGIN PGP SIGNATURE-----\r
  82 Version: GnuPG v1.4.12 (GNU/Linux)\r
  83 \r
  84 iQIcBAEBCAAGBQJPd3tVAAoJEO00zqvie6q84E8P+gKOwSmKz8OyXLQPqr6BL6b8\r
  85 q9KTnad3cVfNSY8aY0fFAYI1RAEMFQSejrdxjir/JniJCZvnNOe1g9gxvCHijXmM\r
  86 sBTA2LQCUWndx7jJLDXMmvP2764JuuBtm3Qr30CEIyUTGUkmfmzuJsPSlPkzlf7e\r
  87 01QLEmOwcCqleh0fziCDwb9oD67d6kPUi5EB6E+3lvgxjQMXS9eogBDdb2nchsbM\r
  88 7JV8y+c/2dIhWxOnKYqcAVDqO3BJujQXvID3U4RlvydE2d+t8dfBfeZnkSiBL+IR\r
  89 /I179cgqg6sPKaNFpzYI+BIaegwki9mfYtjp5hKUsRYRsuyo8cmMIGhYpCagHf4J\r
  90 /FUilqIdIeO8D4/4MY/OuNiqQZ6dN2reFFEwJGuhmqyrWa4QgVcUjbIsjQI73oqX\r
  91 l59GKzEw+pjbOgxfpb+gGsGBkgWXQJ5TQVB3n276UNtN2yscmJlXTfBHNZwdWuCe\r
  92 OW5ZNsB6nl++cgy3VTOVKWbz/SwZXHdbll7Bwj2NqnNfasIW0AR8bqCwEhgZIpXO\r
  93 3fd6kyy634UUz9L+jm1geQ+YTi+2+2Kibm9AMPew6/rGEgxGAllXXZfPEqgYYVqf\r
  94 2w40dIzHJ/eK5ahAFFgOlmhhUuc5KMadJ5Jb6rAXx3bMkwJUKDKkYmeZCyaah5Ft\r
  95 Dl2fgLDKLe4vn50lUw0w\r
  96 =uW5e\r
  97 -----END PGP SIGNATURE-----\r
  98 --=-=-=--\r