fix(test): meet coverage thresholds after refactoring

carbon-design-system · May 8, 2020 · 8c07320 · 8c07320
1 parent 2bc7520
commit 8c07320
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 3 deletions.
diff --git a/src/components/Header/Header.test.jsx b/src/components/Header/Header.test.jsx
@@ -51,6 +51,7 @@ describe('Header testcases', () => {
       {
         label: 'user',
         btnContent: <Avatar fill="white" description="Icon" />,
+        menuLinkName: 'a menu link name',
         childContent: [
           {
             metaData: {

diff --git a/src/components/Header/HeaderAction/HeaderAction.jsx b/src/components/Header/HeaderAction/HeaderAction.jsx
@@ -123,8 +123,5 @@ const HeaderAction = ({ item, index }) => {
 };
 
 HeaderAction.propTypes = HeaderActionPropTypes;
-HeaderAction.defaultProps = {
-  item: { onClick: () => {} },
-};
 
 export default HeaderAction;
diff --git a/src/components/TimePickerSpinner/TimePickerSpinner.test.jsx b/src/components/TimePickerSpinner/TimePickerSpinner.test.jsx
@@ -69,6 +69,19 @@ describe('TimePickerSpinner', () => {
     expect(screen.getByRole('textbox').value).toEqual('00:00');
   });
 
+  test('value is not modified on unrelated keystroke', () => {
+    render(<TimePickerSpinner {...timePickerProps} spinner />);
+
+    screen.getByRole('textbox').focus();
+
+    fireEvent.keyUp(document.activeElement || document.body, {
+      key: 'Escape',
+      code: 'Escape',
+      keyCode: keyCodes.ESCAPE,
+    });
+    expect(screen.getByRole('textbox').value).toEqual('');
+  });
+
   test('work with strings', () => {
     const wrapper = mount(<TimePickerSpinner {...timePickerProps} value="xyz" spinner />);
 
@@ -104,6 +117,7 @@ describe('TimePickerSpinner', () => {
     upButton.simulate('mouseover');
     upButton.simulate('mouseout');
     upButton.simulate('blur');
+    upButton.simulate('mouseout');
     expect(
       wrapper
         .find('.iot--time-picker__wrapper')