Project

General

Profile

Bug #20241

Fix test for mail notifications on 1.15-stable

Added by Daniel Lobato Garcia about 2 years ago. Updated about 1 year ago.

Status:
Closed
Priority:
Normal
Category:
Tests
Target version:
Difficulty:
Triaged:
Bugzilla link:
Team Backlog:
Fixed in Releases:
Found in Releases:

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 (diff)
Added by Daniel Lobato Garcia about 2 years 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 2 years ago

  • Legacy Backlogs Release (now unused) set to 266

#2 Updated by The Foreman Bot about 2 years 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 2 years 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