Don't use expect within Promises (#5466) (#5473)

* Don't use expect within Promises (#5466)

If a call to expect fails within a Promise it will not
be resolved, and causing the test to crash.

The patch aligns the code similar to what is used by all
the other tests.

src/protocol.d.ts

@@ -1,5 +1,5 @@
 // This is generated from /utils/protocol-types-generator/index.js
-type binary = string;
+  type binary = string;
 
 declare module Protocol {
     export module Accessibility {
@@ -15498,5 +15498,6 @@ unsubscribes current runtime agent from Runtime.bindingCalled notifications.
       "Runtime.removeBinding": Runtime.removeBindingReturnValue;
       "Schema.getDomains": Schema.getDomainsReturnValue;
     }
-}
+  }
+
 export default Protocol;

test/dialog.spec.js

@@ -25,16 +25,22 @@ const {
 describe('Page.Events.Dialog', function () {
   setupTestBrowserHooks();
   setupTestPageAndContextHooks();
+
   it('should fire', async () => {
     const { page } = getTestState();
 
-    page.on('dialog', (dialog) => {
+    const onDialog = sinon.stub().callsFake((dialog) => {
+      dialog.accept();
+    });
+    page.on('dialog', onDialog);
+
+    await page.evaluate(() => alert('yo'));
+
+    expect(onDialog.callCount).toEqual(1);
+    const dialog = onDialog.firstCall.args[0];
     expect(dialog.type()).toBe('alert');
     expect(dialog.defaultValue()).toBe('');
     expect(dialog.message()).toBe('yo');
-      dialog.accept();
-    });
-    await page.evaluate(() => alert('yo'));
   });
 
   itFailsFirefox('should allow accepting prompts', async () => {

test/navigation.spec.js

@@ -163,15 +163,21 @@ describe('navigation', function () {
 
       // Make sure that network events do not emit 'undefined'.
       // @see https://crbug.com/750469
-      page.on('request', (request) => expect(request).toBeTruthy());
+      const requests = [];
-      page.on('requestfinished', (request) => expect(request).toBeTruthy());
+      page.on('request', (request) => requests.push('request'));
-      page.on('requestfailed', (request) => expect(request).toBeTruthy());
+      page.on('requestfinished', (request) => requests.push('requestfinished'));
+      page.on('requestfailed', (request) => requests.push('requestfailed'));
+
       let error = null;
       await page
         .goto(httpsServer.EMPTY_PAGE)
         .catch((error_) => (error = error_));
       if (isChrome) expect(error.message).toContain(EXPECTED_SSL_CERT_MESSAGE);
       else expect(error.message).toContain('SSL_ERROR_UNKNOWN');
+
+      expect(requests.length).toBe(2);
+      expect(requests[0]).toBe('request');
+      expect(requests[1]).toBe('requestfailed');
     });
     itFailsFirefox(
       'should fail when navigating to bad SSL after redirects',