From 46db096209d17ff61e0e82825b2e6de28f35ff3c Mon Sep 17 00:00:00 2001 From: Jani Nikula Date: Sat, 1 Dec 2012 01:43:31 +0200 Subject: [PATCH] Re: [Patch v2 06/17] test: add broken roundtrip test --- e7/9458d2ab3e830307230de3e34001507ad1ebef | 116 ++++++++++++++++++++++ 1 file changed, 116 insertions(+) create mode 100644 e7/9458d2ab3e830307230de3e34001507ad1ebef diff --git a/e7/9458d2ab3e830307230de3e34001507ad1ebef b/e7/9458d2ab3e830307230de3e34001507ad1ebef new file mode 100644 index 000000000..4923a40d8 --- /dev/null +++ b/e7/9458d2ab3e830307230de3e34001507ad1ebef @@ -0,0 +1,116 @@ +Return-Path: +X-Original-To: notmuch@notmuchmail.org +Delivered-To: notmuch@notmuchmail.org +Received: from localhost (localhost [127.0.0.1]) + by olra.theworths.org (Postfix) with ESMTP id B9EF0431FAF + for ; Fri, 30 Nov 2012 15:43:41 -0800 (PST) +X-Virus-Scanned: Debian amavisd-new at olra.theworths.org +X-Spam-Flag: NO +X-Spam-Score: -0.7 +X-Spam-Level: +X-Spam-Status: No, score=-0.7 tagged_above=-999 required=5 + tests=[RCVD_IN_DNSWL_LOW=-0.7] autolearn=disabled +Received: from olra.theworths.org ([127.0.0.1]) + by localhost (olra.theworths.org [127.0.0.1]) (amavisd-new, port 10024) + with ESMTP id m2ydW4kYDk8P for ; + Fri, 30 Nov 2012 15:43:37 -0800 (PST) +Received: from mail-la0-f53.google.com (mail-la0-f53.google.com + [209.85.215.53]) (using TLSv1 with cipher RC4-SHA (128/128 bits)) + (No client certificate requested) + by olra.theworths.org (Postfix) with ESMTPS id A19FE431FAE + for ; Fri, 30 Nov 2012 15:43:37 -0800 (PST) +Received: by mail-la0-f53.google.com with SMTP id w12so857105lag.26 + for ; Fri, 30 Nov 2012 15:43:34 -0800 (PST) +X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; + d=google.com; s=20120113; + h=from:to:cc:subject:in-reply-to:references:user-agent:date + :message-id:mime-version:content-type:x-gm-message-state; + bh=5cERExXbjUcxbnyNi4pu8xJuRJL15NVJsudW68nj2CM=; + b=AZV00dUpHXbp6dGtI8LnGgbno7G+aqa7PsxCfxt3sfn9+x4wA9V0jNG4xJJImAtruZ + hH/3OcOEAA70U49+kzpPlNNux6kYXR8TA25sJQpYv1FHKCqKCwq6XSnYwwwiTK+qzxDF + Qw43wTvcMbnwgiiGNh0NUxWuy5yVEJnLjc0tILyqKMTaN45qyKtfknpCRen4HMlvh43l + 9534gKW/ALcWalEXtK+aYZzj9URRTk1W2eNY5I/C5JEo/uWo8RyO3E5hpzOI5XrRHPej + tU+dOOpx2maB5AqClIl5mz65FMbd7MORkm7YRo7EgGmOXaS2Y/2K90/bbT975bulDsFs + D32Q== +Received: by 10.112.29.229 with SMTP id n5mr1532817lbh.130.1354319014783; + Fri, 30 Nov 2012 15:43:34 -0800 (PST) +Received: from localhost (dsl-hkibrasgw4-fe51df00-27.dhcp.inet.fi. + [80.223.81.27]) + by mx.google.com with ESMTPS id p9sm2592758lbc.3.2012.11.30.15.43.33 + (version=SSLv3 cipher=OTHER); Fri, 30 Nov 2012 15:43:33 -0800 (PST) +From: Jani Nikula +To: david@tethera.net, notmuch@notmuchmail.org +Subject: Re: [Patch v2 06/17] test: add broken roundtrip test +In-Reply-To: <1353792017-31459-7-git-send-email-david@tethera.net> +References: <1353792017-31459-1-git-send-email-david@tethera.net> + <1353792017-31459-7-git-send-email-david@tethera.net> +User-Agent: Notmuch/0.14+124~g3b17402 (http://notmuchmail.org) Emacs/23.4.1 + (i686-pc-linux-gnu) +Date: Sat, 01 Dec 2012 01:43:31 +0200 +Message-ID: <87boeeac7w.fsf@nikula.org> +MIME-Version: 1.0 +Content-Type: text/plain; charset=us-ascii +X-Gm-Message-State: + ALoCoQn5PSURj8bAjkbHXH8xkmyy8+CLHfZuA912te23w/z6ptjSuCtOacuySzLb3YWGmUkRHbb4 +Cc: David Bremner +X-BeenThere: notmuch@notmuchmail.org +X-Mailman-Version: 2.1.13 +Precedence: list +List-Id: "Use and development of the notmuch mail system." + +List-Unsubscribe: , + +List-Archive: +List-Post: +List-Help: +List-Subscribe: , + +X-List-Received-Date: Fri, 30 Nov 2012 23:43:41 -0000 + + +BTW, patches up to and including this one could go in even if there's +still stuff to do in the following ones. + +Jani. + + +On Sat, 24 Nov 2012, david@tethera.net wrote: +> From: David Bremner +> +> We demonstrate the current notmuch restore parser being confused by +> message-id's and tags containing non alpha numeric characters +> (particularly space and parentheses are problematic because they are +> not escaped by notmuch dump). +> +> We save the files as hex escaped on disk so that the output from the +> failing test will not confuse the terminal emulator of people running +> the test. +> --- +> test/dump-restore | 9 +++++++++ +> 1 file changed, 9 insertions(+) +> +> diff --git a/test/dump-restore b/test/dump-restore +> index b05399c..a2204fb 100755 +> --- a/test/dump-restore +> +++ b/test/dump-restore +> @@ -85,4 +85,13 @@ test_begin_subtest "dump --output=outfile -- from:cworth" +> notmuch dump --output=dump-outfile-dash-inbox.actual -- from:cworth +> test_expect_equal_file dump-cworth.expected dump-outfile-dash-inbox.actual +> +> +test_expect_success 'roundtripping random message-ids and tags' \ +> + 'test_subtest_known_broken && +> + ${TEST_DIRECTORY}/random-corpus --num-messages=10 --config-path=${NOTMUCH_CONFIG} && +> + notmuch dump | ${TEST_DIRECTORY}/hex-xcode --direction=encode > EXPECTED.$test_count && +> + notmuch tag -random-corpus tag:random-corpus && +> + ${TEST_DIRECTORY}/hex-xcode --direction=decode < EXPECTED.$test_count | notmuch restore 2>/dev/null && +> + notmuch dump | ${TEST_DIRECTORY}/hex-xcode --direction=encode > OUTPUT.$test_count && +> + test_cmp EXPECTED.$test_count OUTPUT.$test_count' +> + +> test_done +> -- +> 1.7.10.4 +> +> _______________________________________________ +> notmuch mailing list +> notmuch@notmuchmail.org +> http://notmuchmail.org/mailman/listinfo/notmuch -- 2.26.2