Have you guys considered using testcontainers[1] for this type of test? The 
only requirement is a local docker install. I've found it very useful for 
testing postgis in a repeatable manner, both in CI and locally.

Thanks,

[1] https://java.testcontainers.org/modules/databases/postgres/

Emilio Lahr-Vivaz
General Atomics, CCRi
________________________________
From: Andrea Aime <andrea.a...@geosolutionsgroup.com>
Sent: Tuesday, July 9, 2024 4:19 AM
To: Jody Garnett <jody.garn...@gmail.com>
Cc: Geoserver-devel <geoserver-devel@lists.sourceforge.net>
Subject: -EXT-Re: [Geoserver-devel] build.geoserver 
geoserver-main-app-schema-online job maintenance / discussion.


WARNING:  This message is from an external source.  Evaluate the message 
carefully BEFORE clicking on links or opening attachments.

And oh, my local postgis database has the postgis extensions installed.


Regards,

Andrea Aime


==
GeoServer Professional Services from the experts!

Visit http://bit.ly/gs-services-us for more information.
==

Ing. Andrea Aime
@geowolf
Technical Lead


GeoSolutions Group
phone: +39 0584 962313

fax:     +39 0584 1660272

mob:   +39  339 8844549


https://www.geosolutionsgroup.com/

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


On Tue, Jul 9, 2024 at 10:18 AM Andrea Aime 
<andrea.a...@geosolutionsgroup.com<mailto:andrea.a...@geosolutionsgroup.com>> 
wrote:
Hi Jody,
the same way you run other online test cases in GeoTools.
For postgis, locally I have the following file in ~/.geoserver:

> cat postgis.properties
#This is an example fixture. Update the values and remove the .example suffix 
to enable the test
#Mon Jan 23 12:28:28 CET 2023
password=mysecret
database=app-schema
driver=org.postgresql.Driver
passwd=mysecret
port=5432
host=localhost
dbtype=postgisng
user=myuser
url=jdbc\:postgresql\://localhost/app-schema

And then with maven in the path:

cd extension/app-schema/app-schema-postgis-test/
mvn clean install  -Ponline -nsu -fae

If you had everything properly set up, the build should end with:

[INFO] Results:
[INFO]
[WARNING] Tests run: 319, Failures: 0, Errors: 0, Skipped: 1

I don't have an Oracle ready for usage but the postgis ones already provide a 
good coverage of
the common cases.



Regards,

Andrea Aime


==
GeoServer Professional Services from the experts!

Visit http://bit.ly/gs-services-us for more information.
==

Ing. Andrea Aime
@geowolf
Technical Lead


GeoSolutions Group
phone: +39 0584 962313

fax:     +39 0584 1660272

mob:   +39  339 8844549


https://www.geosolutionsgroup.com/

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


On Tue, Jul 9, 2024 at 8:29 AM Jody Garnett 
<jody.garn...@gmail.com<mailto:jody.garn...@gmail.com>> wrote:
Niels,

How do you run the app-schema tests?


  1.  I create a ~/.geoserver/postgis.properties with connection parameters 
including a database name
  2.  And then run with:
mvn clean install -Papp-schema-online-test

When I run I get errors such as:

[ERROR] testNoPrimaryKey(org.geoserver.test.onlineTest.WfsOnlinePostgisTest)  
Time elapsed: 0.294 s  <<< ERROR!
org.postgresql.util.PSQLException:
ERROR: function public.ndims(public.geometry) does not exist
  Hint: No function matches the given name and argument types. You might need 
to add explicit type casts.
  Position: 355

Looking at the build server job configuration I am shocked to see this is 
defined as part of the script:

#missing functions required for refdataset sql but removed in postgis 2.1
export PGPASSWORD=''
psql -h localhost -p 5432 -U geotools -d geotoolstest -c 'create or replace 
function public.ndims(g geometry) returns smallint as $$ begin return 
public.st_ndims(g); end; $$ language plpgsql immutable strict;'
psql -h localhost -p 5432 -U geotools -d geotoolstest -c 'create or replace 
function public.srid(g geometry) returns integer as $$ begin return 
public.st_srid(g); end; $$ language plpgsql immutable strict;'

Surely that cannot be right?
Is defining ndims part of the app schema installation instructions? No...

I How do you run the tests locally...
--
Jody Garnett


On Jul 8, 2024 at 5:29:42 AM, Niels Charlier 
<ni...@scitus.be<mailto:ni...@scitus.be>> wrote:


Hello Jody

I have always run those tests locally whenever making changes that affect 
app-schema, not sure about Jenkins issues.

Kind Regards

Niels

On 06/07/2024 00:43, Jody Garnett wrote:
I have a 
PR<https://dficcfc.r.af.d.sendibt2.com/tr/cl/QnmEf4CiLzefQ8tvswpygu_mKOoVQYE_ymKm8YxI8KlC5Oky3xgemTOMFlheDxw7yfA0Ckx8oGqw94a_hRDOlVGXZewXdciN7xvD7M_rEd1R1rtu2mMm59pwZkkH2ZiMCWaeHCpmMWjPZd9E9F6M7TJtVQYL6Juaazw0CeqmAs90WSKZg5UI6sm8-3kOEC137-01Uu5tc2aY7FtDR-RwG1pkloVK2u99LweEcTF6j2KySuBjL90_-4IjweVKflwwU6cfARFQDGeH5iSioGKRuHZ7ZlMdX-73ykakFUTCQcrnS-EW>
 that looks at some gaps in the GeoTools feature model around validation... and 
