Fixes #919 - isNotRhelHost regex too restrictive

- Changed regex to be more wide-ranging
- Fixed a bug in InsightsTotalRiskChartWrapper
where the wrong parameter was passed to isNotRhelHost

.eslintrc

@@ -33,6 +33,7 @@
           "remediate",
           "remediations",
           "repo",
+          "rhc",
           "theforeman",
           "tooltip",
           "unmount",

webpack/ForemanRhCloudHelpers.js

@@ -6,5 +6,8 @@
 export const foremanUrl = path => `${window.URL_PREFIX}${path}`;
 
 export const isNotRhelHost = ({ hostDetails }) =>
-  // eslint-disable-next-line camelcase
-  !new RegExp('red\\s?hat', 'i').test(hostDetails?.operatingsystem_name);
+  // This regex tries matches sane variations of "RedHat", "RHEL" and "RHCOS"
+  !new RegExp('red[\\s\\-]?hat|rh[\\s\\-]?el|rhc[\\s\\-]?os', 'i').test(
+    // eslint-disable-next-line camelcase
+    hostDetails?.operatingsystem_name
+  );

webpack/InsightsHostDetailsTab/InsightsTotalRiskChartWrapper.js

@@ -6,7 +6,7 @@ import { isNotRhelHost } from '../ForemanRhCloudHelpers';
 export const InsightsTotalRiskChartWrapper = props => {
   if (props.status === 'RESOLVED') {
     return (
-      !isNotRhelHost(props.hostDetails) && <InsightsTotalRiskChart {...props} /> // check for RHEL hosts
+      !isNotRhelHost(props) && <InsightsTotalRiskChart {...props} /> // check for RHEL hosts
     );
   }
   return null;