8d/8785908b27fbeb43673addc83a307728867538

   1 Return-Path: <bremner@tesseract.cs.unb.ca>\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 arlo.cworth.org (Postfix) with ESMTP id 6249C6DE0FB1\r
   6  for <notmuch@notmuchmail.org>; Thu, 24 Sep 2015 04:30:33 -0700 (PDT)\r
   7 X-Virus-Scanned: Debian amavisd-new at cworth.org\r
   8 X-Spam-Flag: NO\r
   9 X-Spam-Score: 0.118\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=0.118 tagged_above=-999 required=5 tests=[AWL=0.108, \r
  12  T_HEADER_FROM_DIFFERENT_DOMAINS=0.01] autolearn=disabled\r
  13 Received: from arlo.cworth.org ([127.0.0.1])\r
  14  by localhost (arlo.cworth.org [127.0.0.1]) (amavisd-new, port 10024)\r
  15  with ESMTP id fFiEc76wXxFm for <notmuch@notmuchmail.org>;\r
  16  Thu, 24 Sep 2015 04:30:31 -0700 (PDT)\r
  17 X-Greylist: delayed 954 seconds by postgrey-1.35 at arlo;\r
  18  Thu, 24 Sep 2015 04:30:31 PDT\r
  19 Received: from gitolite.debian.net (gitolite.debian.net [87.98.215.224])\r
  20  by arlo.cworth.org (Postfix) with ESMTPS id 9634F6DE0B7C\r
  21  for <notmuch@notmuchmail.org>; Thu, 24 Sep 2015 04:30:31 -0700 (PDT)\r
  22 Received: from remotemail by gitolite.debian.net with local (Exim 4.80)\r
  23  (envelope-from <bremner@tesseract.cs.unb.ca>)\r
  24  id 1Zf4TL-0006PP-Cs; Thu, 24 Sep 2015 11:13:55 +0000\r
  25 Received: (nullmailer pid 6555 invoked by uid 1000); Thu, 24 Sep 2015\r
  26  11:13:34 -0000\r
  27 From: David Bremner <david@tethera.net>\r
  28 To: notmuch@notmuchmail.org\r
  29 Subject: [PATCH] test suite: don't consider skipped individual tests as\r
  30  failing\r
  31 Date: Thu, 24 Sep 2015 08:13:30 -0300\r
  32 Message-Id: <1443093210-6511-1-git-send-email-david@tethera.net>\r
  33 X-Mailer: git-send-email 2.5.1\r
  34 X-BeenThere: notmuch@notmuchmail.org\r
  35 X-Mailman-Version: 2.1.18\r
  36 Precedence: list\r
  37 List-Id: "Use and development of the notmuch mail system."\r
  38  <notmuch.notmuchmail.org>\r
  39 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  40  <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  41 List-Archive: <http://notmuchmail.org/pipermail/notmuch/>\r
  42 List-Post: <mailto:notmuch@notmuchmail.org>\r
  43 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  44 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  45  <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  46 X-List-Received-Date: Thu, 24 Sep 2015 11:30:33 -0000\r
  47 \r
  48 It isn't completely clear what we want to do here, but\r
  49 \r
  50 1) We currently don't fail if we skip a whole test file (mainly because\r
  51 we neglect to count those skipped tests properly). This change at least\r
  52 makes the two kinds of skipping consistent.\r
  53 \r
  54 2) Automated build environments may have good reasons for building with\r
  55 a minimal set of prereqs, and we don't want to discourage running our\r
  56 test suite by breaking builds.\r
  57 ---\r
  58  test/aggregate-results.sh | 2 +-\r
  59  1 file changed, 1 insertion(+), 1 deletion(-)\r
  60 \r
  61 diff --git a/test/aggregate-results.sh b/test/aggregate-results.sh\r
  62 index b016edb..b30da44 100755\r
  63 --- a/test/aggregate-results.sh\r
  64 +++ b/test/aggregate-results.sh\r
  65 @@ -82,7 +82,7 @@ if [ "$skipped" != "0" ]; then\r
  66      echo "$skipped $tests skipped."\r
  67  fi\r
  68  \r
  69 -if [ $success -gt 0 -a $fixed -eq 0 -a $failed -eq 0 -a $skipped -eq 0 ]\r
  70 +if [ $success -gt 0 -a $fixed -eq 0 -a $failed -eq 0 ]\r
  71  then\r
  72      exit 0\r
  73  else\r
  74 -- \r
  75 2.5.1\r
  76 \r