Le Thu, Aug 22, 2024 at 10:04:45AM +0200, Landry Breuil a écrit :
> Le Thu, Aug 22, 2024 at 09:40:23AM +0200, Landry Breuil a écrit :
> > Le Wed, Aug 21, 2024 at 06:11:32PM +0200, Kirill A. Korinsky a écrit :
> > > On Wed, 21 Aug 2024 17:16:11 +0200,
> > > Landry Breuil <lan...@openbsd.org> wrote:
> > > > 
> > > > Le Wed, Aug 21, 2024 at 05:00:21PM +0200, Landry Breuil a écrit :
> > > > > Le Wed, Aug 21, 2024 at 03:49:45PM +0200, Kirill A. Korinsky a écrit :
> > > > > > ports@,
> > > > > > 
> > > > > > Here a ping from another diff from my
> > > > > 
> > > > > can you explain the dance about dropping/copying tests in pre-test ?
> > > > > why was that needed ? isn't doing MODPY_PYTEST_ARGS = --forked enough 
> > > > > ?
> > > > 
> > > > now that i've tested, this fails with a rather strange error (strange as
> > > > in "i dont understand why moving tests/ around helps") on all test 
> > > > files:
> > > >
> > > 
> > > and I confirm that this is the problem I am overlooking by moving tests.
> 
> it feels we're getting close.. but not yet.

this version with a symlink for the rust lib works, and it's probably what im
going to commit. fixing the tests requiring a throwaway synapse test instance
is left for future work.

Index: Makefile
===================================================================
RCS file: /cvs/ports/net/synapse/Makefile,v
diff -u -r1.82 Makefile
--- Makefile    31 Jul 2024 16:01:52 -0000      1.82
+++ Makefile    25 Aug 2024 07:04:41 -0000
@@ -72,10 +72,20 @@
 TEST_DEPENDS = ${FULLPKGNAME}:${BUILD_PKGPATH} \
                devel/py-mock${MODPY_FLAVOR} \
                devel/py-parameterized${MODPY_FLAVOR} \
+               devel/py-test-forked${MODPY_FLAVOR} \
                www/py-jwt${MODPY_FLAVOR}
 
 do-configure:
        @${MODCARGO_configure}
+
+MODPY_PYTEST_ARGS =    --forked
+
+# some tests fail, but they need a previously running synapse process, not 
just a database
+# E synapse.storage.prepare_database.UpgradeDatabaseException: Uninitialised 
database:
+# run the main synapse process to prepare the database schema before starting 
worker processes.
+# make sure that the rust library is found
+pre-test:
+       ln -sf 
${WRKSRC}/build/lib.openbsd-${OSREV}-${ARCH}-cpython-${MODPY_MAJORMINOR}/synapse/synapse_rust.abi3.so
 ${WRKSRC}/synapse/
 
 # to generate rust modules.inc:
 #  make modcargo-gen-crates and modcargo-gen-crates-licenses

Landry

Reply via email to