Merge pull request #5131 from mabashian/5010-close-button

Removes close button from footer of host details modal

Reviewed-by: https://github.com/apps/softwarefactory-project-zuul
This commit is contained in:
softwarefactory-project-zuul[bot] 2019-10-30 21:04:04 +00:00 committed by GitHub
commit 634504c7a1
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 2 additions and 13 deletions

View File

@ -1,10 +1,5 @@
import React, { useEffect, useState } from 'react';
import {
Button,
Modal as PFModal,
Tab,
Tabs as PFTabs,
} from '@patternfly/react-core';
import { Modal as PFModal, Tab, Tabs as PFTabs } from '@patternfly/react-core';
import CodeMirrorInput from '@components/CodeMirrorInput';
import ContentEmpty from '@components/ContentEmpty';
import PropTypes from 'prop-types';
@ -141,11 +136,6 @@ function HostEventModal({ onClose, hostEvent = {}, isOpen = false, i18n }) {
isOpen={isOpen}
onClose={onClose}
title={i18n._(t`Host Details`)}
actions={[
<Button key="cancel" variant="secondary" onClick={onClose}>
{i18n._(t`Close`)}
</Button>,
]}
>
<Tabs
aria-label={i18n._(t`Tabs`)}

View File

@ -250,8 +250,7 @@ describe('HostEventModal', () => {
const wrapper = mountWithContexts(
<HostEventModal hostEvent={hostEvent} onClose={onClose} isOpen />
);
const closeButton = wrapper.find('ModalBoxFooter Button');
closeButton.simulate('click');
wrapper.find('button[aria-label="Close"]').simulate('click');
expect(onClose).toBeCalled();
});