Arsnael commented on code in PR #2825:
URL: https://github.com/apache/james-project/pull/2825#discussion_r2394004896


##########
server/protocols/webadmin/webadmin-jmap/src/test/java/org/apache/james/webadmin/data/jmap/RunRulesOnMailboxRoutesTest.java:
##########
@@ -568,12 +563,12 @@ void runRulesOnMailboxShouldApplyFlagCriteria() throws 
Exception {
                   {
                     "comparator": "isSet",
                     "field": "flag",
-                    "value": "\\Seen"
+                    "value": "Seen"

Review Comment:
   Regarding this... In the test I need to put `"value": "\\\\Seen" ` for it to 
work, because while debugging I could observe that sparks does the escape when 
we get the request body, and then json deserializer is doing it too behind. 
   
   Can we just do without the `\` for the sake of simplicity?



-- 
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]

Reply via email to