Problem/Motivation

MailhandlerNodeTest is using a process function from the inmail module. Now it is failing due to a modification in the function (3rd argument is added - $key) #2409465: Delete IMAP messages after fetching.

Proposed resolution

Fix the test accordingly to the newly modified process function in inmail.

Remaining tasks

User interface changes

API changes

Data model changes

CommentFileSizeAuthor
#3 mailhandlernodetest_fail-2816531-2.patch1.57 KBGinovski
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Ginovski created an issue. See original summary.

Ginovski’s picture

Ginovski CreditAttribution: Ginovski at MD Systems GmbH commented
Assigned: Unassigned » Ginovski
Status: Active » Needs review

1. Added keys to the process function calls in the test.
There is an unrelated fail to be fixed in the test (nodes are empty and the test is asserting for a node).

Ginovski’s picture

Ginovski CreditAttribution: Ginovski at MD Systems GmbH commented
FileSize
mailhandlernodetest_fail-2816531-2.patch1.57 KB

Patch for #2 comment.

  • mbovan committed ae9efe8 on 8.x-1.x authored by Ginovski 
    Issue #2816531 by Ginovski: MailhandlerNodeTest fail
mbovan’s picture

mbovan CreditAttribution: mbovan at MD Systems GmbH commented
Status: Needs review » Fixed

Committed. Thank you!

There is an unrelated fail to be fixed in the test (nodes are empty and the test is asserting for a node).

It should be fixed in #2816117: Add received header into sample messages.

Ginovski’s picture

Ginovski CreditAttribution: Ginovski at MD Systems GmbH commented

Created a followup for the other test #2816589: Fix test fails caused by MessageProcessor::process() changes in Inmail

Status: Fixed » Needs work

The last submitted patch, 3: mailhandlernodetest_fail-2816531-2.patch, failed testing.

The last submitted patch, 3: mailhandlernodetest_fail-2816531-2.patch, failed testing.

Ginovski’s picture

Ginovski CreditAttribution: Ginovski at MD Systems GmbH commented
Status: Needs work » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.