Skip to content

Commit

Permalink
fix tests after rebase
Browse files Browse the repository at this point in the history
  • Loading branch information
Yasser Elsayed committed Nov 22, 2018
1 parent ef18790 commit 3a0e582
Show file tree
Hide file tree
Showing 4 changed files with 8 additions and 19 deletions.
1 change: 1 addition & 0 deletions frontend/src/pages/RecurringRunsManager.test.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -203,6 +203,7 @@ describe('RecurringRunsManager', () => {

expect(listJobsSpy).toHaveBeenCalledTimes(1);
enableBtn.simulate('click');
await listJobsSpy;
await TestUtils.flushPromises();
expect(listJobsSpy).toHaveBeenCalledTimes(2);
});
Expand Down
2 changes: 1 addition & 1 deletion frontend/src/pages/RecurringRunsManager.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -82,7 +82,7 @@ class RecurringRunsManager extends React.Component<RecurringRunListProps, Recurr

return (<React.Fragment>
<Toolbar actions={toolbarActions} breadcrumbs={[]} pageTitle='Recurring runs' />
<CustomTable columns={columns} rows={rows} selectedIds={selectedIds} disableSelection={true}
<CustomTable columns={columns} rows={rows} ref={this._tableRef} selectedIds={selectedIds} disableSelection={true}
updateSelection={ids => this.setState({ selectedIds: ids })} initialSortColumn={JobSortKeys.CREATED_AT}
reload={this._loadRuns.bind(this)} emptyMessage={'No recurring runs found in this experiment.'} />
</React.Fragment>);
Expand Down
10 changes: 2 additions & 8 deletions frontend/src/pages/__snapshots__/PipelineSelector.test.tsx.snap
Original file line number Diff line number Diff line change
Expand Up @@ -4,14 +4,8 @@ exports[`PipelineSelector calls API to load pipelines 1`] = `
<Fragment>
<Toolbar
actions={Array []}
breadcrumbs={
Array [
Object {
"displayName": "Choose a pipeline",
"href": "",
},
]
}
breadcrumbs={Array []}
pageTitle="Choose a pipeline"
/>
<CustomTable
columns={
Expand Down
Original file line number Diff line number Diff line change
@@ -1,17 +1,11 @@
// Jest Snapshot v1, https://goo.gl/fbAQLP

exports[`RecurringRunsManager calls API to load recurring runs 1`] = `
<div>
<Fragment>
<Toolbar
actions={Array []}
breadcrumbs={
Array [
Object {
"displayName": "Recurring runs",
"href": "",
},
]
}
breadcrumbs={Array []}
pageTitle="Recurring runs"
/>
<CustomTable
columns={
Expand Down Expand Up @@ -72,7 +66,7 @@ exports[`RecurringRunsManager calls API to load recurring runs 1`] = `
selectedIds={Array []}
updateSelection={[Function]}
/>
</div>
</Fragment>
`;

exports[`RecurringRunsManager reloads the list of runs after enable/disabling 1`] = `
Expand Down

0 comments on commit 3a0e582

Please sign in to comment.