Re: pgsql: Add quotes to message

From: Peter Eisentraut <peter_e(at)gmx(dot)net>
To: Gregory Stark <stark(at)enterprisedb(dot)com>
Cc: pgsql-hackers(at)postgresql(dot)org
Subject: Re: pgsql: Add quotes to message
Date: 2009-02-23 12:56:01
Message-ID: 49A29CE1.4000208@gmx.net
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers pgsql-hackers

Gregory Stark wrote:
> petere(at)postgresql(dot)org (Peter Eisentraut) writes:
>
>> Log Message:
>> -----------
>> Add quotes to message
>
> errdetail("Returned type %s does not match expected type "
> ! "%s in column \"%s\".",
> OidIsValid(returned->attrs[i]->atttypid) ?
> format_type_be(returned->attrs[i]->atttypid) :
> _(dropped_column_type),
>
> I'm surprised there isn't a regression test for this case.

Me too actually. ;-) In general, the plpgsql test doesn't appear to
cover the error paths very well at all. I changed the wording of about
50 error messages the other day and only two or three of those appeared
in a regression test case.

(The test coverage framework doesn't cover the PLs yet, so I don't have
anything more precise to say on that.)

In response to

Browse pgsql-committers by date

  From Date Subject
Next Message User Bmomjian 2009-02-23 14:02:13 pg-migrator - src: Use assert rather than Assert because we don't link
Previous Message Gregory Stark 2009-02-23 10:45:22 Re: pgsql: Add quotes to message

Browse pgsql-hackers by date

  From Date Subject
Next Message Grzegorz Jaskiewicz 2009-02-23 13:43:24 Re: 8.4 features presentation
Previous Message Gregory Stark 2009-02-23 10:45:22 Re: pgsql: Add quotes to message