Hello,
I fixed a bug in geoserver app-schema
(https://github.com/geotools/geotools/pull/3367
https://osgeo-org.atlassian.net/browse/GEOT-6801). The change I made was
very small (one character actually) and in one specific class. To me it
seemed appropriate to write a unit test. Since it is JDBC an integration
test would only be run online with db, but still require a lot of amount
of work to create the right table, mapping etc... Mocking seems useful
to me.
Anyway, because one of the classes I need is final, I had to use
PowerMock to write the test. The Reviewer of the patch ask to check with
the list if it is okay to add the PowerMock library to the test scope. I
know traditionally we have more integration tests than unit tests, but I
think it wouldn't be bad to add more unit tests in general.
Kind Regards
Niels
_______________________________________________
Geoserver-devel mailing list
Geoserver-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geoserver-devel
