Apply suggestions from code review

byronhulcher · Constance · orhantoy · web-flow · commit 5446dbedb1d9 · 2021-08-05T11:02:03.000-04:00
Co-authored-by: Constance &lt;constancecchen@users.noreply.github.com&gt;
Co-authored-by: Orhan Toy &lt;toyorhan@gmail.com&gt;
diff --git a/x-pack/plugins/enterprise_search/public/applications/app_search/components/crawler/components/crawler_status_banner.test.tsx b/x-pack/plugins/enterprise_search/public/applications/app_search/components/crawler/components/crawler_status_banner.test.tsx
@@ -45,7 +45,7 @@ describe('CrawlerStatusBanner', () => {
     CrawlerStatus.Suspending,
   ].forEach((status) => {
     describe(`when the status is ${status}`, () => {
-      it('renders an appropriate CrawlerStatusBanner', () => {
+      it('does not render a banner/callout', () => {
         setMockValues({
           mostRecentCrawlRequestStatus: status,
         });
diff --git a/x-pack/plugins/enterprise_search/public/applications/app_search/components/crawler/crawler_overview_logic.test.ts b/x-pack/plugins/enterprise_search/public/applications/app_search/components/crawler/crawler_overview_logic.test.ts
@@ -222,7 +222,7 @@ describe('CrawlerOverviewLogic', () => {
       });
 
       describe('on failure', () => {
-        it('it flashes an error message', async () => {
+        it('flashes an error message', async () => {
           http.post.mockReturnValueOnce(Promise.reject('error'));
 
           CrawlerOverviewLogic.actions.startCrawl();
@@ -250,7 +250,7 @@ describe('CrawlerOverviewLogic', () => {
       });
 
       describe('on failure', () => {
-        it('it flashes an error message', async () => {
+        it('flashes an error message', async () => {
           jest.spyOn(CrawlerOverviewLogic.actions, 'getLatestCrawlRequests');
           http.post.mockReturnValueOnce(Promise.reject('error'));
 
@@ -297,7 +297,7 @@ describe('CrawlerOverviewLogic', () => {
           CrawlerStatus.Running,
           CrawlerStatus.Canceling,
         ].forEach((status) => {
-          it(`Creates a new timeout for status ${status}`, async () => {
+          it(`creates a new timeout for status ${status}`, async () => {
             jest.spyOn(CrawlerOverviewLogic.actions, 'createNewTimeoutForCrawlRequests');
             http.get.mockReturnValueOnce(Promise.resolve([{ status }]));
 
@@ -323,7 +323,7 @@ describe('CrawlerOverviewLogic', () => {
             expect(CrawlerOverviewLogic.actions.fetchCrawlerData).toHaveBeenCalled();
           });
 
-          it(`can optionally supresses fetching data  ${status}`, async () => {
+          it(`optionally supresses fetching data for status ${status}`, async () => {
             jest.spyOn(CrawlerOverviewLogic.actions, 'clearTimeoutId');
             jest.spyOn(CrawlerOverviewLogic.actions, 'fetchCrawlerData');
             http.get.mockReturnValueOnce(Promise.resolve([{ status }]));
