On 27 March 2015 at 17:53, Vadim Zeitlin <[email protected]> wrote:
> On Fri, 27 Mar 2015 10:05:32 +0100 Mateusz Loskot <[email protected]> wrote:
>
> ML> > OK, thanks for your support! I won't be able to finish this today, but
> ML> > I'll try to do it tomorrow or, if this fails too, early next week. One
> way
> ML> > or the other, I really want to get CATCH in.
> ML>
> ML> No problem, sounds good.
>
> OK, this is done now, see https://github.com/SOCI/soci/pull/298
>
> I'd like to merge this soon
+1
> and then also rename/move the tests as the
> "assert" part, which annoyed me even before, just totally won't make any
> sense after switching to CATCH.
+1
> My favourite solution would be to just move the tests one level up in the
> file system, i.e. get rid of the subdirectory completely. But we could also
> keep the hierarchy and just rename it to something else, especially if
> somebody has any good proposals for the subdirectory name ("catch" doesn't
> count, because it doesn't really matter what do the tests use for their
> checks, the name should rather describe what do they do and not how).
Two options which are equally good to me:
1. Move tests/assert/* one level up to tests/
2. Rename tests/assert to tests/{system|functional|integrate}, which may open
the structure for other test categories (e.g. plain unit tests that
don't test backends functional
integration with any physical DB env).
Please, choose whichever suits you.
Best regards,
--
Mateusz Loskot, http://mateusz.loskot.net
------------------------------------------------------------------------------
Dive into the World of Parallel Programming The Go Parallel Website, sponsored
by Intel and developed in partnership with Slashdot Media, is your hub for all
things parallel software development, from weekly thought leadership blogs to
news, videos, case studies, tutorials and more. Take a look and join the
conversation now. http://goparallel.sourceforge.net/
_______________________________________________
soci-users mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/soci-users
