Ing. Simone Giannecchini
@simogeo
Founder/Director
GeoSolutions S.A.S.
Via di Montramito 3/A
55054 Massarosa (LU)
Italy
phone: +39 0584 962313
fax: +39 0584 1660272
mob: +39 333 8128928
Alessio: thanks – that answers my question and allays my concerns. I am still +1 in any case.
Brad
From: Alessio Fabiani <alessio.fabiani@anonymised.com>
Sent: Friday, 31 August 2018 10:43 PM
To: Brad Hards <bradh@anonymised.com>; Emanuele Tajariol <etj@anonymised.com>
Cc: Andrea Aime <andrea.aime@anonymised.com>; Alessio Fabiani <alessio.fabiani@anonymised.com>; Geoserver-devel <geoserver-devel@lists.sourceforge.net>
Subject: Re: [Geoserver-devel] [GSIP-164] Promoting geofence and geofence-server as extensions modules
Dear all,
I agree that the test coverage might be indeed increased, nevertheless even if the percentage is not so high, notice that the core classes have a very good coverage.
As an instance the lower value (15%) on GeoFence module regards a package containing just POJOs used by the client to unmarshal instances through Spring Remoting. They are just plain classes representing some GeoFence objects ( org.geoserver.geofence.rest.xml).
On the other side most of the core classes have almost 100% coverage.
I would love (and for sure I will) add more tests on the rules and roles caching features and on the AuthenticationManager.
Until now the tests I have added should also cover:
1. The load of the context from spring
2. All the possible REST operations that can be done through the APIs
3. Most of the operations that currently are possible via the WEB GUI
For sue something missing would be:
1. More tests on the Limits filters by area
2. Tests on the possibility of making specific attributes hidden/read-only/writable
Especially for those two, manual testing of course has been done successfully. Automatic testing should be added.
If you think that it would be better to add them at this stage, I can spend some more time before promoting the module, otherwise I can collect all those observations on some specific JIRAs.
Thoughts? Feedbacks?
Also @Emanuele Tajariol as the original author of the GSIP, what do you think about that?
Thanks everyone for dedicating time to this topic.
Alessio.
Il giorno sab 25 ago 2018 alle ore 01:21 <bradh@anonymised.com> ha scritto:
I should have noted I already added my vote on the GSIP.
I was looking for Alessio’s opinion on the quality of the tests – test coverage (especially by line) isn’t the only thing that adds to value.
Brad
From: Andrea Aime <andrea.aime@anonymised.com>
Sent: Saturday, 25 August 2018 12:02 AM
To: Brad Hards <bradh@anonymised.com>
Cc: Alessio Fabiani <alessio.fabiani@anonymised.com>; Geoserver-devel <geoserver-devel@lists.sourceforge.net>
Subject: Re: [Geoserver-devel] [GSIP-164] Promoting geofence and geofence-server as extensions modules
Hi Brad,
the test coverage requirements are indeed on the low side, maybe we should discuss a separate GSIP about increasing them.
But as far as rules go, Alessio's proposal is within them.
Cheers
Andrea
On Sat, Aug 18, 2018 at 3:38 AM <bradh@anonymised.com> wrote:
The proposal wiki page still says 2.13. Are you targeting 2.14 or 2.15?
Also code coverage still seems pretty low for a security feature. Are you satisfied that the important parts have appropriate validation?
Brad
From: Alessio Fabiani <alessio.fabiani@anonymised.com>
Sent: Friday, 17 August 2018 1:57 AM
To: Geoserver-devel <geoserver-devel@lists.sourceforge.net>
Subject: [Geoserver-devel] [GSIP-164] Promoting geofence and geofence-server as extensions modules
Dear PSC,
recently I worked on making geofence modules stable and complete enough to be evaluated as official GeoServer extensions.
Currently I have updated the checklist here
https://github.com/geoserver/geoserver/wiki/GSIP-164
and pushed a set of PRs [1][2][3][4] resulting in the following code coverage results
[1] https://github.com/geoserver/geoserver/pull/3058
[2] https://github.com/geoserver/geoserver/pull/3052
[3] https://github.com/geoserver/geoserver/pull/3046
[4] https://github.com/geoserver/geoserver/pull/3024
--
==
GeoServer Professional Services from the experts! Visit http://goo.gl/it488V for more information.
Ing. Alessio Fabiani
@alfa7691
Founder/Technical Lead
GeoSolutions S.A.S.
Via di Montramito 3/A - 55054 Massarosa (LU) - Italy
phone: +39 0584 962313
fax: +39 0584 1660272
mob: +39 331 6233686
http://www.geo-solutions.it
http://twitter.com/geosolutions_it
-------------------------------------------------------
Con riferimento alla normativa sul trattamento dei dati personali (Reg. UE 2016/679 - Regolamento generale sulla protezione dei dati “GDPR”), si precisa che ogni circostanza inerente alla presente email (il suo contenuto, gli eventuali allegati, etc.) è un dato la cui conoscenza è riservata al/i solo/i destinatario/i indicati dallo scrivente. Se il messaggio Le è giunto per errore, è tenuta/o a cancellarlo, ogni altra operazione è illecita. Le sarei comunque grato se potesse darmene notizia.
This email is intended only for the person or entity to which it is addressed and may contain information that is privileged, confidential or otherwise protected from disclosure. We remind that - as provided by European Regulation 2016/679 “GDPR” - copying, dissemination or use of this e-mail or the information herein by anyone other than the intended recipient is prohibited. If you have received this email by mistake, please notify us immediately by telephone or e-mail.
------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot_______________________________________________
Geoserver-devel mailing list
Geoserver-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geoserver-devel
--
Regards, Andrea Aime == GeoServer Professional Services from the experts! Visit http://goo.gl/it488V for more information. == Ing. Andrea Aime @geowolf Technical Lead GeoSolutions S.A.S. Via di Montramito 3/A 55054 Massarosa (LU) phone: +39 0584 962313 fax: +39 0584 1660272 mob: +39 339 8844549 http://www.geo-solutions.it http://twitter.com/geosolutions_it ------------------------------------------------------- Con riferimento alla normativa sul trattamento dei dati personali (Reg. UE 2016/679 - Regolamento generale sulla protezione dei dati “GDPR”), si precisa che ogni circostanza inerente alla presente email (il suo contenuto, gli eventuali allegati, etc.) è un dato la cui conoscenza è riservata al/i solo/i destinatario/i indicati dallo scrivente. Se il messaggio Le è giunto per errore, è tenuta/o a cancellarlo, ogni altra operazione è illecita. Le sarei comunque grato se potesse darmene notizia. This email is intended only for the person or entity to which it is addressed and may contain information that is privileged, confidential or otherwise protected from disclosure. We remind that - as provided by European Regulation 2016/679 “GDPR” - copying, dissemination or use of this e-mail or the information herein by anyone other than the intended recipient is prohibited. If you have received this email by mistake, please notify us immediately by telephone or e-mail.
--
==
GeoServer Professional Services from the experts! Visit http://goo.gl/it488V for more information.
Ing. Alessio Fabiani
@alfa7691
Founder/Technical Lead
GeoSolutions S.A.S.
Via di Montramito 3/A - 55054 Massarosa (LU) - Italy
phone: +39 0584 962313
fax: +39 0584 1660272
mob: +39 331 6233686
http://www.geo-solutions.it
http://twitter.com/geosolutions_it
-------------------------------------------------------
Con riferimento alla normativa sul trattamento dei dati personali (Reg. UE 2016/679 - Regolamento generale sulla protezione dei dati “GDPR”), si precisa che ogni circostanza inerente alla presente email (il suo contenuto, gli eventuali allegati, etc.) è un dato la cui conoscenza è riservata al/i solo/i destinatario/i indicati dallo scrivente. Se il messaggio Le è giunto per errore, è tenuta/o a cancellarlo, ogni altra operazione è illecita. Le sarei comunque grato se potesse darmene notizia.
This email is intended only for the person or entity to which it is addressed and may contain information that is privileged, confidential or otherwise protected from disclosure. We remind that - as provided by European Regulation 2016/679 “GDPR” - copying, dissemination or use of this e-mail or the information herein by anyone other than the intended recipient is prohibited. If you have received this email by mistake, please notify us immediately by telephone or e-mail.
------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot_______________________________________________
Geoserver-devel mailing list
Geoserver-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geoserver-devel