[GitHub] [camel-quarkus] jamesnetherton commented on pull request #3585: Add test coverage for consuming email with attachments

2022-03-03 Thread GitBox
jamesnetherton commented on pull request #3585: URL: https://github.com/apache/camel-quarkus/pull/3585#issuecomment-1058014612 Actually after doing 50 test runs there still seems to be some instability. Not sure if it's the way I have composed the tests or some issue in `camel-mail`. I'm g

[GitHub] [camel-quarkus] jamesnetherton commented on pull request #3585: Add test coverage for consuming email with attachments

2022-03-03 Thread GitBox
jamesnetherton commented on pull request #3585: URL: https://github.com/apache/camel-quarkus/pull/3585#issuecomment-1057997535 I have reworked the tests to switch from `mock-javamail` to using a containerized mock mail server. Hopefully this helps to fix things. It also opens the door to t

[GitHub] [camel-quarkus] jamesnetherton commented on pull request #3585: Add test coverage for consuming email with attachments

2022-03-02 Thread GitBox
jamesnetherton commented on pull request #3585: URL: https://github.com/apache/camel-quarkus/pull/3585#issuecomment-1056951957 Looks like there's an NPE to resolve (which is not occurring for me locally) -- This is an automated message from the Apache Git Service. To respond to the m

[GitHub] [camel-quarkus] jamesnetherton commented on pull request #3585: Add test coverage for consuming email with attachments

2022-03-02 Thread GitBox
jamesnetherton commented on pull request #3585: URL: https://github.com/apache/camel-quarkus/pull/3585#issuecomment-1056853646 > I wonder if there would be interest in testing with a secure flavor, I mean imaps, pop3s and on. Yes, in general the extension needs more test coverage to