From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.1 (2015-04-28) on dcvr.yhbt.net X-Spam-Level: X-Spam-ASN: AS31976 209.132.180.0/23 X-Spam-Status: No, score=-3.4 required=3.0 tests=AWL,BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,FREEMAIL_FORGED_FROMDOMAIN,FREEMAIL_FROM, HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,RCVD_IN_DNSWL_HI shortcircuit=no autolearn=ham autolearn_force=no version=3.4.1 Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by dcvr.yhbt.net (Postfix) with ESMTP id E912B1F404 for ; Wed, 12 Sep 2018 18:36:21 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1728041AbeILXmG (ORCPT ); Wed, 12 Sep 2018 19:42:06 -0400 Received: from mail-pl1-f193.google.com ([209.85.214.193]:38531 "EHLO mail-pl1-f193.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1727651AbeILXmG (ORCPT ); Wed, 12 Sep 2018 19:42:06 -0400 Received: by mail-pl1-f193.google.com with SMTP id u11-v6so1390522plq.5 for ; Wed, 12 Sep 2018 11:36:19 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding; bh=PgiUStnVmKb4M8dSzY7UGEOPkr8sDxV3Yr3Kmm4D6Po=; b=CsdhEMxGVrhjsSRqMUTw91fS0WXPk9nD9PFZr8dBAZPyWoRTcNr2mv1c+YBIZwM/sq aQoZ+1MchHTGMdZW2qvOclev/0UULzAOeUfuE9999QHM5nTW7VdS/ja748K9G8lqAwTP 0LMprfFPTVWJsiUIbSBN7Xe/W/BibhvioN1sJbciuKptxXYobPbsorqJqh40B4GvB2E1 UaMXd9fsXgLAtASA6x8TEgF/E/xhQELeoDK+CWRqRah981c407OnVzByfP8Y7ZnXm2V4 3K0cE1G27xzo/tYgRbuIxY5gbicCn6oQDKed3tkPPe8ZtCmDizNCVTfqdkyjalwyMdhX uonw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references:mime-version:content-transfer-encoding; bh=PgiUStnVmKb4M8dSzY7UGEOPkr8sDxV3Yr3Kmm4D6Po=; b=iBLys0WPLFxtdmI7NxLluzhPU32BHpSLhL6K88NHZsKPTvWuKkMQdZP6MuQcaAEp0C oU4LVlI8vZlUoL6GIvL32xkkPFeWH2askLCjYYITb0OgB5yndA/g5eh22M4g/AWQZHZX K3egEkB+cEQWT5+etzz6oJPp/GDjnwWStVP4AIitZiryfwNdSE/3ALGrUbyfPOTWCeDF sgJ5qcDwvKxeX+VEEn5ypIg/mHZgqwUfW9GoUk7vWrVNolKEb4EJN1YUTqt/HcMqDKcP uTn/lHPDyXODIpCoKe6jGA51CWsG3R1tInIzadjKw6IXonLPdipWDBAk9kllsL48b4pO AXqQ== X-Gm-Message-State: APzg51A+nZhDbcqd75xH+5YgGgZBmVSB0boyz8eT94+ewXQQZjVJNag4 vieMt/5yU+5Xg+pKG8fwzPMDDtPD X-Google-Smtp-Source: ANB0VdblelxsQd3C6/3CTosjbcGewo1U8mIX5mhX+nTpr+jgcMoN0Y8VKILeK4Y8O8XwW+Q5r7bnVw== X-Received: by 2002:a17:902:4503:: with SMTP id m3-v6mr3705678pld.168.1536777379427; Wed, 12 Sep 2018 11:36:19 -0700 (PDT) Received: from localhost.localdomain (c-76-102-118-240.hsd1.ca.comcast.net. [76.102.118.240]) by smtp.gmail.com with ESMTPSA id x23-v6sm2817983pff.9.2018.09.12.11.36.18 (version=TLS1_2 cipher=ECDHE-RSA-AES128-SHA bits=128/128); Wed, 12 Sep 2018 11:36:18 -0700 (PDT) From: William Chargin To: sunshine@sunshineco.com Cc: git@vger.kernel.org, jrnieder@gmail.com, peff@peff.net, wchargin@gmail.com Subject: [PATCH v4] test_dir_is_empty: fix edge cases with newlines and hyphens Date: Wed, 12 Sep 2018 11:35:34 -0700 Message-Id: <20180912183534.7486-1-wchargin@gmail.com> X-Mailer: git-send-email 2.18.0.549.gd66323a05 In-Reply-To: References: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Sender: git-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: git@vger.kernel.org While the `test_dir_is_empty` function appears correct in most normal use cases, it can improperly pass if a directory contains a filename with a newline, and can improperly fail if an empty directory looks like an argument to `ls`. This patch changes the implementation to check that the output of `ls -a` has at most two lines (for `.` and `..`), which should be better behaved, and adds the `--` delimiter before the directory name when invoking `ls`. The newly added unit test fails before this change and passes after it. Signed-off-by: William Chargin --- This patch depends on "t: factor out FUNNYNAMES as shared lazy prereq" (2018-08-06), which is now in master. I originally wrote this patch for the standalone Sharness library, but that library advises that such patches be sent to the Git mailing list first. Tested on GNU/Linux (Mint 18.2) and macOS (10.13). t/t0000-basic.sh | 43 +++++++++++++++++++++++++++++++++++++++++ t/test-lib-functions.sh | 2 +- 2 files changed, 44 insertions(+), 1 deletion(-) diff --git a/t/t0000-basic.sh b/t/t0000-basic.sh index 850f651e4e..a5c57c6aa5 100755 --- a/t/t0000-basic.sh +++ b/t/t0000-basic.sh @@ -821,6 +821,49 @@ test_expect_success 'tests clean up even on failures' " EOF " +test_expect_success FUNNYNAMES \ + 'test_dir_is_empty behaves even in pathological cases' " + run_sub_test_lib_test \ + dir-empty 'behavior of test_dir_is_empty' <<-\\EOF && + test_expect_success 'should pass with actually empty directory' ' + mkdir empty_dir && + test_dir_is_empty empty_dir + ' + test_expect_success 'should fail with a normal filename' ' + mkdir nonempty_dir && + >nonempty_dir/some_file && + ! test_dir_is_empty nonempty_dir + ' + test_expect_success 'should fail with dot-newline-dot filename' ' + mkdir pathological_dir && + >\"pathological_dir/. + .\" && + ! test_dir_is_empty pathological_dir + ' + test_expect_success 'should pass with an empty directory \"-l\"' ' + mkdir -- -l && + test_dir_is_empty -l && + rmdir -- -l + ' + test_expect_success 'should pass with an empty directory \"--wat\"' ' + mkdir -- --wat && + test_dir_is_empty --wat && + rmdir -- --wat + ' + test_done + EOF + check_sub_test_lib_test dir-empty <<-\\EOF + > ok 1 - should pass with actually empty directory + > ok 2 - should fail with a normal filename + > ok 3 - should fail with dot-newline-dot filename + > ok 4 - should pass with an empty directory \"-l\" + > ok 5 - should pass with an empty directory \"--wat\" + > # passed all 5 test(s) + > 1..5 + EOF +" + + ################################################################ # Basics of the basics diff --git a/t/test-lib-functions.sh b/t/test-lib-functions.sh index 4207af4077..3df6b8027f 100644 --- a/t/test-lib-functions.sh +++ b/t/test-lib-functions.sh @@ -576,7 +576,7 @@ test_path_exists () { # Check if the directory exists and is empty as expected, barf otherwise. test_dir_is_empty () { test_path_is_dir "$1" && - if test -n "$(ls -a1 "$1" | egrep -v '^\.\.?$')" + if test "$(ls -a1 -- "$1" | wc -l)" -gt 2 then echo "Directory '$1' is not empty, it contains:" ls -la "$1" -- 2.18.0.549.gd66323a05