Re: Add regression tests for ROLE (USER)

Поиск
Список
Период
Сортировка
От Tom Lane
Тема Re: Add regression tests for ROLE (USER)
Дата
Msg-id 16000.1373209909@sss.pgh.pa.us
обсуждение исходный текст
Ответ на Re: Add regression tests for ROLE (USER)  (Fabien COELHO <coelho@cri.ensmp.fr>)
Ответы Re: Add regression tests for ROLE (USER)
Re: Add regression tests for ROLE (USER)
Список pgsql-hackers
Fabien COELHO <coelho@cri.ensmp.fr> writes:
>> Generally, I think that the tests which return a syntax error are of 
>> limited value and should probably be dropped.

> I think that it is not that simple: it is a good value to check that the 
> syntax error message conveys a useful information for the user, and that 
> changes to the parser rules do not alter good quality error messages.

It's good to check those things when a feature is implemented.  However,
once it's done, the odds of the bison parser breaking are very low.
Thus, the benefit of testing that over again thousands of times a day
is pretty tiny.

> Moreover, the cost of such tests in time must be quite minimal.

I'm not convinced (see above) and in any case the benefit is even more
minimal.

(Note that semantic errors, as opposed to syntax errors, are a different
question.)
        regards, tom lane



В списке pgsql-hackers по дате отправления:

Предыдущее
От: Robins Tharakan
Дата:
Сообщение: Re: Add more regression tests for dbcommands
Следующее
От: Andres Freund
Дата:
Сообщение: Re: Add regression tests for ROLE (USER)