You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ponymail.apache.org by GitBox <gi...@apache.org> on 2020/08/25 16:46:26 UTC

[GitHub] [incubator-ponymail-unit-tests] sebbASF opened a new issue #1: Need script tests

sebbASF opened a new issue #1:
URL: https://github.com/apache/incubator-ponymail-unit-tests/issues/1


   The current test harness only tests the shared part of the archiver which parses emails.
   
   There need to be some tests for the functioning of the archiver when processing input messages, and for the importer when uploading mail from files.
   
   This will either need a test ES instance, or a way of emulating the behaviour of the ElasticSearch API. Or both.


----------------------------------------------------------------
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:
users@infra.apache.org

[GitHub] [incubator-ponymail-unit-tests] sebbASF commented on issue #1: Need script tests

Posted by GitBox <gi...@apache.org>.
sebbASF commented on issue #1:
URL: https://github.com/apache/incubator-ponymail-unit-tests/issues/1#issuecomment-680181502


   I think we also need to test against earlier versions to ensure compatibility


----------------------------------------------------------------
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:
users@infra.apache.org

[GitHub] [incubator-ponymail-unit-tests] Humbedooh commented on issue #1: Need script tests

Posted by GitBox <gi...@apache.org>.
Humbedooh commented on issue #1:
URL: https://github.com/apache/incubator-ponymail-unit-tests/issues/1#issuecomment-680144542


   We could extend upon this, and have the following steps run:
   
   - spin up ES
   - set up foal via non-interactive config
   - import from an mbox corpus we know
   - use the archiver on a select number of messages
   - spin up the UI backend and ensure that we are able to fetch the messages we need
   - Possibly test AAA in the UI?
   


----------------------------------------------------------------
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:
users@infra.apache.org

[GitHub] [incubator-ponymail-unit-tests] Humbedooh commented on issue #1: Need script tests

Posted by GitBox <gi...@apache.org>.
Humbedooh commented on issue #1:
URL: https://github.com/apache/incubator-ponymail-unit-tests/issues/1#issuecomment-680184583


   With regards to which elements in particular?
   The old backend and UI will obviously not be compatible with the new DB and structure.
   We might however be able to test migrations from old DBs to Foal - ensuring that once before and after you migrate, permalinks etc are stable.


----------------------------------------------------------------
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:
users@infra.apache.org

[GitHub] [incubator-ponymail-unit-tests] sebbASF commented on issue #1: Need script tests

Posted by GitBox <gi...@apache.org>.
sebbASF commented on issue #1:
URL: https://github.com/apache/incubator-ponymail-unit-tests/issues/1#issuecomment-680195331


   The archiver in particular needs to generate the same output for the same input and settings.
   For example, the current tests don't exercise the part of the archiver that adds the archived-at header.
   
   This is no longer added, but there may be other parts of the code that affect the output.
   
   Similar considerations apply to the importer.
   These are the two main items that need testing.
   
   Also need to test the backend to check the output is compatible with the published API.
   


----------------------------------------------------------------
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:
users@infra.apache.org