LanKhuat commented on a change in pull request #377:
URL: https://github.com/apache/james-project/pull/377#discussion_r610476045
##########
File path:
server/data/data-jmap/src/test/java/org/apache/james/jmap/api/change/MailboxChangeRepositoryContract.java
##########
@@ -376,7 +376,7 @@ default void getChangesShouldNotReturnMoreThanMaxChanges() {
}
@Test
- default void
getChangesShouldReturnEmptyWhenNumberOfChangesExceedMaxChanges() {
+ default void
getChangesShouldReturnThrowWhenNumberOfChangesExceedMaxChanges() {
Review comment:
getChangesShouldThrow...
##########
File path:
server/protocols/jmap-rfc-8621-integration-tests/jmap-rfc-8621-integration-tests-common/src/main/scala/org/apache/james/jmap/rfc8621/contract/EmailChangesMethodContract.scala
##########
@@ -197,6 +197,110 @@ trait EmailChangesMethodContract {
}
}
+ @Test
+ def shouldDailWithCannotCalculateChangesWhenSingleChangeIsTooLarge(server:
GuiceJamesServer): Unit = {
+ val mailboxProbe: MailboxProbeImpl =
server.getProbe(classOf[MailboxProbeImpl])
+ val path: MailboxPath = MailboxPath.forUser(BOB, "mailbox1")
+
+ mailboxProbe.createMailbox(path)
+
+ val message: Message = Message.Builder
+ .of
+ .setSubject("test")
+ .setBody("testmail", StandardCharsets.UTF_8)
+ .build
+ val messageId1: MessageId = mailboxProbe.appendMessage(BOB.asString(),
path, AppendCommand.from(message)).getMessageId
+ val messageId2: MessageId = mailboxProbe.appendMessage(BOB.asString(),
path, AppendCommand.from(message)).getMessageId
+ val messageId3: MessageId = mailboxProbe.appendMessage(BOB.asString(),
path, AppendCommand.from(message)).getMessageId
+ val messageId4: MessageId = mailboxProbe.appendMessage(BOB.asString(),
path, AppendCommand.from(message)).getMessageId
+ val messageId5: MessageId = mailboxProbe.appendMessage(BOB.asString(),
path, AppendCommand.from(message)).getMessageId
+ val messageId6: MessageId = mailboxProbe.appendMessage(BOB.asString(),
path, AppendCommand.from(message)).getMessageId
+
+ val oldState: State = waitForNextState(server,
AccountId.fromUsername(BOB), State.INITIAL)
Review comment:
Did you test this on the distributed environment? You append 6 messages
so it should have 6 more states after the INITIAL one.
##########
File path:
server/protocols/jmap-rfc-8621-integration-tests/jmap-rfc-8621-integration-tests-common/src/main/scala/org/apache/james/jmap/rfc8621/contract/EmailChangesMethodContract.scala
##########
@@ -168,9 +168,9 @@ trait EmailChangesMethodContract {
val response = `given`
.header(ACCEPT.toString, ACCEPT_RFC8621_VERSION_HEADER)
.body(request)
- .when
+ .when
.post
- .`then`
+ .`then`
Review comment:
indent
--
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.
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
