From 35d5b214386ea31cdf814095e477e747c118c0bf Mon Sep 17 00:00:00 2001 From: Paul Irish Date: Wed, 29 Sep 2021 16:48:15 -0700 Subject: [PATCH] report(test): split out axe test (#13142) --- .../test/renderer/report-renderer-axe-test.js | 86 +++++++++++++++++++ report/test/renderer/report-renderer-test.js | 55 ------------ 2 files changed, 86 insertions(+), 55 deletions(-) create mode 100644 report/test/renderer/report-renderer-axe-test.js diff --git a/report/test/renderer/report-renderer-axe-test.js b/report/test/renderer/report-renderer-axe-test.js new file mode 100644 index 000000000000..3da22d20a188 --- /dev/null +++ b/report/test/renderer/report-renderer-axe-test.js @@ -0,0 +1,86 @@ +/** + * @license Copyright 2021 The Lighthouse Authors. 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 */ + +import jsdom from 'jsdom'; + +import {Util} from '../../renderer/util.js'; +import {DOM} from '../../renderer/dom.js'; +import {DetailsRenderer} from '../../renderer/details-renderer.js'; +import {CategoryRenderer} from '../../renderer/category-renderer.js'; +import {ReportRenderer} from '../../renderer/report-renderer.js'; +import sampleResultsOrig from '../../../lighthouse-core/test/results/sample_v2.json'; + +describe('ReportRendererAxe', () => { + describe('with aXe', () => { + let axe; + let renderer; + let sampleResults; + + beforeAll(async () => { + const {window} = new jsdom.JSDOM(); + const dom = new DOM(window.document); + const detailsRenderer = new DetailsRenderer(dom); + const categoryRenderer = new CategoryRenderer(dom, detailsRenderer); + renderer = new ReportRenderer(dom, categoryRenderer); + sampleResults = Util.prepareReportResult(sampleResultsOrig); + + // Needed by axe-core + // https://github.com/dequelabs/axe-core/blob/581c441c/doc/examples/jsdom/test/a11y.js#L24 + global.window = global.self = window; + global.Node = global.self.Node; + global.Element = global.self.Element; + + // axe-core must be imported after the global polyfills + axe = (await import('axe-core')).default; + }); + + afterAll(() => { + global.window = undefined; + global.Node = undefined; + global.Element = undefined; + }); + + it('renders without axe violations', () => { + const container = renderer._dom._document.createElement('main'); + const output = renderer.renderReport(sampleResults, container); + + renderer._dom._document.body.appendChild(container); + + const config = { + rules: { + // Reports may have duplicate ids + // https://github.com/GoogleChrome/lighthouse/issues/9432 + 'duplicate-id': {enabled: false}, + 'duplicate-id-aria': {enabled: false}, + 'landmark-no-duplicate-contentinfo': {enabled: false}, + // The following rules are disable for axe-core + jsdom compatibility + // https://github.com/dequelabs/axe-core/tree/b573b1c1/doc/examples/jest_react#to-run-the-example + 'color-contrast': {enabled: false}, + 'link-in-text-block': {enabled: false}, + // Report has empty links prior to i18n-ing. + 'link-name': {enabled: false}, + // May not be a real issue. https://github.com/dequelabs/axe-core/issues/2958 + 'nested-interactive': {enabled: false}, + }, + }; + + return new Promise(resolve => { + axe.run(output, config, (error, {violations}) => { + expect(error).toBeNull(); + expect(violations).toEqual([]); + resolve(); + }); + }); + // This test takes 40s on fast hardware, and 50-60s on GHA. + // https://github.com/dequelabs/axe-core/tree/b573b1c1/doc/examples/jest_react#timeout-issues + }, + /* timeout= */ 100 * 1000 + ); + }); +}); diff --git a/report/test/renderer/report-renderer-test.js b/report/test/renderer/report-renderer-test.js index 36dcdf1cdef5..3c24567105cd 100644 --- a/report/test/renderer/report-renderer-test.js +++ b/report/test/renderer/report-renderer-test.js @@ -233,59 +233,4 @@ describe('ReportRenderer', () => { .querySelectorAll('.lh-audit--notapplicable').length; assert.strictEqual(notApplicableCount, notApplicableElementCount); }); - - describe('axe-core', () => { - let axe; - - beforeAll(async () =>{ - // Needed by axe-core - // https://github.com/dequelabs/axe-core/blob/581c441c/doc/examples/jsdom/test/a11y.js#L24 - global.window = global.self; - global.Node = global.self.Node; - global.Element = global.self.Element; - - // axe-core must be imported after the global polyfills - axe = (await import('axe-core')).default; - }); - - afterAll(() => { - global.window = undefined; - global.Node = undefined; - global.Element = undefined; - }); - - it('renders without axe violations', () => { - const container = renderer._dom._document.createElement('main'); - const output = renderer.renderReport(sampleResults, container); - renderer._dom._document.body.appendChild(container); - - const config = { - rules: { - // Reports may have duplicate ids - // https://github.com/GoogleChrome/lighthouse/issues/9432 - 'duplicate-id': {enabled: false}, - 'duplicate-id-aria': {enabled: false}, - 'landmark-no-duplicate-contentinfo': {enabled: false}, - // The following rules are disable for axe-core + jsdom compatibility - // https://github.com/dequelabs/axe-core/tree/b573b1c1/doc/examples/jest_react#to-run-the-example - 'color-contrast': {enabled: false}, - 'link-in-text-block': {enabled: false}, - // Report has empty links prior to i18n-ing. - 'link-name': {enabled: false}, - // May not be a real issue. https://github.com/dequelabs/axe-core/issues/2958 - 'nested-interactive': {enabled: false}, - }, - }; - - return new Promise(resolve => { - axe.run(output, config, (error, {violations}) => { - expect(error).toBeNull(); - expect(violations).toEqual([]); - resolve(); - }); - }); - // This test takes 40s on fast hardware, and 50-60s on GHA. - // https://github.com/dequelabs/axe-core/tree/b573b1c1/doc/examples/jest_react#timeout-issues - }, /* timeout= */ 100 * 1000); - }); });