Create unit and functional tests (as needed) for System Monitor logging system

+  /**
+   * Determines if there are any active incidents on the system.
+   *
+   * @return array
+   *   The array of result values.
+   */
+  public function hasActiveIncident() {
+    return !empty($this->getActiveIncidents());
+  }

This actually returns a boolean.

+  /**
+   * Retrieve all parent incident logs for a specific set of event IDs.
+   *
+   * @param array $event_ids
+   *   The IDs to lookup in the logs.
+   *
+   * @return array
+   *   An array of database rows as an object.
+   */
+  public function getParentIncidentLogs(array $event_ids) {

This method returns a $query->execute()->fetchAllAssoc('log_id'), which could also be null.

+  /**
+   * Retrieve all incident logs for a specific set of event IDs.
+   *
+   * @param array $event_ids
+   *   The IDs to lookup in the logs.
+   *
+   * @return array
+   *   An array of database rows as an object.
+   */
+  public function getIncidentLogs(array $event_ids) {

Same as above

Actually should there be two hasActiveIncident()?

disregard

Disregard last comment this looks good.