Skip to content

Commit

Permalink
core(config): faster category validation (#6445)
Browse files Browse the repository at this point in the history
  • Loading branch information
patrickhulce authored Nov 1, 2018
1 parent 6f05215 commit 76ec168
Showing 1 changed file with 6 additions and 1 deletion.
7 changes: 6 additions & 1 deletion lighthouse-core/config/config.js
Original file line number Diff line number Diff line change
Expand Up @@ -64,13 +64,18 @@ function validateCategories(categories, audits, groups) {
return;
}

const auditsKeyedById = new Map((audits || []).map(audit =>
/** @type {[string, LH.Config.AuditDefn]} */
([audit.implementation.meta.id, audit])
));

Object.keys(categories).forEach(categoryId => {
categories[categoryId].auditRefs.forEach((auditRef, index) => {
if (!auditRef.id) {
throw new Error(`missing an audit id at ${categoryId}[${index}]`);
}

const audit = audits && audits.find(a => a.implementation.meta.id === auditRef.id);
const audit = auditsKeyedById.get(auditRef.id);
if (!audit) {
throw new Error(`could not find ${auditRef.id} audit for category ${categoryId}`);
}
Expand Down

0 comments on commit 76ec168

Please sign in to comment.