Niels made the perfectly sensible recommendation to run the app-schema online 
tests...


  1.  Checking 
build.geoserver.org<https://dficcfc.r.af.d.sendibt2.com/tr/cl/XzwiyRYFMgufdjTffCOZKuLas1HzYo0A3G72jOEFBlOrZhk15PUdzFckDA1jFwcllUEiVQf6DKw68XxItKGLGa_2j5AP4TDPcUVDSLIaM6ZZkc1fnToljiZcp6hoYtejFQ9NY3IFWDnwTnMoyGKjIND376ahzpSRmwOuyi-qg9HKc_mfbNkdj2ZiWqCxPnIqI3RM8HSIx-C2D5RqmkXtDuQ-f-Ihutyv_OxLvVyT-j0MrOkLGFf2fr-6nKMioMiX750R24zo5Vw>
  the 
geoserver-main-app-schema-online<https://dficcfc.r.af.d.sendibt2.com/tr/cl/Ds5yMP0W1nLQwCo3m3ZabKQg8Jh23U86KCnuN506HyWCU_4ODqDvVJrIJzYbBRDZwAuXUBwR71PGYTp23w6jz_JmJtiPcq0MQTZygJYByi5KCStdM_mNHPWewpnAKJmLclqereIIZ0vgfvcNEtpqj82LW2kzfLFIpFvLJSVpK3L8sKE6Kq9c2I80ouevPJIq7lfJfNVHLgfK9fgV766Z2zy1Ky67C9bcm__h3JLGg8QycLVrCSz9SHnqqmb7T3Xh0n3NLvQjE3kM-iKcCDi8clbWOmRL7HOz2o-uC2ugxZml7bxBSUnbVUPUnUJjRQFX_fjZEPrpBXq4DznMFs1KD7p71tgFScz9>
 is disabled
  2.  Looking at the configuration it was java 8 based, so this has been 
disabled for some time
  3.  The setup has a lot of messing around to get the ojdbc7.jar to talk to 
oracle-db.dev.boundlessgeo.com<https://dficcfc.r.af.d.sendibt2.com/tr/cl/unOkFWXeyJmZIY8qRTRfEOWS7MGyOk4vkdzoJyUiwkcH9yRsDojjPS_a77MX248fX9iZJGLRLX_qf5Rhme4iUNeTWwPvePI2TTI46ny9WtEcprgjPVkdaKgW5yZV8sbuW4InphYTDd0l1E7xMMxaUvthsp69SyJB2DiWJ4IJiYU7u2Kgb2xPk_qYdiLx57HZKfzX2_SwJtpS5nD-ZmeJm7F5DF2VZKTj6qgTUd6HnHQnvGM9X1JvFm60YhkBoEIJfXLt7P-eHDHbvODtvxVsyQK_lQ>
 ... so quite a long time.

Do we have an active module maintainer (or experienced developer) for 
application schema?

Trying it out:

  *   I do not think the steps to copy ojdbc7.jar are required any more (since 
such a jar is now available via maven dependency)
  *   Changing to Java 11 and commenting out the oracle.properties test fixed 
and running the job 
shows<https://dficcfc.r.af.d.sendibt2.com/tr/cl/LfbWvC9tnFQ2Kw5ymOBeMXXeT8KmZn9PWK0NdXq52y5AYgfeX8mz06QT5ntFE-CKVt-ayEtPArTWSd2tixGSIEsOCw19g25HJNXHOpr7XEwluNq1_BUiIrzgr0flipi2EIBE8KbuuRpPJXKaGRA2Lc0Nq-UGcqd0p5tdbGWmOc5AA4-VnX7OxNt0O9ruU8yWc999jWsXXc3VeywtedDPUsdFmae6wzsRP-4lxATCFNfq7_TZn2NA9YJxiX_sa-razs2GEF-J8k9jk_XwnjqgwFLRgTL0v7uOSSiOdKI3ZO1kYTVxFfvfGfeD35FNb9SHic25FOFs2A6n63ff920JMw9Ed0YNZZmfuzP9lex9>
 ...

[INFO] Application Schema Support ......................... SUCCESS [  4.694 s]
[INFO] Application Schema Module .......................... SUCCESS [  9.101 s]
[INFO] Application Schema Integration Test ................ FAILURE [01:28 min]
[INFO] Sample DataAccess Integration Test ................. SUCCESS [  4.919 s]
[INFO] Application Schema Integration Online Test with Postgis Database SUCCESS 
[01:14 min]
[INFO] Application Schema Solr Integration Test ........... SUCCESS [  2.659 s]
[INFO] Application Schema Indexes Integration Test ........ SUCCESS [  1.319 s]
[INFO] Application Schema Geopackage Integration Test ..... FAILURE [ 11.730 s]

So what do you think? Does this test have value? For now I will enable it but 
not schedule it @daily (in case anyone else wants to look).

Thanks
--
Jody Garnett
_______________________________________________
Geoserver-devel mailing list
Geoserver-devel@lists.sourceforge.net<mailto:Geoserver-devel@lists.sourceforge.net>
https://lists.sourceforge.net/lists/listinfo/geoserver-devel
_______________________________________________
Geoserver-devel mailing list
Geoserver-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geoserver-devel

Reply via email to