From e1400a37ca56727e51e166e95dbcdbd205708d45 Mon Sep 17 00:00:00 2001
From: Mozafar Haider <mozafar@dhis2.org>
Date: Mon, 18 Mar 2024 16:07:10 +0000
Subject: [PATCH] refactor: apply code review comments (pairing session)

---
 .../button/src/split-button/split-button.js   |  5 +++--
 .../src/split-button/split-button.test.js     | 21 +------------------
 2 files changed, 4 insertions(+), 22 deletions(-)

diff --git a/components/button/src/split-button/split-button.js b/components/button/src/split-button/split-button.js
index de39f8e036..fadbc21a92 100644
--- a/components/button/src/split-button/split-button.js
+++ b/components/button/src/split-button/split-button.js
@@ -7,6 +7,7 @@ import PropTypes from 'prop-types'
 import React, { Component } from 'react'
 import css from 'styled-jsx/css'
 import { Button } from '../index.js'
+import i18n from '../locales/index.js'
 
 const rightButton = css.resolve`
     button {
@@ -113,8 +114,8 @@ class SplitButton extends Component {
                     tabIndex={tabIndex}
                     className={cx(className, rightButton.className)}
                     dataTest={`${dataTest}-toggle`}
-                    title="Toggle dropdown"
-                    aria-label="Toggle dropdown"
+                    title={i18n.t('Toggle dropdown')}
+                    aria-label={i18n.t('Toggle dropdown')}
                 >
                     {arrow}
                 </Button>
diff --git a/components/button/src/split-button/split-button.test.js b/components/button/src/split-button/split-button.test.js
index 58f1394cc4..d7c8ef2027 100644
--- a/components/button/src/split-button/split-button.test.js
+++ b/components/button/src/split-button/split-button.test.js
@@ -11,7 +11,7 @@ describe('SplitButton', () => {
         expect(getByText('Click me')).toBeInTheDocument()
     })
 
-    it('toggles dropdown when right button is clicked', () => {
+    it('toggles dropdown when left button is clicked', () => {
         const { getByTestId, queryByTestId } = render(<SplitButton />)
         const toggleButton = getByTestId('dhis2-uicore-splitbutton-toggle')
 
@@ -56,25 +56,6 @@ describe('SplitButton', () => {
         })
     })
 
-    it("does not close dropdown 'SpaceBar' key is pressed", async () => {
-        const { getByTestId } = render(
-            <SplitButton component={<div>Dropdown Content</div>} />
-        )
-
-        const toggleButton = getByTestId('dhis2-uicore-splitbutton-toggle')
-        fireEvent.click(toggleButton)
-        expect(getByTestId('dhis2-uicore-splitbutton-menu')).toBeInTheDocument()
-
-        fireEvent.keyDown(document, { key: ' ' })
-
-        // Use waitFor to wait for the DOM to update
-        await waitFor(() => {
-            expect(
-                getByTestId('dhis2-uicore-splitbutton-menu')
-            ).toBeInTheDocument()
-        })
-    })
-
     it('closes dropdown when escape key is pressed', async () => {
         const { getByTestId, queryByTestId } = render(
             <SplitButton component={<div>Dropdown Content</div>} />