Bug #20241

Fix test for mail notifications on 1.15-stable

Added by Daniel Lobato Garcia about 1 year ago. Updated 9 days ago.

Status:Closed
Priority:Normal
Assignee:Daniel Lobato Garcia
Category:Tests
Target version:1.15.2
Difficulty: Team Backlog:
Triaged: Fixed in Releases:
Bugzilla link: Found in Releases:
Pull request:https://github.com/theforeman/foreman/pull/4652

Description

One of the tests in test/unit/tasks/reports_test is testing for delivery.subject =~ /Configuration Management Summary/
This made sense in develop because since #16159 mail alerts have that subject. However on the 1.15-stable branch, it still has the old subject, "Puppet Summary".

Associated revisions

Revision 18346101
Added by Daniel Lobato Garcia about 1 year ago

Fixes #20241 - Fix test for mail notifications

One of the tests in test/unit/tasks/reports_test is testing for
delivery.subject =~ /Configuration Management Summary/
This made sense in develop because since #16159 mail alerts have that
subject. However on the 1.15-stable branch, it still has the old
subject, "Puppet Summary".

History

#1 Updated by Daniel Lobato Garcia about 1 year ago

  • Legacy Backlogs Release (now unused) set to 266

#2 Updated by The Foreman Bot about 1 year ago

  • Status changed from New to Ready For Testing
  • Assignee set to Daniel Lobato Garcia
  • Pull request https://github.com/theforeman/foreman/pull/4652 added

#3 Updated by Michael Moll about 1 year ago

  • Status changed from Ready For Testing to Closed

PR was merged, but issues autoclose only when pushed to develop.

Also available in: Atom PDF