From mboxrd@z Thu Jan 1 00:00:00 1970 From: =?UTF-8?q?=C3=86var=20Arnfj=C3=B6r=C3=B0=20Bjarmason?= Subject: [PATCH v2 0/4] Support for running tests outside t/ + don't run a TODO test Date: Thu, 19 Aug 2010 16:05:57 +0000 Message-ID: <1282233961-27203-1-git-send-email-avarab@gmail.com> References: <1282138473-15613-1-git-send-email-avarab@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: QUOTED-PRINTABLE Cc: Junio C Hamano , =?UTF-8?q?=C3=86var=20Arnfj=C3=B6r=C3=B0=20Bjarmason?= To: git@vger.kernel.org X-From: git-owner@vger.kernel.org Thu Aug 19 18:07:09 2010 Return-path: Envelope-to: gcvg-git-2@lo.gmane.org Received: from vger.kernel.org ([209.132.180.67]) by lo.gmane.org with esmtp (Exim 4.69) (envelope-from ) id 1Om7dv-0000My-Me for gcvg-git-2@lo.gmane.org; Thu, 19 Aug 2010 18:07:04 +0200 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752277Ab0HSQGb convert rfc822-to-quoted-printable (ORCPT ); Thu, 19 Aug 2010 12:06:31 -0400 Received: from mail-wy0-f174.google.com ([74.125.82.174]:62011 "EHLO mail-wy0-f174.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751696Ab0HSQG3 (ORCPT ); Thu, 19 Aug 2010 12:06:29 -0400 Received: by wyb32 with SMTP id 32so2365213wyb.19 for ; Thu, 19 Aug 2010 09:06:28 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=domainkey-signature:received:received:from:to:cc:subject:date :message-id:x-mailer:in-reply-to:references:mime-version :content-type:content-transfer-encoding; bh=sJfa1N/PiNzWr+LVpCwPbXffQtaadNirX7TjoTxbgfA=; b=BOH5VMiZZNW6KLAKMjRQyE+knXerkbQvL1x3k64u3EYr2cZ04k9lSmeGCehErnQKS9 A+pl2perqnoKv6fBpEvXMmEvJP3/xcEuGqh75lzZoPgr33j9Tjdc11HLrMtO5WPrBBak vKbphFAAHfOorwmQHfXLHcvJ3LAz93ZCIvshg= DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=from:to:cc:subject:date:message-id:x-mailer:in-reply-to:references :mime-version:content-type:content-transfer-encoding; b=aDK1KfKxE7JJq6I6P1felYBedD1jUadJWER907ypMyvcpVeZL93kjmOZ8xUmfld0vJ 3G5KExwavPX4+aKcY9FNeqBO/AX32NEcUJ86/ruiXoYfbOe8SoMXxV7YnswJNuEqWAs2 +27xA0Swh25DIQCHI6Z1tjOqo35A63NIADohg= Received: by 10.227.158.15 with SMTP id d15mr53048wbx.46.1282233988186; Thu, 19 Aug 2010 09:06:28 -0700 (PDT) Received: from v.nix.is (v.nix.is [109.74.193.250]) by mx.google.com with ESMTPS id q17sm470663wbe.12.2010.08.19.09.06.27 (version=SSLv3 cipher=RC4-MD5); Thu, 19 Aug 2010 09:06:27 -0700 (PDT) X-Mailer: git-send-email 1.7.2.1.446.g168052 In-Reply-To: <1282138473-15613-1-git-send-email-avarab@gmail.com> Sender: git-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: git@vger.kernel.org Archived-At: On Tue, Aug 17, 2010 at 19:12, Junio C Hamano wrote= : > Sverre Rabbelier writes: > >> E.g., something like: >> >> test_in_new_harness 0 1 0 0 ' >> test_expect_failure 'pretend we have fixed a known breakage' ' >> : >> ' >> ' > > Now the question becomes who tests "test_in_new_harness" ;-) > > I'd rather prefer not to touch this one. Isn't whatever the outer wr= apper > used flexible enough to be taught about this kind of thing and filter= it? The problem is that you have to modify *all* the outer wrappers. Since the test suite can be run with any TAP consumer that isn't viable, and at the very least having to do: prove --source=3DSome::Custom::Git Thing ... Is tedious, and requires us to maintain the Some::Custom::Git thingy. Anyway, here's a better patch. It allows us to run a pasing TODO test, check if it works, and if so not mark *that* test as passing TODO, but simply as passing. As a side benefit it makes the test lib more awesome. =C3=86var Arnfj=C3=B6r=C3=B0 Bjarmason (4): test-lib: Use $TEST_DIRECTORY or $GIT_BUILD_DIR instead of $(pwd) and ../ test-lib: Use "$GIT_BUILD_DIR" instead of "$TEST_DIRECTORY"/../ test-lib: Allow overriding of TEST_DIRECTORY t/t0000-basic.sh: Run the passing TODO test inside its own test-lib t/t0000-basic.sh | 31 +++++++++++++++++++++++++++++++ t/test-lib.sh | 39 +++++++++++++++++++++++---------------- 2 files changed, 54 insertions(+), 16 deletions(-) --=20 1.7.2.1.414.g9bf49