Review Comments Implemented

packages/terra-core-docs/src/terra-dev-site/doc/dropdown-button/About.1.doc.mdx

@@ -37,6 +37,11 @@ This component requires the following peer dependencies be installed in your app
 
 <!-- AUTO-GENERATED-CONTENT:END -->
 
+## Implementation Notes:
+The Form-Field component must be composed inside the [Base][1] component with a locale in order for it to load the correct translation strings.
+
+[1]: https://engineering.cerner.com/terra-core/components/terra-base/base/base
+
 ## Usage
 
 ```jsx
@@ -62,3 +67,26 @@ import DropdownButton, { Item } from 'terra-dropdown-button';
 
 ## Item Props
 <ItemPropsTable />
+
+## Testing
+
+Terra Alert uses `uuid` which changes the component's description id dynamically. To mock the return value with the Jest testing library, `jest.spyOn` can be used.
+
+If Enzyme `shallow` is being used for the tests then the mock may not be required depending on the depth of the returned wrapper. However, if `mount` is used then `uuid` should be mocked as shown below:
+
+```js
+import { v4 as uuidv4 } from 'uuid';
+
+let mockSpyUuid;
+
+// using a variable may result in failures. For best results, mock return value.
+beforeAll(() => {
+  mockSpyUuid = jest.spyOn(uuidv4, 'v4').mockReturnValue('00000000-0000-0000-0000-000000000000');
+});
+
+// restore the mock
+afterAll(() => {
+  mockSpyUuid.mockRestore();
+});
+
+```

packages/terra-dropdown-button/CHANGELOG.md

@@ -2,7 +2,7 @@
 
 ## Unreleased
 * Added
-  * Added aria-haspoup and aria-controls attributes for dropdown button.
+  * Added 'aria-haspoup' and 'aria-controls' attributes for dropdown button.
 
 ## 1.36.0 - (August 8, 2023)
 

packages/terra-dropdown-button/src/_Dropdown.jsx

@@ -44,6 +44,7 @@ const Dropdown = ({
   requestClose, isOpen, targetRef, children, width, refCallback, buttonRef, getSelectedOptionText, ...customProps
 }) => {
   const buttonFocused = useRef(false);
+  const { menuId, buttonId } = customProps;
   useEffect(() => {
     // added this change to bring focus back to button when dropdown list is closed.
     if (buttonFocused.current && buttonRef) {
@@ -76,8 +77,8 @@ const Dropdown = ({
             width={width}
             refCallback={refCallback}
             getSelectedOptionText={getSelectedOptionText}
-            menuId={customProps.menuId}
-            buttonId={customProps.buttonId}
+            menuId={menuId}
+            buttonId={buttonId}
           >
             {children}
           </DropdownList>

packages/terra-dropdown-button/tests/jest/DropdownButton.test.jsx

@@ -103,6 +103,8 @@ describe('Dropdown Button', () => {
         </DropdownButton>
       </ThemeContextProvider>,
     );
+    const dropDownButtonId = wrapper.find('#dropDown');
+    expect(dropDownButtonId.exists()).toBe(true);
     expect(wrapper).toMatchSnapshot();
   });