On Thu, Jul 30, 2015 at 5:54 AM, Andreas Karlsson <andreas@proxel.se> wrote:
> 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.
+Simple module used to check data dump ordering of pg_dump with tables
+linked with foreign keys.
The README mentions that this is to test pg_dump, perhaps referring to
the TAP tests makes sense as well?
> Another thought: 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?
That seems worth doing, hence added something for it. Thanks for the suggestion.
At the same time I have added an entry in .gitignore for the generated
tmp_check/.
Regards,
--
Michael