upping to EUI 0.0.51 and fixing tests that broke (#19653)

* upping to EUI 0.0.51 and fixing tests that broke

* upping x-pack eui to 51 and refreshing snapshots

* adjusting onQueryChange API to match EUI

* adding lock files

* updating snapshots
This commit is contained in:
Bill McConaghy 2018-06-05 20:25:45 -04:00 committed by GitHub
parent 87cccfc101
commit 145163a51e
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
11 changed files with 28 additions and 15 deletions

View file

@ -77,7 +77,7 @@
"url": "https://github.com/elastic/kibana.git"
},
"dependencies": {
"@elastic/eui": "v0.0.49",
"@elastic/eui": "v0.0.51",
"@elastic/filesaver": "1.1.2",
"@elastic/numeral": "2.3.2",
"@elastic/ui-ace": "0.2.3",

View file

@ -73,6 +73,7 @@ exports[`apmUiEnabled 1`] = `
type="apmApp"
/>
}
layout="vertical"
textAlign="center"
title="APM"
/>
@ -103,6 +104,7 @@ exports[`apmUiEnabled 1`] = `
type="loggingApp"
/>
}
layout="vertical"
textAlign="center"
title="Logging"
/>
@ -133,6 +135,7 @@ exports[`apmUiEnabled 1`] = `
type="monitoringApp"
/>
}
layout="vertical"
textAlign="center"
title="Metrics"
/>
@ -163,6 +166,7 @@ exports[`apmUiEnabled 1`] = `
type="securityApp"
/>
}
layout="vertical"
textAlign="center"
title="Security Analytics"
/>
@ -318,6 +322,7 @@ exports[`render 1`] = `
type="loggingApp"
/>
}
layout="vertical"
textAlign="center"
title="Logging"
/>
@ -348,6 +353,7 @@ exports[`render 1`] = `
type="monitoringApp"
/>
}
layout="vertical"
textAlign="center"
title="Metrics"
/>
@ -378,6 +384,7 @@ exports[`render 1`] = `
type="securityApp"
/>
}
layout="vertical"
textAlign="center"
title="Security Analytics"
/>

View file

@ -27,7 +27,6 @@ exports[`Table editing should show an input field 1`] = `
<EuiFormControlLayout
compressed={false}
fullWidth={false}
iconSide="left"
isLoading={false}
>
<EuiValidatableControl>

View file

@ -29,6 +29,7 @@ exports[`Relationships should render dashboards normally 1`] = `
<EuiDescriptionList
align="left"
compressed={false}
textStyle="normal"
type="row"
>
<React.Fragment
@ -216,6 +217,7 @@ exports[`Relationships should render index patterns normally 1`] = `
<EuiDescriptionList
align="left"
compressed={false}
textStyle="normal"
type="row"
>
<React.Fragment
@ -396,6 +398,7 @@ exports[`Relationships should render searches normally 1`] = `
<EuiDescriptionList
align="left"
compressed={false}
textStyle="normal"
type="row"
>
<React.Fragment
@ -574,6 +577,7 @@ exports[`Relationships should render visualizations normally 1`] = `
<EuiDescriptionList
align="left"
compressed={false}
textStyle="normal"
type="row"
>
<React.Fragment

View file

@ -10,7 +10,9 @@ exports[`Table should render normally 1`] = `
"multiSelect": "or",
"name": "Type",
"options": Array [
2,
Object {
"value": 2,
},
],
"type": "field_value_selection",
},

View file

@ -46,7 +46,7 @@ describe('Table', () => {
selectionConfig: {
onSelectionChange: () => {},
},
filterOptions: [2],
filterOptions: [{ value: 2 }],
onDelete: () => {},
onExport: () => {},
getEditUrl: () => {},

View file

@ -194,7 +194,7 @@ export class ObjectsTable extends Component {
this.setState({ selectedSavedObjects });
};
onQueryChange = query => {
onQueryChange = ({ query }) => {
// TODO: investigate why this happens at EUI level
if (isSameQuery(query, this.state.activeQuery)) {
return;

View file

@ -41,6 +41,8 @@ describe('Search', () => {
});
it('should call parent function when query is changed', async () => {
//This test is brittle as it knows about implementation details
// (EuiFieldSearch uses onKeyup instead of onChange to handle input)
const onQueryChange = jest.fn();
const component = mount(
<Search
@ -49,8 +51,7 @@ describe('Search', () => {
onQueryChange={onQueryChange}
/>
);
component.find('input').simulate('keyUp');
component.find('EuiSearchFilters').prop('onChange')(query);
expect(onQueryChange).toHaveBeenCalledTimes(2);
component.find('input').simulate('keyup', { target: { value: 'new filter' } });
expect(onQueryChange).toHaveBeenCalledTimes(1);
});
});

View file

@ -76,7 +76,7 @@
"yargs": "4.7.1"
},
"dependencies": {
"@elastic/eui": "0.0.49",
"@elastic/eui": "0.0.51",
"@elastic/node-crypto": "0.1.2",
"@elastic/node-phantom-simple": "2.2.4",
"@elastic/numeral": "2.3.2",

View file

@ -10,9 +10,9 @@
esutils "^2.0.2"
js-tokens "^3.0.0"
"@elastic/eui@0.0.49":
version "0.0.49"
resolved "https://registry.yarnpkg.com/@elastic/eui/-/eui-0.0.49.tgz#583b3c1526df352b99a6cf13bd750d6d5adfb949"
"@elastic/eui@0.0.51":
version "0.0.51"
resolved "https://registry.yarnpkg.com/@elastic/eui/-/eui-0.0.51.tgz#5d809af270dd9994a609fd01eaa84e21a62fff98"
dependencies:
brace "^0.10.0"
classnames "^2.2.5"

View file

@ -77,9 +77,9 @@
version "0.0.0"
uid ""
"@elastic/eui@0.0.49", "@elastic/eui@v0.0.49":
version "0.0.49"
resolved "https://registry.yarnpkg.com/@elastic/eui/-/eui-0.0.49.tgz#583b3c1526df352b99a6cf13bd750d6d5adfb949"
"@elastic/eui@0.0.51", "@elastic/eui@v0.0.51":
version "0.0.51"
resolved "https://registry.yarnpkg.com/@elastic/eui/-/eui-0.0.51.tgz#5d809af270dd9994a609fd01eaa84e21a62fff98"
dependencies:
brace "^0.10.0"
classnames "^2.2.5"