felixauringer commented on PR #2587:
URL: https://github.com/apache/james-project/pull/2587#issuecomment-2630382859
Hey,
I have it already rebased on the current master and encountered some
problems during build with the command `mvn clean install
-Dmaven.javadoc.skip=true -DskipTests --pl :james-server-postgres-app --am`.
After I fixed the following things, the build worked. Maybe it helps you :)
Best,
Felix
### Import of `MessageParser` instead of `MessageParserImpl`
The following files import and use
`org.apache.james.mailbox.store.mail.model.impl.MessageParser` (which is only
an interface):
-
`mailbox/postgres/src/test/java/org/apache/james/mailbox/postgres/DeleteMessageListenerTest.java`
-
`mailbox/postgres/src/test/java/org/apache/james/mailbox/postgres/DeleteMessageListenerWithRLSTest.java`
-
`mailbox/postgres/src/test/java/org/apache/james/mailbox/postgres/PostgresMailboxManagerAttachmentTest.java`
-
`mailbox/postgres/src/test/java/org/apache/james/mailbox/postgres/PostgresMailboxManagerProvider.java`
-
`mailbox/postgres/src/test/java/org/apache/james/mailbox/postgres/PostgresTestSystemFixture.java`
Importing `org.apache.james.mailbox.store.mail.model.impl.MessageParserImpl`
instead fixes that error.
### Missing dependency in `pom.xml`
Some test case uses the in-memory event store. Hence, the following is
needed in `server/apps/postgres-app/pom.xml`:
```xml
<dependency>
<groupId>${james.groupId}</groupId>
<artifactId>event-sourcing-event-store-memory</artifactId>
<scope>test</scope>
</dependency>
```
(Insert before `james-server-cli` for the correct ordering.)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]