Refs #35833 - Revert "Fixes #35684 - Drop Applied catalog lines"

This reverts commit 4f65dd5. It's
problematic because Foreman can't identify reports as Puppet reports if
there are no entries. Until Foreman has an API where it can be
identified as Puppet even if there are no logs we should continue
sending it.

files/report.rb

@@ -141,7 +141,7 @@ def logs_to_array logs
       next if log.level == :debug
 
       # skipping catalog summary run messages, we dont want them in Foreman's db
-      next if log.message =~ /^(Finished catalog run|Applied catalog) in \d+.\d+ seconds$/
+      next if log.message =~ /^Finished catalog run in \d+.\d+ seconds$/
 
       # Match Foreman's slightly odd API format...
       l = { 'log' => { 'sources' => {}, 'messages' => {} } }

spec/static_fixtures/report-format-6.json

@@ -210,6 +210,17 @@
         },
         "level": "notice"
       }
+    },
+    {
+      "log": {
+        "sources": {
+          "source": "//slave01.rackspace.theforeman.org/Puppet"
+        },
+        "messages": {
+          "message": "Applied catalog in 14.26 seconds"
+        },
+        "level": "notice"
+      }
     }
   ]
 }