From 996e127a0256fe751c423732bb339e7149673afb Mon Sep 17 00:00:00 2001
From: Kirill Morozov <grifx.design@gmail.com>
Date: Mon, 30 May 2022 13:24:59 +0300
Subject: [PATCH] Fixed Selected partitions label (#2042)

* Fixed Selected partitions label

* Console logs removed
---
 .../Topics/Topic/Details/Messages/Filters/Filters.tsx         | 2 +-
 .../components/Topics/Topic/Details/Messages/Filters/utils.ts | 4 ++--
 .../Topics/Topic/Details/Messages/__test__/utils.spec.ts      | 4 ++--
 3 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/kafka-ui-react-app/src/components/Topics/Topic/Details/Messages/Filters/Filters.tsx b/kafka-ui-react-app/src/components/Topics/Topic/Details/Messages/Filters/Filters.tsx
index 9eb7884b76..ad3aec6b81 100644
--- a/kafka-ui-react-app/src/components/Topics/Topic/Details/Messages/Filters/Filters.tsx
+++ b/kafka-ui-react-app/src/components/Topics/Topic/Details/Messages/Filters/Filters.tsx
@@ -181,7 +181,7 @@ const Filters: React.FC<FiltersProps> = ({
       partitions.map((partition: Partition) => {
         return {
           value: partition.partition,
-          label: String(partition.partition),
+          label: `Partition #${partition.partition.toString()}`,
         };
       })
     );
diff --git a/kafka-ui-react-app/src/components/Topics/Topic/Details/Messages/Filters/utils.ts b/kafka-ui-react-app/src/components/Topics/Topic/Details/Messages/Filters/utils.ts
index 8b29b5b820..16dbc7db69 100644
--- a/kafka-ui-react-app/src/components/Topics/Topic/Details/Messages/Filters/utils.ts
+++ b/kafka-ui-react-app/src/components/Topics/Topic/Details/Messages/Filters/utils.ts
@@ -53,7 +53,7 @@ export const getSelectedPartitionsFromSeekToParam = (
         if (selectedPartitionIds?.includes(partition)) {
           return {
             value: partition,
-            label: partition.toString(),
+            label: `Partition #${partition.toString()}`,
           };
         }
 
@@ -64,6 +64,6 @@ export const getSelectedPartitionsFromSeekToParam = (
 
   return partitions.map(({ partition }) => ({
     value: partition,
-    label: partition.toString(),
+    label: `Partition #${partition.toString()}`,
   }));
 };
diff --git a/kafka-ui-react-app/src/components/Topics/Topic/Details/Messages/__test__/utils.spec.ts b/kafka-ui-react-app/src/components/Topics/Topic/Details/Messages/__test__/utils.spec.ts
index c58bc09b0e..06a13d641b 100644
--- a/kafka-ui-react-app/src/components/Topics/Topic/Details/Messages/__test__/utils.spec.ts
+++ b/kafka-ui-react-app/src/components/Topics/Topic/Details/Messages/__test__/utils.spec.ts
@@ -101,7 +101,7 @@ describe('utils', () => {
     it('returns parsed partition from params when partition list includes selected partition', () => {
       searchParams.set('seekTo', '42::0');
       expect(getSelectedPartitionsFromSeekToParam(searchParams, part)).toEqual([
-        { label: '42', value: 42 },
+        { label: 'Partition #42', value: 42 },
       ]);
     });
     it('returns parsed partition from params when partition list NOT includes selected partition', () => {
@@ -113,7 +113,7 @@ describe('utils', () => {
     it('returns partitions when param "seekTo" is not defined', () => {
       searchParams.delete('seekTo');
       expect(getSelectedPartitionsFromSeekToParam(searchParams, part)).toEqual([
-        { label: '42', value: 42 },
+        { label: 'Partition #42', value: 42 },
       ]);
     });
   });