| От | Andreas Karlsson |
|---|---|
| Тема | Re: Improving test coverage of extensions with pg_dump |
| Дата | |
| Msg-id | 55B93D9F.7060709@proxel.se обсуждение |
| Ответ на | Re: Improving test coverage of extensions with pg_dump (Michael Paquier <michael.paquier@gmail.com>) |
| Ответы |
Re: Improving test coverage of extensions with pg_dump
|
| Список | pgsql-hackers |
I have reviewed this patch and it compiles runs and the new test case passes. The code is also clean and the test seems like a useful regression test. What I do not like though is how the path src/test/tables_fk/t/ tells us nothing about what features are of PostgreSQL are tested here. For this I personally prefer the earlier versions where I think that was clear. Another though: would it be worthwhile to also add an assertion to check if the data really was restored properly or would that just be redundant code? -- Andreas
В списке pgsql-hackers по дате отправления:
Сайт использует файлы cookie для корректной работы и повышения удобства. Нажимая кнопку «Принять» или продолжая пользоваться сайтом, вы соглашаетесь на их использование в соответствии с Политикой в отношении обработки cookie ООО «ППГ», в том числе на передачу данных из файлов cookie сторонним статистическим и рекламным службам. Вы можете управлять настройками cookie через параметры вашего браузера