Comments

testassets/InteropTestsGrpcWebWebsite/Tests/custom-environment.js

@@ -1,6 +1,7 @@
 const JestPuppeteerEnvironment = require("jest-environment-puppeteer").TestEnvironment;
 
-// expect-puppeteer requires jest's expect to be on the global object
+// expect-puppeteer requires jest's expect to be on the global object.
+// If the global object isn't populated then there is a null reference error in expect-puppeteer.
 global.expect = require('expect').expect;
 const expect = require('expect-puppeteer').expect;
 
@@ -9,8 +10,7 @@ class CustomEnvironment extends JestPuppeteerEnvironment {
     async setup() {
         await super.setup();
 
-        // Workaround puppeteer bug
-        // https://github.com/argos-ci/jest-puppeteer/issues/586
+        // Workaround puppeteer bug: https://github.com/argos-ci/jest-puppeteer/issues/586
         if (this.global.context.isIncognito === undefined) {
             this.global.context.isIncognito = () => false;
         }
@@ -20,10 +20,10 @@ class CustomEnvironment extends JestPuppeteerEnvironment {
         var page = this.global.page;
         await page.goto('http://localhost:8081', { waitUntil: 'networkidle0' });
 
-        // Wait for Blazor to finish loading
+        // Wait for Blazor to finish loading.
         await expect(page).toMatchTextContent('gRPC-Web interop tests');
 
-        // Get test names
+        // Get test names.
         this.global.__GRPC_WEB_TEST_NAMES__ = await page.evaluate(() => getTestNames('GrpcWeb'));
         this.global.__GRPC_WEB_TEXT_TEST_NAMES__ = await page.evaluate(() => getTestNames('GrpcWebText'));
    }