Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

DGC-000: Adding Pa11y. #890

Merged
merged 4 commits into from Sep 10, 2020
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
7 changes: 6 additions & 1 deletion .gitignore
Expand Up @@ -136,7 +136,12 @@ drush/sites/.checksums

# GovCon 2018 Navigation Asset(s)
docroot/themes/custom/twentyeighteen/js/dist/navigation.js
docroot/themes/custom/twentyeighteen/npm-debug.log# Deprecation detector rules
docroot/themes/custom/twentyeighteen/npm-debug.log

# GovCon 2019 Navigation Asset(s)
docroot/themes/custom/twentynineteen/js/dist

# Deprecation detector rules
.rules

# BLT 11.2.0 update to support simulated deploys
Expand Down
2 changes: 1 addition & 1 deletion .node-version
@@ -1 +1 @@
8.9.1
10.x
3 changes: 1 addition & 2 deletions acquia-pipelines.yml
Expand Up @@ -18,9 +18,8 @@ events:
- composer validate --no-check-all --ansi
- composer install --ansi
- source ${BLT_DIR}/scripts/pipelines/setup_env
- nvm install v8.9.1
- nvm install v13.8.0
- npm install -g npm
- npm install --global yarn
- validate:
type: script
script:
Expand Down
3 changes: 3 additions & 0 deletions blt/blt.yml
Expand Up @@ -26,6 +26,9 @@ command-hooks:
frontend-assets:
dir: '${repo.root}'
command: ./blt/scripts/frontend-build.sh
frontend-test:
dir: ${repo.root}/docroot/themes/custom/twentynineteen
command: npm run tests
cm:
strategy: config-split
allow-overrides: true
24 changes: 24 additions & 0 deletions docroot/themes/custom/twentynineteen/.pa11yci.js
@@ -0,0 +1,24 @@
/**
* @file
* Pa11y config.
*/

const isCI = process.env.CI;
const baseURL = isCI ? 'http://127.0.0.1:8888' : 'http://drupalgovcon.lndo.site:8080';

// Add urls for a11y testing here.
const urls = [
'/',
];

module.exports = {
defaults: {
standard: 'WCAG2AA',
hideElements: ['svg'],
ignore: ['notice', 'warning'],
chromeLaunchConfig: {
args: ['--no-sandbox']
}
},
urls: urls.map(url => `${baseURL}${url}`)
};
200 changes: 0 additions & 200 deletions docroot/themes/custom/twentynineteen/js/dist/drupal-navigation.js

This file was deleted.