Fixes #37038 - Use correct total and owned links

for host statuses in error column in Monitor > Host statuses.
theforeman · Jan 15, 2024 · 20798d2 · 20798d2
1 parent 670ee16
commit 20798d2
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/webpack/assets/javascripts/react_app/components/HostStatuses/HostStatusesSelectors.js b/webpack/assets/javascripts/react_app/components/HostStatuses/HostStatusesSelectors.js
@@ -35,10 +35,10 @@ export const selectHostStatusWarnOwnedPath = (state, statusName) =>
   selectHostStatus(state, statusName)?.warn_owned_path;
 
 export const selectHostStatusErrorTotalPath = (state, statusName) =>
-  selectHostStatus(state, statusName)?.error_owned_path;
+  selectHostStatus(state, statusName)?.error_total_path;
 
 export const selectHostStatusErrorOwnedPath = (state, statusName) =>
-  selectHostStatus(state, statusName)?.error_total_path;
+  selectHostStatus(state, statusName)?.error_owned_path;
 /* eslint-enable camelcase */
 
 export const selectHostStatusTotalPaths = (state, statusName) => ({

diff --git a/...cripts/react_app/components/HostStatuses/__snapshots__/HostStatusesSelectors.test.js.snap b/...cripts/react_app/components/HostStatuses/__snapshots__/HostStatusesSelectors.test.js.snap
@@ -62,15 +62,15 @@ Array [
 
 exports[`HostStatusesSelectors selectors selects selectHostStatusOwnedPaths 1`] = `
 Object {
-  "errorOwnedPath": "/hosts?search=status+%3D+error",
+  "errorOwnedPath": "/hosts?search=owner+%3D+current_user+AND+%28status+%3D+error",
   "okOwnedPath": "/hosts?search=owner+%3D+current_user+AND+%28status+%3D+ok",
   "warnOwnedPath": "/hosts?search=owner+%3D+current_user+AND+%28status+%3D+warn",
 }
 `;
 
 exports[`HostStatusesSelectors selectors selects selectHostStatusTotalPaths 1`] = `
 Object {
-  "errorTotalPath": "/hosts?search=owner+%3D+current_user+AND+%28status+%3D+error",
+  "errorTotalPath": "/hosts?search=status+%3D+error",
   "okTotalPath": "/hosts?search=status+%3D+ok",
   "warnTotalPath": "/hosts?search=status+%3D+warn",
 }