Re: pgindent (was Re: [COMMITTERS] pgsql: Preventive maintenance in advance of pgindent run.)

Piotr Stefaniak <postgres@piotr-stefaniak.me> writes:
> On 2017-06-13 18:22, Tom Lane wrote:
>> Also, I am wondering about the test cases under tests/.  I do not
>> see anything in the Makefile or elsewhere suggesting how those are
>> to be used.  It would sure be nice to have some quick smoke-test
>> to check that a build on a new platform is working.

> They'd started out like David Holland's tests for his tradcpp(1), with a
> similar makefile (again, BSD make). But I was tenaciously asked to use
> Kyua (a testing framework that is the standard regression test mechanism
> for FreeBSD) instead, so now the makefile's existence and use is a great
> secret and the file is not under any source control. Adaption of the
> indent test suite to Kyua made the makefile more inelegant, but I'm
> attaching it to this email in hope that you can do something useful with
> it. I can only guess that you have the option to use Kyua instead, but I
> don't know the tool at all.

Ah, thanks.  I hacked up a gmake rule for this:

test: $(INDENT)
	cp $(srcdir)/tests/*.list .
	for testsrc in $(srcdir)/tests/*.0; do \
	test=`basename "$$testsrc" .0`; \
	./$(INDENT) $$testsrc $$test.out -P$(srcdir)/tests/$$test.pro || echo FAILED >>$$test.out; \
	diff -u $$testsrc.stdout $$test.out || exit 1; \
	done

and I'm getting one failure, which I don't understand:

--- ./tests/f_decls.0.stdout	2017-05-21 19:40:38.507303623 -0400
+++ f_decls.out	2017-06-14 13:28:49.212871476 -0400
@@ -1,4 +1,4 @@
-char *
+char	       *
 x(void)
 {
 	type		identifier;
@@ -13,7 +13,7 @@
 	return NULL;
 }
 
-int *
+int	       *
 y(void)
 {
 
Does that test case pass for you?

			regards, tom lane