fix(cli/upload): Fix GitHub status check

packages/cli/src/autorun/autorun.js

@@ -1,5 +1,5 @@
 /**
- * @license Copyright 2019 Google Inc. All Rights Reserved.
+ * @license Copyright 2021 Google Inc. All Rights Reserved.
  * Licensed under the Apache License, Version 2.0 (the "License"); you may not use this file except in compliance with the License. You may obtain a copy of the License at http://www.apache.org/licenses/LICENSE-2.0
  * Unless required by applicable law or agreed to in writing, software distributed under the License is distributed on an "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the License for the specific language governing permissions and limitations under the License.
  */

packages/cli/src/upload/upload.js

@@ -247,8 +247,8 @@ async function runGithubStatusCheck(options, targetUrlMap) {
     for (const group of groupedResults) {
       const rawUrl = group[0].url;
       const urlLabel = getUrlLabelForGithub(rawUrl, options);
-      const failedResults = group.filter(result => result.level === 'error');
-      const warnResults = group.filter(result => result.level === 'warn');
+      const failedResults = group.filter(result => result.level === 'error' && !result.passed);
+      const warnResults = group.filter(result => result.level === 'warn' && !result.passed);
       const state = failedResults.length ? 'failure' : 'success';
       const context = getGitHubContext(urlLabel, options);
       const warningsLabel = warnResults.length ? ` with ${warnResults.length} warning(s)` : '';

packages/cli/test/autorun-github.test.js

@@ -0,0 +1,210 @@
+/**
+ * @license Copyright 2021 Google Inc. All Rights Reserved.
+ * Licensed under the Apache License, Version 2.0 (the "License"); you may not use this file except in compliance with the License. You may obtain a copy of the License at http://www.apache.org/licenses/LICENSE-2.0
+ * Unless required by applicable law or agreed to in writing, software distributed under the License is distributed on an "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the License for the specific language governing permissions and limitations under the License.
+ */
+'use strict';
+
+/* eslint-env jest */
+
+const path = require('path');
+const rimraf = require('rimraf');
+const fs = require('fs');
+
+const ApiClient = require('@lhci/utils/src/api-client.js');
+
+const {
+  createServer: createGithubServer,
+} = require('./fixtures/autorun-github/mock-github-server.js');
+const {runCLI, startServer, safeDeleteFile} = require('./test-utils.js');
+
+describe('Lighthouse CI autorun CLI with GitHub status check', () => {
+  const autorunDir = path.join(__dirname, 'fixtures/autorun-github');
+  const lighthouseciDir = path.join(autorunDir, '.lighthouseci');
+
+  let server;
+  let serverBaseUrl;
+  let apiClient;
+  let project;
+
+  let githubServer;
+
+  beforeAll(async () => {
+    if (fs.existsSync(lighthouseciDir)) rimraf.sync(lighthouseciDir);
+    githubServer = await createGithubServer();
+
+    server = await startServer();
+    serverBaseUrl = `http://localhost:${server.port}/`;
+
+    apiClient = new ApiClient({rootURL: serverBaseUrl});
+    project = await apiClient.createProject({name: 'Test'});
+  });
+
+  afterAll(async () => {
+    if (server) {
+      server.process.kill();
+      await safeDeleteFile(server.sqlFile);
+    }
+    if (githubServer) {
+      await githubServer.close();
+    }
+  });
+
+  it('should submit status check to GitHub', async () => {
+    const {stdout, stderr, status} = await runCLI(
+      [
+        'autorun',
+        '--upload.githubToken=githubToken',
+        `--upload.githubApiHost=http://localhost:${githubServer.port}`,
+        `--upload.serverBaseUrl=${serverBaseUrl}`,
+        `--upload.token=${project.token}`,
+      ],
+      {
+        cwd: autorunDir,
+        env: {
+          LHCI_BUILD_CONTEXT__CURRENT_HASH: 'hash',
+          LHCI_BUILD_CONTEXT__GITHUB_REPO_SLUG: 'GoogleChrome/lighthouse-ci',
+          LHCI_NO_LIGHTHOUSERC: undefined,
+        },
+      }
+    );
+
+    expect(stdout).toMatchInlineSnapshot(`
+      "✅  .lighthouseci/ directory writable
+      ✅  Configuration file found
+      ✅  Chrome installation found
+      ⚠️   GitHub token not set
+      Healthcheck passed!
+
+      Automatically determined ./build as \`staticDistDir\`.
+      Set it explicitly in lighthouserc.json if incorrect.
+
+      Started a web server on port XXXX...
+      Running Lighthouse 1 time(s) on http://localhost:XXXX/bad.html
+      Run #1...done.
+      Running Lighthouse 1 time(s) on http://localhost:XXXX/good.html
+      Run #1...done.
+      Done running Lighthouse!
+
+
+      Uploading median LHR of http://localhost:XXXX/bad.html...success!
+      Open the report at https://storage.googleapis.com/lighthouse-infrastructure.appspot.com/reports/XXXX-XXXX.report.html
+      Uploading median LHR of http://localhost:XXXX/good.html...success!
+      Open the report at https://storage.googleapis.com/lighthouse-infrastructure.appspot.com/reports/XXXX-XXXX.report.html
+      GitHub token found, attempting to set status...
+      GitHub accepted \\"failure\\" status for \\"lhci/url/bad.html\\".
+      GitHub accepted \\"success\\" status for \\"lhci/url/good.html\\".
+
+      "
+    `);
+    expect(stderr).toMatchInlineSnapshot(`
+      "Checking assertions against 2 URL(s), 2 total run(s)
+
+      1 result(s) for http://localhost:XXXX/bad.html :
+
+        X  viewport failure for minScore assertion
+             Does not have a \`<meta name=\\"viewport\\">\` tag with \`width\` or \`initial-scale\`
+             https://web.dev/viewport/
+
+              expected: >=0.9
+                 found: 0
+            all values: 0
+
+      1 result(s) for http://localhost:XXXX/good.html :
+
+        ✅  viewport passing for minScore assertion
+             Has a \`<meta name=\\"viewport\\">\` tag with \`width\` or \`initial-scale\`
+             https://web.dev/viewport/
+
+              expected: >=0.9
+                 found: 1
+            all values: 1
+
+      Assertion failed. Exiting with status code 1.
+      assert command failed. Exiting with status code 1.
+      "
+    `);
+    expect(status).toEqual(1);
+  }, 180000);
+
+  it('should submit status check to GitHub with invalid token', async () => {
+    const {stdout, stderr, status} = await runCLI(
+      [
+        'autorun',
+        '--upload.githubToken=invalidToken',
+        `--upload.githubApiHost=http://localhost:${githubServer.port}`,
+        `--upload.serverBaseUrl=${serverBaseUrl}`,
+        `--upload.token=${project.token}`,
+      ],
+      {
+        cwd: autorunDir,
+        env: {
+          LHCI_BUILD_CONTEXT__CURRENT_HASH: 'hash',
+          LHCI_BUILD_CONTEXT__GITHUB_REPO_SLUG: 'GoogleChrome/lighthouse-ci',
+          LHCI_NO_LIGHTHOUSERC: undefined,
+        },
+      }
+    );
+
+    expect(stdout).toMatchInlineSnapshot(`
+      "✅  .lighthouseci/ directory writable
+      ✅  Configuration file found
+      ✅  Chrome installation found
+      ⚠️   GitHub token not set
+      Healthcheck passed!
+
+      Automatically determined ./build as \`staticDistDir\`.
+      Set it explicitly in lighthouserc.json if incorrect.
+
+      Started a web server on port XXXX...
+      Running Lighthouse 1 time(s) on http://localhost:XXXX/bad.html
+      Run #1...done.
+      Running Lighthouse 1 time(s) on http://localhost:XXXX/good.html
+      Run #1...done.
+      Done running Lighthouse!
+
+
+      Uploading median LHR of http://localhost:XXXX/bad.html...success!
+      Open the report at https://storage.googleapis.com/lighthouse-infrastructure.appspot.com/reports/XXXX-XXXX.report.html
+      Uploading median LHR of http://localhost:XXXX/good.html...success!
+      Open the report at https://storage.googleapis.com/lighthouse-infrastructure.appspot.com/reports/XXXX-XXXX.report.html
+      GitHub token found, attempting to set status...
+      GitHub responded with 401
+
+
+      GitHub responded with 401
+
+
+
+      "
+    `);
+    expect(stderr).toMatchInlineSnapshot(`
+      "Checking assertions against 2 URL(s), 2 total run(s)
+
+      1 result(s) for http://localhost:XXXX/bad.html :
+
+        X  viewport failure for minScore assertion
+             Does not have a \`<meta name=\\"viewport\\">\` tag with \`width\` or \`initial-scale\`
+             https://web.dev/viewport/
+
+              expected: >=0.9
+                 found: 0
+            all values: 0
+
+      1 result(s) for http://localhost:XXXX/good.html :
+
+        ✅  viewport passing for minScore assertion
+             Has a \`<meta name=\\"viewport\\">\` tag with \`width\` or \`initial-scale\`
+             https://web.dev/viewport/
+
+              expected: >=0.9
+                 found: 1
+            all values: 1
+
+      Assertion failed. Exiting with status code 1.
+      assert command failed. Exiting with status code 1.
+      "
+    `);
+    expect(status).toEqual(1);
+  }, 180000);
+});

packages/cli/test/fixtures/autorun-github/build/bad.html

@@ -0,0 +1,10 @@
+<!DOCTYPE html>
+<html>
+  <head>
+    <meta charset="utf-8" />
+    <title>bad test page for autorun usage</title>
+  </head>
+  <body>
+    test
+  </body>
+</html>

packages/cli/test/fixtures/autorun-github/build/good.html

@@ -0,0 +1,11 @@
+<!DOCTYPE html>
+<html>
+  <head>
+    <meta charset="utf-8" />
+    <title>good test page for autorun usage</title>
+    <meta name="viewport" content="width=device-width, initial-scale=1">
+  </head>
+  <body>
+    test
+  </body>
+</html>

packages/cli/test/fixtures/autorun-github/lighthouserc.json

@@ -0,0 +1,20 @@
+{
+  "ci": {
+    "collect": {
+      "numberOfRuns": 1
+    },
+    "assert": {
+      "assertMatrix": [
+        {
+          "assertions": {
+            "viewport": "error"
+          }
+        }
+      ],
+      "includePassedAssertions": true
+    },
+    "upload": {
+      "target": "temporary-public-storage"
+    }
+  }
+}

packages/cli/test/fixtures/autorun-github/mock-github-server.js

@@ -0,0 +1,55 @@
+/**
+ * @license Copyright 2021 Google Inc. All Rights Reserved.
+ * Licensed under the Apache License, Version 2.0 (the "License"); you may not use this file except in compliance with the License. You may obtain a copy of the License at http://www.apache.org/licenses/LICENSE-2.0
+ * Unless required by applicable law or agreed to in writing, software distributed under the License is distributed on an "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the License for the specific language governing permissions and limitations under the License.
+ */
+'use strict';
+const createHttpServer = require('http').createServer;
+const express = require('express');
+
+/**
+ * @return {Promise<{app: Parameters<typeof createHttpServer>[1]}>}
+ */
+async function createApp() {
+  const app = express();
+
+  app.post('/repos/GoogleChrome/lighthouse-ci/statuses/hash', (req, res) => {
+    if (!req.headers.authorization || req.headers.authorization !== 'token githubToken') {
+      res.status(401);
+      res.json();
+      return;
+    }
+
+    setTimeout(() => {
+      res.status(201);
+      res.json({});
+    }, 100);
+  });
+
+  return {app};
+}
+
+/**
+ * @return {Promise<{port: number, close: () => Promise<void>}>}
+ */
+async function createServer() {
+  const {app} = await createApp();
+
+  return new Promise(resolve => {
+    const server = createHttpServer(app);
+    server.listen(0, () => {
+      const serverAddress = server.address();
+      const listenPort =
+        typeof serverAddress === 'string' || !serverAddress ? 0 : serverAddress.port;
+
+      resolve({
+        port: listenPort,
+        close: async () => {
+          await new Promise(r => server.close(r));
+        },
+      });
+    });
+  });
+}
+
+module.exports = {createApp, createServer};