correct other describe calls

packages/kbn-pm/src/utils/projects.test.ts

@@ -224,7 +224,7 @@ describe('#topologicallyBatchProjects', () => {
     expect(expectedBatches).toMatchSnapshot();
   });
 
-  describe('batchByWorkspace = true', async () => {
+  describe('batchByWorkspace = true', () => {
     test('batches projects topologically based on their project dependencies and workspaces', async () => {
       const batches = topologicallyBatchProjects(projects, graph, { batchByWorkspace: true });
 

src/legacy/core_plugins/data/public/filter/filter_manager/filter_manager.test.ts

@@ -219,7 +219,7 @@ describe('filter_manager', () => {
     });
   });
 
-  describe('add filters', async () => {
+  describe('add filters', () => {
     test('app state should accept a single filter', async function() {
       updateSubscription = filterManager.getUpdates$().subscribe(updateListener);
       const f1 = getFilter(FilterStateStore.APP_STATE, false, false, 'age', 34);
@@ -535,7 +535,7 @@ describe('filter_manager', () => {
     });
   });
 
-  describe('remove filters', async () => {
+  describe('remove filters', () => {
     test('remove on empty should do nothing and not fire events', async () => {
       updateSubscription = filterManager.getUpdates$().subscribe(updateListener);
       await filterManager.removeAll();

src/legacy/ui/public/courier/fetch/request/serialize_fetch_params/serialize_fetch_params.test.js

@@ -81,7 +81,7 @@ describe('headers', () => {
     return JSON.parse(requestParts[0]);
   };
 
-  describe('search request preference', async () => {
+  describe('search request preference', () => {
     test('should be set to sessionId when courier:setRequestPreference is "sessionId"', async () => {
       const config = {
         get: () => {

src/legacy/ui/public/kfetch/kfetch.test.ts

@@ -133,7 +133,7 @@ describe('kfetch', () => {
     }
   });
 
-  describe('when throwing response error (KFetchError)', async () => {
+  describe('when throwing response error (KFetchError)', () => {
     let error: KFetchError;
     beforeEach(async () => {
       fetchMock.get('*', { status: 404, body: { foo: 'bar' } });
@@ -348,7 +348,7 @@ describe('kfetch', () => {
     });
   });
 
-  describe('when interceptors return synchronously', async () => {
+  describe('when interceptors return synchronously', () => {
     let resp: any;
     beforeEach(async () => {
       fetchMock.get('*', { foo: 'bar' });
@@ -381,7 +381,7 @@ describe('kfetch', () => {
     });
   });
 
-  describe('when interceptors return promise', async () => {
+  describe('when interceptors return promise', () => {
     let resp: any;
     beforeEach(async () => {
       fetchMock.get('*', { foo: 'bar' });

test/api_integration/apis/management/saved_objects/relationships.js

@@ -52,7 +52,7 @@ export default function ({ getService }) {
     const getSavedObjectTypesQuery = types => coerceToArray(types).map(type => `savedObjectTypes=${type}`).join('&');
     const defaultQuery = getSavedObjectTypesQuery(['visualization', 'index-pattern', 'search', 'dashboard']);
 
-    describe('searches', async () => {
+    describe('searches', () => {
       it('should validate search response schema', async () => {
         await supertest
           .get(`${baseApiUrl}/search/960372e0-3224-11e8-a572-ffca06da1357?${defaultQuery}`)
@@ -145,7 +145,7 @@ export default function ({ getService }) {
       });
     });
 
-    describe('dashboards', async () => {
+    describe('dashboards', () => {
       it('should validate dashboard response schema', async () => {
         await supertest
           .get(`${baseApiUrl}/dashboard/b70c7ae0-3224-11e8-a572-ffca06da1357?${defaultQuery}`)
@@ -240,7 +240,7 @@ export default function ({ getService }) {
       });
     });
 
-    describe('visualizations', async () => {
+    describe('visualizations', () => {
       it('should validate visualization response schema', async () => {
         await supertest
           .get(`${baseApiUrl}/visualization/a42c0580-3224-11e8-a572-ffca06da1357?${defaultQuery}`)
@@ -320,7 +320,7 @@ export default function ({ getService }) {
       });
     });
 
-    describe('index patterns', async () => {
+    describe('index patterns', () => {
       it('should validate visualization response schema', async () => {
         await supertest
           .get(`${baseApiUrl}/index-pattern/8963ca30-3224-11e8-a572-ffca06da1357?${defaultQuery}`)

test/functional/apps/dashboard/create_and_add_embeddables.js

@@ -29,7 +29,7 @@ export default function ({ getService, getPageObjects }) {
   const browser = getService('browser');
   const dashboardAddPanel = getService('dashboardAddPanel');
 
-  describe('create and add embeddables', async () => {
+  describe('create and add embeddables', () => {
     before(async () => {
       await PageObjects.dashboard.loadSavedDashboard('few panels');
     });

test/functional/apps/dashboard/dashboard_filter_bar.js

@@ -27,12 +27,12 @@ export default function ({ getService, getPageObjects }) {
   const pieChart = getService('pieChart');
   const PageObjects = getPageObjects(['dashboard', 'header', 'visualize']);
 
-  describe('dashboard filter bar', async () => {
+  describe('dashboard filter bar', () => {
     before(async () => {
       await PageObjects.dashboard.gotoDashboardLandingPage();
     });
 
-    describe('Add a filter bar', async function () {
+    describe('Add a filter bar', function () {
       before(async () => {
         await PageObjects.dashboard.gotoDashboardLandingPage();
       });
@@ -50,7 +50,7 @@ export default function ({ getService, getPageObjects }) {
       });
     });
 
-    describe('filter editor field list', async function () {
+    describe('filter editor field list', function () {
       this.tags(['skipFirefox']);
 
       before(async () => {
@@ -80,7 +80,7 @@ export default function ({ getService, getPageObjects }) {
       });
     });
 
-    describe('filter pills', async function () {
+    describe('filter pills', function () {
       before(async () => {
         await filterBar.ensureFieldEditorModalIsClosed();
         await PageObjects.dashboard.gotoDashboardLandingPage();

test/functional/apps/dashboard/dashboard_filtering.js

@@ -34,13 +34,13 @@ export default function ({ getService, getPageObjects }) {
   const dashboardPanelActions = getService('dashboardPanelActions');
   const PageObjects = getPageObjects(['dashboard', 'header', 'visualize']);
 
-  describe('dashboard filtering', async function () {
+  describe('dashboard filtering', function () {
     this.tags('smoke');
     before(async () => {
       await PageObjects.dashboard.gotoDashboardLandingPage();
     });
 
-    describe('adding a filter that excludes all data', async () => {
+    describe('adding a filter that excludes all data', () => {
       before(async () => {
         await PageObjects.dashboard.clickNewDashboard();
         await PageObjects.dashboard.setTimepickerInDataRange();
@@ -107,7 +107,7 @@ export default function ({ getService, getPageObjects }) {
       });
     });
 
-    describe('using a pinned filter that excludes all data', async () => {
+    describe('using a pinned filter that excludes all data', () => {
       before(async () => {
         await filterBar.toggleFilterPinned('bytes');
         await PageObjects.header.waitUntilLoadingHasFinished();
@@ -169,7 +169,7 @@ export default function ({ getService, getPageObjects }) {
       });
     });
 
-    describe('disabling a filter unfilters the data on', async () => {
+    describe('disabling a filter unfilters the data on', () => {
       before(async () => {
         await filterBar.toggleFilterEnabled('bytes');
         await PageObjects.header.waitUntilLoadingHasFinished();
@@ -227,7 +227,7 @@ export default function ({ getService, getPageObjects }) {
       });
     });
 
-    describe('nested filtering', async () => {
+    describe('nested filtering', () => {
       before(async () => {
         await PageObjects.dashboard.gotoDashboardLandingPage();
       });

test/functional/apps/dashboard/dashboard_listing.js

@@ -30,7 +30,7 @@ export default function ({ getService, getPageObjects }) {
       await PageObjects.dashboard.initTests();
     });
 
-    describe('create prompt', async () => {
+    describe('create prompt', () => {
       it('appears when there are no dashboards', async function () {
         const promptExists = await PageObjects.dashboard.getCreateDashboardPromptExists();
         expect(promptExists).to.be(true);
@@ -59,7 +59,7 @@ export default function ({ getService, getPageObjects }) {
       });
     });
 
-    describe('delete', async function () {
+    describe('delete', function () {
       it('default confirm action is cancel', async function () {
         await PageObjects.dashboard.searchForDashboardWithName(dashboardName);
         await PageObjects.dashboard.checkDashboardListingSelectAllCheckbox();

test/functional/apps/dashboard/dashboard_options.js

@@ -23,7 +23,7 @@ export default function ({ getService, getPageObjects }) {
   const retry = getService('retry');
   const PageObjects = getPageObjects(['dashboard']);
 
-  describe('dashboard data-shared attributes', async () => {
+  describe('dashboard data-shared attributes', () => {
     let originalTitles = [];
 
     before(async () => {

test/functional/apps/dashboard/dashboard_query_bar.js

@@ -25,7 +25,7 @@ export default function ({ getService, getPageObjects }) {
   const queryBar = getService('queryBar');
   const PageObjects = getPageObjects(['dashboard', 'discover']);
 
-  describe('dashboard query bar', async () => {
+  describe('dashboard query bar', () => {
     before(async () => {
       await PageObjects.dashboard.loadSavedDashboard('dashboard with filter');
     });

test/functional/apps/dashboard/dashboard_time.js

@@ -56,7 +56,7 @@ export default function ({ getPageObjects, getService }) {
       });
     });
 
-    describe('dashboard with stored timed', async function () {
+    describe('dashboard with stored timed', function () {
       it('is saved with time', async function () {
         await PageObjects.dashboard.switchToEditMode();
         await PageObjects.timePicker.setAbsoluteRange(fromTime, toTime);

test/functional/apps/dashboard/embed_mode.js

@@ -24,7 +24,7 @@ export default function ({ getService, getPageObjects }) {
   const PageObjects = getPageObjects(['dashboard', 'common']);
   const browser = getService('browser');
 
-  describe('embed mode', async () => {
+  describe('embed mode', () => {
     before(async () => {
       await PageObjects.dashboard.loadSavedDashboard('few panels');
     });

test/functional/apps/dashboard/empty_dashboard.js

@@ -24,7 +24,7 @@ export default function ({ getService, getPageObjects }) {
   const dashboardAddPanel = getService('dashboardAddPanel');
   const PageObjects = getPageObjects(['dashboard']);
 
-  describe('empty dashboard', async () => {
+  describe('empty dashboard', () => {
     before(async () => {
       await PageObjects.dashboard.clickNewDashboard();
     });

test/functional/apps/dashboard/full_screen_mode.js

@@ -25,7 +25,7 @@ export default function ({ getService, getPageObjects }) {
   const dashboardPanelActions = getService('dashboardPanelActions');
   const PageObjects = getPageObjects(['dashboard', 'common']);
 
-  describe('full screen mode', async () => {
+  describe('full screen mode', () => {
     before(async () => {
       await PageObjects.dashboard.loadSavedDashboard('few panels');
     });

test/functional/apps/dashboard/view_edit.js

@@ -53,7 +53,7 @@ export default function ({ getService, getPageObjects }) {
       });
     });
 
-    describe('shows lose changes warning', async function () {
+    describe('shows lose changes warning', function () {
       describe('and loses changes on confirmation', function () {
         beforeEach(async function () {
           await PageObjects.dashboard.gotoDashboardEditMode(dashboardName);
@@ -195,7 +195,7 @@ export default function ({ getService, getPageObjects }) {
       });
     });
 
-    describe('Does not show lose changes warning', async function () {
+    describe('Does not show lose changes warning', function () {
       it('when time changed is not stored with dashboard', async function () {
         await PageObjects.dashboard.gotoDashboardEditMode(dashboardName);
         await PageObjects.dashboard.saveDashboard(dashboardName, { storeTimeWithDashboard: false });

test/functional/apps/discover/_discover.js

@@ -381,7 +381,7 @@ export default function ({ getService, getPageObjects }) {
       }
     });
 
-    describe('query #2, which has an empty time range', async () => {
+    describe('query #2, which has an empty time range', () => {
       const fromTime = '1999-06-11 09:22:11.000';
       const toTime = '1999-06-12 11:21:04.000';
 
@@ -402,7 +402,7 @@ export default function ({ getService, getPageObjects }) {
       });
     });
 
-    describe('filter editor', async function () {
+    describe('filter editor', function () {
       it('should add a phrases filter', async function () {
         await filterBar.addFilter('extension.raw', 'is one of', 'jpg');
         expect(await filterBar.hasFilter('extension.raw', 'jpg')).to.be(true);

test/functional/apps/visualize/_markdown_vis.js

@@ -29,15 +29,15 @@ export default function ({ getPageObjects, getService }) {
 <h3>Inline HTML that should not be rendered as html</h3>
   `;
 
-  describe('visualize app', async () => {
+  describe('visualize app', () => {
     before(async function () {
       await PageObjects.visualize.navigateToNewVisualization();
       await PageObjects.visualize.clickMarkdownWidget();
       await PageObjects.visualize.setMarkdownTxt(markdown);
       await PageObjects.visualize.clickGo();
     });
 
-    describe('markdown vis', async () => {
+    describe('markdown vis', () => {
 
       it('should not have inspector enabled', async function () {
         await inspector.expectIsNotEnabled();

test/functional/apps/visualize/_pie_chart.js

@@ -28,7 +28,7 @@ export default function ({ getService, getPageObjects }) {
   const fromTime = '2015-09-19 06:31:44.000';
   const toTime = '2015-09-23 18:31:44.000';
 
-  describe('pie chart', async function () {
+  describe('pie chart', function () {
     const vizName1 = 'Visualization PieChart';
     before(async function () {
       log.debug('navigateToApp visualize');

test/functional/apps/visualize/_point_series_options.js

@@ -184,7 +184,7 @@ export default function ({ getService, getPageObjects }) {
       });
     });
 
-    describe('timezones', async function () {
+    describe('timezones', function () {
       const expectedLabels = [
         '2015-09-20 00:00',
         '2015-09-21 00:00',

test/functional/apps/visualize/_tile_map.js

@@ -209,7 +209,7 @@ export default function ({ getService, getPageObjects }) {
 
       });
 
-      describe('Only request data around extent of map option', async () => {
+      describe('Only request data around extent of map option', () => {
 
         it('when checked adds filters to aggregation', async () => {
           const vizName1 = 'Visualization TileMap';

test/functional/apps/visualize/_visualize_listing.js

@@ -25,7 +25,7 @@ export default function ({ getPageObjects }) {
   describe('visualize listing page', function describeIndexTests() {
     const vizName = 'Visualize Listing Test';
 
-    describe('create and delete', async function () {
+    describe('create and delete', function () {
 
       before(async function () {
         await PageObjects.visualize.gotoVisualizationLandingPage();

x-pack/legacy/plugins/canvas/server/routes/workpad.test.js

@@ -252,7 +252,7 @@ Array [
     });
   });
 
-  describe(`GET ${routePrefix}/find`, async () => {
+  it(`GET ${routePrefix}/find`, async () => {
     const request = {
       method: 'GET',
       url: `${routePrefix}/find?name=abc&page=2&perPage=10`,

x-pack/legacy/plugins/grokdebugger/server/lib/license_pre_routing_factory/__tests__/license_pre_routing_factory.js

@@ -45,7 +45,7 @@ describe('license_pre_routing_factory', () => {
       });
     });
 
-    describe('isAvailable is true', async () => {
+    describe('isAvailable is true', () => {
       beforeEach(() => {
         mockLicenseCheckResults = {
           isAvailable: true

x-pack/legacy/plugins/monitoring/server/lib/elasticsearch_settings/__tests__/find_reason.js

@@ -129,7 +129,7 @@ describe('Elasticsearch Settings Find Reason for No Data', () => {
     expect(result).to.eql({ found: false });
   });
 
-  describe('exporters', async () => {
+  describe('exporters', () => {
     it('should warn if all exporters are disabled', async () => {
       const input = {
         exporters: {

x-pack/legacy/plugins/reporting/export_types/csv/server/__tests__/execute_job.js

@@ -296,7 +296,7 @@ describe('CSV Execute Job', function () {
     });
   });
 
-  describe('Cells with formula values', async () => {
+  describe('Cells with formula values', () => {
     it('returns `csv_contains_formulas` when cells contain formulas', async function () {
       mockServer.config().get.withArgs('xpack.reporting.csv.checkForFormulas').returns(true);
       callWithRequestStub.onFirstCall().returns({

x-pack/test/api_integration/apis/management/index_lifecycle_management/nodes.js

@@ -33,7 +33,7 @@ export default function ({ getService }) {
       });
     });
 
-    describe('detail', async () => {
+    describe('detail', () => {
       it('should return the node stats when providing a custom node attribute', async () => {
         // Load the stats from ES js client
         const nodeStats = await getNodesStats();