Lexicographically sort location tags (elastic#54832)

Sort location tags lexicographically, fixes skipped test by providing a stable, non-time-based sort order
jkelastic · Jan 17, 2020 · ca5e2bd · ca5e2bd
1 parent 7a33ca9
commit ca5e2bd
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 7 deletions.
diff --git a/...onents/functional/location_map/__tests__/__snapshots__/location_status_tags.test.tsx.snap b/...onents/functional/location_map/__tests__/__snapshots__/location_status_tags.test.tsx.snap
diff --git a/.../uptime/public/components/functional/location_map/__tests__/location_status_tags.test.tsx b/.../uptime/public/components/functional/location_map/__tests__/location_status_tags.test.tsx
@@ -11,7 +11,7 @@ import { MonitorLocation } from '../../../../../common/runtime_types/monitor';
 import { LocationStatusTags } from '../';
 
 // Failing: https://github.com/elastic/kibana/issues/54818
-describe.skip('LocationStatusTags component', () => {
+describe('LocationStatusTags component', () => {
   let monitorLocations: MonitorLocation[];
 
   it('renders properly against props', () => {

diff --git a/.../legacy/plugins/uptime/public/components/functional/location_map/location_status_tags.tsx b/.../legacy/plugins/uptime/public/components/functional/location_map/location_status_tags.tsx
@@ -61,9 +61,9 @@ export const LocationStatusTags = ({ locations }: Props) => {
     }
   });
 
-  // Sort by recent timestamp
+  // Sort lexicographically by label
   upLocations.sort((a, b) => {
-    return a.timestamp < b.timestamp ? 1 : b.timestamp < a.timestamp ? -1 : 0;
+    return a.label > b.label ? 1 : b.label > a.label ? -1 : 0;
   });
 
   moment.updateLocale('en', {