Received: from malur.postgresql.org ([217.196.149.56]) by arkaria.postgresql.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_CBC_SHA1:256) (Exim 4.89) (envelope-from ) id 1iZjNk-0003Py-SG for pgsql-hackers@arkaria.postgresql.org; Tue, 26 Nov 2019 22:32:29 +0000 Received: from localhost ([127.0.0.1] helo=malur.postgresql.org) by malur.postgresql.org with esmtp (Exim 4.89) (envelope-from ) id 1iZjNj-0004aW-4n for pgsql-hackers@arkaria.postgresql.org; Tue, 26 Nov 2019 22:32:27 +0000 Received: from makus.postgresql.org ([2001:4800:3e1:1::229]) by malur.postgresql.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_CBC_SHA1:256) (Exim 4.89) (envelope-from ) id 1iZjNi-0004aP-Oj for pgsql-hackers@lists.postgresql.org; Tue, 26 Nov 2019 22:32:26 +0000 Received: from sss.pgh.pa.us ([66.207.139.130]) by makus.postgresql.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1iZjNc-00063k-3p for pgsql-hackers@postgresql.org; Tue, 26 Nov 2019 22:32:25 +0000 Received: from sss1.sss.pgh.pa.us (localhost [127.0.0.1]) by sss.pgh.pa.us (8.14.4/8.14.4) with ESMTP id xAQMWGKP003771; Tue, 26 Nov 2019 17:32:16 -0500 From: Tom Lane To: Peter Eisentraut cc: Stephen Frost , Robert Haas , Alvaro Herrera , "pgsql-hackers@postgresql.org" Subject: Re: [HACKERS] Regression tests vs existing users in an installation In-reply-to: <499f2909-d6e4-4f50-d88a-0e3dc9530f4e@2ndquadrant.com> References: <20160716035456.GA181422@alvherre.pgsql> <25245.1468683514@sss.pgh.pa.us> <23690.1468854782@sss.pgh.pa.us> <1382.1561520002@sss.pgh.pa.us> <27282.1561668606@sss.pgh.pa.us> <20190628154110.GM2480@tamriel.snowman.net> <25897.1561828908@sss.pgh.pa.us> <499f2909-d6e4-4f50-d88a-0e3dc9530f4e@2ndquadrant.com> Comments: In-reply-to Peter Eisentraut message dated "Tue, 26 Nov 2019 22:10:15 +0100" MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-ID: <3769.1574807536.1@sss.pgh.pa.us> Date: Tue, 26 Nov 2019 17:32:16 -0500 Message-ID: <3770.1574807536@sss.pgh.pa.us> List-Id: List-Help: List-Subscribe: List-Post: List-Owner: List-Archive: Precedence: bulk Peter Eisentraut writes: > I found this directory, and it seems like a good place to add some more > tests (other thread), but I'm puzzled why it's hidden under modules/, > since it's not, well, a module. Why is it not in src/test/? Doing it like this allowed it to be picked up automatically by the buildfarm script. If we'd put it in a new src/test directory, the buildfarm script would have had to be taught about that, and we'd have no coverage until owners updated their machines. So it seemed expedient to do it like this. regards, tom lane