On Mon, Nov 23, 2020 at 9:02 AM Cosmin CIORANU <cosmin.cior...@gmail.com>
wrote:

> In order to identify the issue I tried to build geoserver and run all the
> tests, but it failed miserably, I might need some things installed, but I
> do not have the time to run everything.
>

GeoServer needs nothing more than what is used to build GeoTools,
the build servers are installing JDK 8 (or 11) and maven, that's all.
"Failed miserably" is a good way to vent frustration, but does not help us
to understand
what has actually happened, please share details.

>From the build logs, the point stacker behavior changed due to your
changes, and breaks
expectations in GeoServer tests. The test in question was just expecting a
HTML table
with 2 rows, but it's getting none... maybe the exception the test was set
there to protect
against has come back, or maybe (thinking out loud) there are troubles
writing out JSONArray.

https://github.com/geoserver/geoserver/blob/master/src/wms/src/test/java/org/geoserver/wms/wms_1_1_1/GetFeatureInfoTest.java#L287


Note: The present PR represents an update that was ported locally (own
> development) in geotools x21  (geoserver 2.15) and geotools x14
> (geoserver
> 2.8) and used in production, I thought It will be beneficial for others
> to
> have the update.
>

Yep, thanks for contributing!

Cheers
Andrea


== 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.*
_______________________________________________
GeoTools-Devel mailing list
GeoTools-Devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geotools-devel

Reply via email to